C# 如何避免由于不同的原因使用foreach循环获取文件列表_C#

C# 如何避免由于不同的原因使用foreach循环获取文件列表

C# 如何避免由于不同的原因使用foreach循环获取文件列表,c#,C#,下面是我要做的：我有一个远程服务器（例如：svr01、svr02、svr03）。使用GetFileList读取目录，获取所有文件并与我的文件名匹配，然后复制到本地驱动器如果有匹配的文件，我也会将它们添加到XML文件中我试着做下面的事 class Program { static void Main(string[] args) { var getfiles = new fileshare.Program(); string realnam

下面是我要做的：

我有一个远程服务器（例如：svr01、svr02、svr03）。使用

GetFileList

读取目录，获取所有文件并与我的文件名匹配，然后复制到本地驱动器

如果有匹配的文件，我也会将它们添加到XML文件中

我试着做下面的事

 class Program
{
    static void Main(string[] args)
    {
        var getfiles = new fileshare.Program();

        string realname = "*main*";
        string Location = "SVR01";
        bool anymatch = false;
         foreach (var file in getfiles.GetFileList(realname,Location))
        {anymatch=true;}
         if (anymatch == true)
         { baseMeta(); }
        foreach (var file in getfiles.GetFileList(realname,Location))
        {getfiles.copytolocal(file.FullName); }

        }

    private FileInfo[] GetFileList(string pattern,string Location)
    {

        try
        {
            switch (Location)
            {
                case "SVR01":
                    {
                        var di = new DirectoryInfo(@"\\SVR01\Dev");
                        return di.GetFiles(pattern);
                    }
                case "SVR02":
                    {
                        var di = new DirectoryInfo(@"\\SVR02\Dev");
                        return di.GetFiles(pattern);
                    }
                case "SVR03":
                    {
                        var di = new DirectoryInfo(@"\\SVR03\Prod");
                        return di.GetFiles(pattern);
                    }
                default: throw new ArgumentOutOfRangeException();

            }
        }
        catch(Exception ex)
        { Console.Write(ex.ToString());
        return null;
        }

    }
    private void copytolocal(string filename)
    {
        string nameonly = Path.GetFileName(filename);
        File.Copy(filename,Path.Combine(@"c:\",nameonly),true);

    }

    private void baseMeta()
    {

                XmlWriter xmlWrite = XmlWriter.Create(@"c:\basexml");
                xmlWrite.WriteStartElement("job");
                xmlWrite.WriteElementString("Name", "test");
                xmlWrite.WriteElementString("time", DateTime);
                xmlWrite.Close();

    }
}

但这段代码让我担心，因为我做了两次相同的过程，请任何人指导我如何避免这种情况

 foreach (var file in getfiles.GetFileList(realname,Location))
 {
     anymatch=true;}
     if (anymatch == true)
     { 
          baseMeta(); 
     }
     foreach (var file in getfiles.GetFileList(realname,Location))
     {
         getfiles.copytolocal(file.FullName); 
     }
 }

即使我试图找出它是否匹配任何文件，我也会退出第一个foreach循环，生成basemeta（），然后转到下一个foreach循环来完成剩余的过程。

使用这个

var files = getfiles.GetFileList(realname, Location);
if (files.Length > 0)  
{ 
    baseMeta(); 
    foreach(var file in files)
    {
        getfiles.copytolocal(file.FullName); 
    }
}

用这个

var files = getfiles.GetFileList(realname, Location);
if (files.Length > 0)  
{ 
    baseMeta(); 
    foreach(var file in files)
    {
        getfiles.copytolocal(file.FullName); 
    }
}

试试这个：

Create方法检查文件是否存在，并在单个循环中执行所有操作。你的陈述不太清楚你什么时候抄袭。。使用您要复制或创建xml条目的条件

你的对手是什么？？如果要检查是否存在任何文件，请使用

var fileList = getfiles.GetFileList(realname,Location);

  if( fileList.Count() > 0)
  {
    baseMeta();      
  }
  foreach (var file in fileList)
  {

      // copy the file if match does not exist..
      getfiles.copytolocal(file.FullName);
  }

但Foreach在集合中循环（如果它有任何项）。因此，您不必关心文件的数量

如果你想按照你的代码在每个副本上做输入，那么为什么你需要检查任何匹配等。它会在每个文件副本上创建输入

foreach (var file in getfiles.GetFileList(realname,Location))
{
  baseMeta();      
  // copy the file 
  getfiles.copytolocal(file.FullName);
}