C# windows form avoid for loop for directory.getlogicaldrives ()

Question

0.00/5 (No votes)

See more:

Hello,
How to check in all PC hard drives, if a folder exists ?
I would like to control at one time and display a message that says yes, find in such disc.
In my example, the for condition displays as much message as it finds disk ...
How to search without a for loop ?
Thank you

What I have tried:

C#

<pre>          string[] str1 = Directory.GetLogicalDrives();
                    // for (initialization, condition, iteration)
                    for (int i = 0; i < str1.Length; ++i)
                    //  MessageBox.Show(str1[i]);

                    if (Directory.Exists(str1[i] + "Dossier1"))
                    {
                        MessageBox.Show("exists in : " + str1[i]);
                    }
                    else
                    {

                        MessageBox.Show("does not exist in : " + str1[i]);

                    }

Posted 27-Apr-19 2:17am

LSB71

Updated 27-Apr-19 20:31pm

v2

Add a Solution

Comments

BillWoodruff 30-Apr-19 4:46am

I suggest you read this: https://blogs.msdn.microsoft.com/jaredpar/2009/12/10/the-file-system-is-unpredictable/

And this: https://stackoverflow.com/a/11710169/133321

3 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Answer 1 · 2019-04-27T02:31:00

1) Save the answer until after the loop, then display a single "yes / no" message.
Or
2) Use Linq methods:

C#

string[] hasFolder = Directory.GetLogicalDrives().Where(d => Directory.Exists(Path.Combine(d, "Dossier"))).ToArray();

The hasFolder array contains all the drives that contain the folder you are seeking.

Gerry Schmitz · Answer 2 · 2019-04-27T02:32:00

Solution 2

C#

Using System.Text;

StringBuilder sb = new StringBuilder();

(start of loop)
...
sb.AppendLine("xxx exists / does not exist on zzz");
...
(end of loop)

MessageBox.Show( sb.ToString() );

Posted 27-Apr-19 2:32am

Gerry Schmitz

LSB71 · Answer 3 · 2019-04-27T20:31:00

C#

string[] drives = Directory.GetLogicalDrives().Where(d => Directory.Exists(Path.Combine(d, "Dossier"))).ToArray();
 string Résult = string.Join(", ", drives);
 if (Résult != "")
 {
     MessageBox.Show("exists");
 }
 else
 {
     MessageBox.Show("does not exist");
 }

Excellent, but can we do better!
thank you

C# windows form avoid for loop for directory.getlogicaldrives ()

3 solutions

Solution 1

Solution 2

Solution 3

Add your solution here

Preview 0