Trying to delete all files in a folder exept 2 selected files in c#

Question

I'm just trying to write a simple program that clears out my "Path" folder exept for 2 .exe files preselected. The problem is that I only succeed in deleting the first one (file1.exe). What am I doing wrong? Forgive me if I'm using the logical operators but in this moment is like my head is about to explode...

string[] filePaths = Directory.GetFiles(Path); foreach (string filePath in filePaths) { var name = new FileInfo(filePath).Name; name = name.ToLower(); if (name != "file1.exe" || name != "file2.exe") { File.Delete(filePath); } }

Your problem is the premise behind the "I have two coins in my pocket that total $0.55 and one is not a nickel" riddle. — Scott Chamberlain
– Scott Chamberlain, Commented Oct 21, 2015 at 13:57

Habib · Accepted Answer · 2015-10-21 13:56:28Z

3

Use && instead of || for your condition.

if (name != "file1.exe" && name != "file2.exe")

You can also filter out the files with LINQ expression like:

var filePaths = Directory.GetFiles(Path) .Where(r=> !r.Equals("file1.exe", StringComparison.InvariantCultureIgnoreCase) && !r.Equals("file2.exe", StringComparison.InvariantCultureIgnoreCase));

and then you can do:

foreach (string filePath in filePaths) { File.Delete(filePath); }

edited Oct 21, 2015 at 13:56

answered Oct 21, 2015 at 13:52

Habib

224k30 gold badges420 silver badges446 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

Habib Over a year ago

It shouldn't. make sure your file is named as same as in the string for comparison.

Joe Over a year ago

When you do your string comparison, make sure you ignore case.

Joe Over a year ago

Also... add me as another vote for using && instead of ||

Vombato Over a year ago

Yes thank you, it was an uppercase letter that was doing the trick!

Joe Over a year ago

Great... I'm glad the Stack Overflow community was able to help you. Since you're new, I'd like to remind you to close out this question by accepting an answer.

Kallen · Accepted Answer · 2015-10-21 13:55:11Z

You are basically saying, if name is not equal to "file1.exe" OR name != "file2.exe" then delete a file path, the problem is you are using a OR logical operator so it only deletes one. Try using &&.

Collectives™ on Stack Overflow

Trying to delete all files in a folder exept 2 selected files in c#

2 Answers 2

5 Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

Comments

Related