Method to get all files within folder and subfolders that will return a list

You can use Directory.GetFiles to replace your method.

 Directory.GetFiles(dirPath, "*", SearchOption.AllDirectories) 

Simply use this:

public static List<String> GetAllFiles(String directory) { return Directory.GetFiles(directory, "*.*", SearchOption.AllDirectories).ToList(); } 

And if you want every file, even extensionless ones:

public static List<String> GetAllFiles(String directory) { return Directory.GetFiles(directory, "*", SearchOption.AllDirectories).ToList(); } 

you can use something like this :

string [] filePaths = Directory.GetFiles(path, "*.*", SearchOption.AllDirectories); 

instead of using "." you can type the name of the file or just the type like "*.txt" also SearchOption.AllDirectories is to search in all subfolders you can change that if you only want one level more about how to use it on here

I am not sure of why you're adding the strings to files, which is declared as a field rather than a temporary variable. You could change the signature of DirSearch to:

private List<string> DirSearch(string sDir) 

And, after the catch block, add:

return files; 

Alternatively, you could create a temporary variable inside of your method and return it, which seems to me the approach you might desire. Otherwise, each time you call that method, the newly found strings will be added to the same list as before and you'll have duplicates.

This is for anyone that is trying to get a list of all files in a folder and its sub-folders and save it in a text document. Below is the full code including the “using” statements, “namespace”, “class”, “methods” etc. I tried commenting as much as possible throughout the code so you could understand what each part is doing. This will create a text document that contains a list of all files in all folders and sub-folders of any given root folder. After all, what good is a list (like in Console.WriteLine) if you can’t do something with it. Here I have created a folder on the C drive called “Folder1” and created a folder inside that one called “Folder2”. Next I filled folder2 with a bunch of files, folders and files and folders within those folders. This example code will get all the files and create a list in a text document and place that text document in Folder1. Caution: you shouldn’t save the text document to Folder2 (the folder you are reading from), that would be just bad practice. Always save it to another folder.
I hope this helps someone down the line.

using System; using System.IO; namespace ConsoleApplication4 { class Program { public static void Main(string[] args) { // Create a header for your text file string[] HeaderA = { "****** List of Files ******" }; System.IO.File.WriteAllLines(@"c:\Folder1\ListOfFiles.txt", HeaderA); // Get all files from a folder and all its sub-folders. Here we are getting all files in the folder // named "Folder2" that is in "Folder1" on the C: drive. Notice the use of the 'forward and back slash'. string[] arrayA = Directory.GetFiles(@"c:\Folder1/Folder2", "*.*", SearchOption.AllDirectories); { //Now that we have a list of files, write them to a text file. WriteAllLines(@"c:\Folder1\ListOfFiles.txt", arrayA); } // Now, append the header and list to the text file. using (System.IO.StreamWriter file = new System.IO.StreamWriter(@"c:\Folder1\ListOfFiles.txt")) { // First - call the header foreach (string line in HeaderA) { file.WriteLine(line); } file.WriteLine(); // This line just puts a blank space between the header and list of files. // Now, call teh list of files. foreach (string name in arrayA) { file.WriteLine(name); } } } // These are just the "throw new exception" calls that are needed when converting the array's to strings. // This one is for the Header. private static void WriteAllLines(string v, string file) { //throw new NotImplementedException(); } // And this one is for the list of files. private static void WriteAllLines(string v, string[] arrayA) { //throw new NotImplementedException(); } } } 

String[] allfiles = System.IO.Directory.GetFiles("path/to/dir", "*.*", System.IO.SearchOption.AllDirectories); 

A LINQ solution:

public IEnumerable<FileInfo> DirSearch(string sDir){ var directoryInfo = new DirectoryInfo(sDir); return directoryInfo.GetDirectories() .SelectMany(directory => DirSearch(directory.FullName)) .Union(directoryInfo.GetFiles()); } 

This will return FileInfos and not strings, because they are easier to work with I think.

Try this
class Program { static void Main(string[] args) {

 getfiles get = new getfiles(); List<string> files = get.GetAllFiles(@"D:\Rishi"); foreach(string f in files) { Console.WriteLine(f); } Console.Read(); } } class getfiles { public List<string> GetAllFiles(string sDirt) { List<string> files = new List<string>(); try { foreach (string file in Directory.GetFiles(sDirt)) { files.Add(file); } foreach (string fl in Directory.GetDirectories(sDirt)) { files.AddRange(GetAllFiles(fl)); } } catch (Exception ex) { Console.WriteLine(ex.Message); } return files; } }