c# internal string array method

Question

I want to ask a user to specify a folder path using a method and save this in the array, then allow that array to be used later on. The problem I have is defining a return type. How should I structure the method?

internal void selectFolderTxt(out string [] files)
{
    FolderBrowserDialog fbd = new FolderBrowserDialog();
    fbd.RootFolder = Environment.SpecialFolder.MyComputer;//This causes the folder to begin at the root folder or your documents
    if (fbd.ShowDialog() == DialogResult.OK)
    {
       string[] files = Directory.GetFiles(fbd.SelectedPath, "*.txt", SearchOption.AllDirectories);//change this to specify file type
    }
    else
    {
        // prevents crash
    }
}

P.S. I Only just started learning to use methods.

What is the return type you need? just replace here ==> internal [-->void<--] selectFolderTxt( — Marco Bong
– Marco Bong, Commented Feb 25, 2016 at 8:15

Community · Accepted Answer · 2017-05-23 12:23:24Z

1

I change the solution little bit.

Single Exist Point important

Why should a function have only one exit-point?

internal string[] selectFolderTxt() {
    string[] resultFiles = null;

    FolderBrowserDialog fbd = new FolderBrowserDialog();
    fbd.RootFolder = Environment.SpecialFolder.MyComputer;//This causes the folder to begin at the root folder or your documents
    if (fbd.ShowDialog() == DialogResult.OK)
    {
       resultFiles = Directory.GetFiles(fbd.SelectedPath, "*.txt", SearchOption.AllDirectories);//change this to specify file type
    }

    return resultFiles 
}

edited May 23, 2017 at 12:23

CommunityBot

11 silver badge

answered Feb 25, 2016 at 8:38

Sercan Timoçin

6785 silver badges11 bronze badges

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

1 Comment

Chong Ching Over a year ago

Don't forget the semi-colon.

user586399 · Accepted Answer · 2016-02-25 08:15:28Z

0

internal string[] selectFolderTxt() {

    FolderBrowserDialog fbd = new FolderBrowserDialog();
    fbd.RootFolder = Environment.SpecialFolder.MyComputer;//This causes the folder to begin at the root folder or your documents
    if (fbd.ShowDialog() == DialogResult.OK)
    {
       return Directory.GetFiles(fbd.SelectedPath, "*.txt", SearchOption.AllDirectories);//change this to specify file type
    }
    else
    {
    // prevents crash
       return null;
    }
}

Usage:

string[] files = selectFolderTxt();
if (files != null)
{
   // use files
}
else
{
   // the user cancelled dialog
}

answered Feb 25, 2016 at 8:15

user586399

2 Comments

Chong Ching Over a year ago

Thank you times a million this helped so much. I will re-think things in future. This allows for much more versatility.

Sercan Timoçin Over a year ago

It is solution but This method doesnt have Single Exit Point . You can take a look stackoverflow.com/questions/4838828/…

Leonid Malyshev · Accepted Answer · 2016-02-25 08:15:51Z

0

You should use boolean (internal boolean selectFolderTxt(out string [] files) ). True if OK, false if error or if the user canceled otherwise false.

answered Feb 25, 2016 at 8:15

Leonid Malyshev

4851 gold badge6 silver badges14 bronze badges

Collectives™ on Stack Overflow

c# internal string array method

3 Answers 3

1 Comment

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related