与C#中的BackgroundWorkerr一起使用时,File.Move抛出错误

时间:2012-07-20 19:51:42

标签: c# file-io backgroundworker

解决

  

我发现GetNewFolderNameBasedOnDate方法在内部没有关闭文件。我已修复该方法,现在正常工作

我正在尝试使用C#中的BackgroundWorker进程将所选文件从一个文件夹移动到另一个文件夹。这是我的DoWork()方法,用于确定是移动文件还是仅复制。我的File.Move()抛出一个异常,“进程无法访问该文件,因为它正被另一个进程使用”。我尝试了stackoverflow中的线程中提到的不同方法。

private void FileProcessor_DoWork(object sender, DoWorkEventArgs e)
{
    // Copy files
    long bytes = 0;
    string destSubFolder = String.Empty;
    string destFile = string.Empty;
    foreach (FileInfo file in oSettings.SourceFiles)
    {
        try
        {
            this.BeginInvoke(OnChange, new object[] { new UIProgress(file.Name, bytes, oSettings.MaxBytes) });

            destSubFolder = GetNewFolderNameBasedOnDate(file);

            //Create a new subfolder under the current active folder
            string newPath = Path.Combine(oSettings.TargetFolder, destSubFolder);
            // Create a new target folder, if necessary.
            if (!System.IO.Directory.Exists(newPath))
            {
                System.IO.Directory.CreateDirectory(newPath);
            }
            destFile = Path.Combine(oSettings.TargetFolder, destSubFolder, file.Name);

            if (chkDeleteSourceFiles.Checked)
            {
                FileInfo f = new FileInfo(file.FullName);

                if (f.Exists)
                {
                    File.Move(file.FullName, destFile);
                }
            }
            else
            {
                File.Copy(file.FullName, destFile, true);
            }

            //Thread.Sleep(2000);
        }
        catch (Exception ex)
        {
            UIError err = new UIError(ex, file.FullName);
            this.Invoke(OnError, new object[] { err });
            if (err.result == DialogResult.Cancel) break;
        }
        bytes += file.Length;
    }
}

我也尝试删除“RunWorkerCompleted”方法中的文件。但没有解决问题。尝试删除列表中的最后一个文件时失败。

private void FileProcessor_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    // Operation completed, update UI
    ChangeUI(false);

    foreach (FileInfo file in oSettings.SourceFiles)
    {
        File.Delete(file.FullName);
    }
}

GetNewFolderNameBasedOnDate()调用GetDateTaken()这是罪魁祸首。之前我没有使用FileStream对象但是使用Image myImage = Image.FromFile(filename);我不知道Image.FromFile会锁定文件。

private DateTime GetDateTaken(string fileName)
{
    try
    {
        using (FileStream fs = new FileStream(fileName, FileMode.Open, FileAccess.Read))
        {
            Image myImage = Image.FromStream(fs);
            PropertyItem propItem = myImage.GetPropertyItem(36867);
            DateTime dtaken;

            //Convert date taken metadata to a DateTime object
            string sdate = Encoding.UTF8.GetString(propItem.Value).Trim();
            string secondhalf = sdate.Substring(sdate.IndexOf(" "), (sdate.Length - sdate.IndexOf(" ")));
            string firsthalf = sdate.Substring(0, 10);
            firsthalf = firsthalf.Replace(":", "-");
            sdate = firsthalf + secondhalf;
            dtaken = DateTime.Parse(sdate);
            return dtaken;
        }
    }
    catch (Exception ex)
    {
        return DateTime.Now;
    }
}

2 个答案:

答案 0 :(得分:0)

不要创建新的FileInfo对象,而是保持简单并重复使用相同的对象。我怀疑问题是您在代码中有多个对同一文件的引用,这可以防止它被删除。尝试这样的东西来移动它:

if (chkDeleteSourceFiles.Checked)
{
    if (file.Exists)
    {
        file.MoveTo(destFile);
    }
}

答案 1 :(得分:0)

我的猜测是,它是对OnChange的BeginInvoke调用以及保留在该文件上的新UIProgress()对象。 UIProgress会打开文件吗?您可以尝试使用Invoke()并查看是否有帮助。