从目录中选择随机文件

Question

，关于如何提高这种方法有什么建议？我目前使用它来从目录中选择一张壁纸

我知道你不应该使用arraylist了，但我无法想到altrnative 也我不知道如何过滤不只是一种类型文件（即jpg gif png）在目录info中。

任何建议或调整将是非常美妙

private string getrandomfile(string path) 
     { 
      ArrayList al = new ArrayList(); 
      DirectoryInfo di = new DirectoryInfo(path); 
      FileInfo[] rgFiles = di.GetFiles("*.*"); 
      foreach (FileInfo fi in rgFiles) 
      { 
       al.Add(fi.FullName); 
      } 

      Random r = new Random(); 
      int x = r.Next(0,al.Count); 

      return al[x].ToString(); 

     } 

感谢

崩溃

Answer 1

改为使用伪随机数生成器的一个实例。

// Use a class variable so that the RNG is only created once. 
private Random generator; 
private Random Generator 
{ 
    get 
    { 
     if (this.generator == null) 
     { 
      this.generator = new Random(); 
     } 
     return this.generator; 
    } 
} 
private string getrandomfile(string path) 
{ 
    string file = null; 
    if (!string.IsNullOrEmpty(path)) 
    { 
     var extensions = new string[] { ".png", ".jpg", ".gif" }; 
     try 
     { 
      var di = new DirectoryInfo(path); 
      var rgFiles = di.GetFiles("*.*") 
          .Where(f => extensions.Contains(f.Extension 
                   .ToLower()); 
      int fileCount = rgFiles.Count(); 
      if (fileCount > 0) 
      { 
       int x = this.Generator.Next(0, fileCount); 
       file = rgFiles.ElementAt(x).FullName; 
      } 
     } 
     // probably should only catch specific exceptions 
     // throwable by the above methods. 
     catch {} 
    } 
    return file; 
} 

Answer 2

你真的需要ArrayList中的一切，你应该能够消除它而直接使用数组一旦你产生了一个随机数。

此外，你应该检查路径是否有效......如果由用户指定的...

Answer 3

为什么不使用LINQ：

var files = Directory.GetFiles(path, "*.*").Where(s => Regex.Match(s, @"\.(jpg|gif|png)$").Success); 
string randFile = path + files.ToList()[r.Next(0, files.Count())]; 

Answer 4

像往常一样 - 存在于皮肤猫不止一种方法。我已经建立了tvanfosson（正确）的答案，不是因为这是'更'正确的;但是因为我认为这是一个有用的方法。

private static string getRandomFile(string path) 
{ 
    try 
    { 
     var extensions = new string[] { ".png", ".jpg", ".gif" }; 

     var di = new DirectoryInfo(path); 
     return (di.GetFiles("*.*") 
          .Where(f => extensions.Contains(f.Extension 
                   .ToLower())) 
          .OrderBy(f => Guid.NewGuid()) 
          .First()).FullName ;    
    } 
    catch { return ""; } 
} 

Answer 5

我做了一些改动

这里是我最后使用的代码，我剪下一些conditonals的监守他们并不真正的问题（如果没有文件，它会返回null反正也没必要测试两次）。我还纠正了一些小的语法错误，一位用户指出退回应该下移。

也是关于Random类，林不知道为什么它是坏的，以保持调用它，但我没有看到它的必要的，因为这将只运行一次，每次10〜15分钟。即使如此，如果找到文件，它也只会创建该类。

感谢大家的帮助（tvanfosson）

private string getrandomfile2(string path) 
    { 
     string file = null; 
     if (!string.IsNullOrEmpty(path)) 
     { 
      var extensions = new string[] { ".png", ".jpg", ".gif" }; 
      try 
      { 
       var di = new DirectoryInfo(path); 
       var rgFiles = di.GetFiles("*.*").Where(f => extensions.Contains(f.Extension.ToLower())); 
       Random R = new Random(); 
       file = rgFiles.ElementAt(R.Next(0,rgFiles.Count())).FullName; 
      } 
      // probably should only catch specific exceptions 
      // throwable by the above methods. 
      catch {} 
     } 
     return file; 
    }