C# 服务重命名文件！

Question

我有一个 Windows 服务，它获取带有元数据（FIDEF）的文件和相应的视频文件，并使用 XSLT 转换 XML（FIDEF）。

我获取 FIDEF 的文件目录列表，如果存在同名视频文件，它会对其进行翻译。这工作正常，但它是在计时器上每分钟搜索一次。我正在尝试处理相同文件名进入输入目录但已在输出目录中的情况。我只是将输出名称更改为 (copy) 因此，如果另一个文件输入，我应该得到 (copy)(copy).mov 但该服务不会以输出中已有的同一目录的文件名启动，它会工作一次并且然后似乎没有拾取任何新文件。

任何帮助都会很棒，因为我尝试了一些方法但没有好的结果。我相信它是重命名方法，但我已经放置了大部分代码，以防出现清理问题或其他问题。 （原谅有些名字只是尝试不同的东西）。

    private void getFileList()
    {
        //Get FILE LIST FROM Directory
        try
        {
            // Process Each String/File In Directory
            string result;
            //string filename;
             filepaths = null;
             filepaths = Directory.GetFiles(path, Filetype);

            foreach (string s in filepaths)
            {

                for (int i = 0; i < filepaths.Length; i++)
                {
                    //Result Returns Video Name
                    result = Path.GetFileNameWithoutExtension(filepaths[i]);
                    FileInfo f = new FileInfo(filepaths[i]);

                    PreformTranslation(f, outputPath + result , result);


                }
            }

        }
        catch (Exception e)
        {
            EventLog.WriteEntry("Error " + e);
        }


    }

    private void MoveVideoFiles(String Input, String Output)
    {
        File.Move(Input, Output);

    }
    private string GetUniqueName(string name)
    {


         //Original Filename
        String ValidName = name;
        //remove FIDEF from filename
        String Justname1 = Path.GetFileNameWithoutExtension(name);
        //get .mov extension 
        String Extension2 = Path.GetExtension(Justname1);
        //get filename with NO extensions
        String Justname = Path.GetFileNameWithoutExtension(Justname1);
        //get .Fidef
        String Extension = Path.GetExtension(name);
        int cnt = 0;

        //string[] FileName = Justname.Split('(');
        //string Name = FileName[0];

        while (File.Exists(ValidName)==true)
        {
            ValidName = outputPath + Justname + "(Copy)" + Extension2 + Extension;
            cnt++;

        }
        return ValidName;
    }
    private string getMovFile(string name)
    {
        String ValidName = name;
        String Ext = Path.GetExtension(name);
        String JustName = Path.GetFileNameWithoutExtension(name);

        while(File.Exists(ValidName))
        {
            ValidName = outputPath + JustName + "(Copy)" + Ext;
        }
        return ValidName;
    }



    //Preforms the translation requires XSL & FIDEF name.
    private void PreformTranslation(FileInfo FileName, String OutputFileName , String result)
    {

        string FidefName = OutputFileName + ".FIDEF";
        String CopyName;
        String copyVidName = outputPath + result;

            XslCompiledTransform myXslTransform;
            myXslTransform = new XslCompiledTransform();
            try
            {
                myXslTransform.Load(XSLname);

            }
            catch 
            {
                EventLog.WriteEntry("Error in loading XSL");
            }
            try
            {   //only process FIDEF's with corresponding Video file
                if (AllFidef == "no")
                {
                    //Check if video exists if yes,
                    if (File.Exists(path + result))
                    {
                        //Check for FIDEF File Already Existing in the Output Directory. 
                        if (File.Exists(FidefName))
                        {
                            //Get unique name
                            CopyName = GetUniqueName(FidefName);
                            copyVidName= getMovFile(copyVidName);


                            //Translate and create new FIDEF. 

                            //double checking the file is here
                            if (File.Exists(outputPath + result))
                            {
                                myXslTransform.Transform(FileName.ToString(), CopyName);
                                File.Delete(FileName.ToString());
                                MoveVideoFiles(path + result, copyVidName);

                            }
                            ////Move Video file with Corresponding Name. 

                        }


                        else
                        {  //If no duplicate file exsists in Directory just move. 
                            myXslTransform.Transform(FileName.ToString(), OutputFileName + ".FIDEF");
                            MoveVideoFiles(path + result, outputPath + result);
                        }
                    }

                    }
                else
                {
                    //Must have FIDEF extension
                    //Processes All FIDEFS and moves any video files if found. 
                    myXslTransform.Transform(FileName.ToString(), OutputFileName + ".FIDEF"); 
                    if (File.Exists(path + result))
                    {
                        MoveVideoFiles(path + result, outputPath + result);
                    }


                }
            }
            catch (Exception e)
            {
                EventLog.WriteEntry("Error Transforming " + "FILENAME = " + FileName.ToString()
                    + " OUTPUT_FILENAME = " + OutputFileName + "\r\n" +"\r\n"+  e);

            }

        }

Answer 1

你的代码有很多错误。对于初学者来说，getFileList 有不需要的内部 for 循环。摆脱它。您的 foreach 循环有 s，它可以替换 for 循环中的 filepaths[i]。另外，不要使用 outputPath + result 来创建文件路径。请改用 Path.Combine(outputPath, result) ，因为 Path.Combine 会为您处理目录字符。另外，您需要为 getFileList 起一个更好的名称，因为这根本不是该方法的作用。不要让你的方法名称成为骗子。

我只想摆脱MoveVideoFiles。编译器也可能如此。

GetUniqueName 仅当您的文件名采用 name.mov.fidef 形式（我假设是这样）时才有效。不过，您确实需要更好的变量名称，否则以后它将成为维护的噩梦。我会去掉 while 循环条件中的 == true ，但这是可选的。 while 内的赋值是文件被覆盖的原因。你总是生成相同的名称（something(Copy).mov.fidef），据我所知，如果文件存在，我认为你会永远循环堆栈。您需要修复该循环以生成新名称（并且不要忘记Path.Combine）。也许是这样的（注意这是未经测试的）：

int copyCount = 0;
while (File.Exists(ValidName))
{
    const string CopyName = "(Copy)";
    string copyString = copyCount == 0 ? CopyName : (CopyName + "(" + copyCount + ")");
    string tempName = Justname + copyString + Extension2 + Extension;
    ValidName = Path.Combine(outputPath, tempName);
    copyCount++;
}

这会为第一个副本生成 something(Copy).mov.fidef，something(Copy)(2).mov.fidef > 第二个，依此类推。也许不是你想要的，但你可以做出调整。

此时你有很多事情要做。 getMovFile 看起来好像可以以与 GetUniqueName 相同的方式使用工作。你会明白的。祝你好运。