3

以下代码给了我一个 System.IO.IOException 消息“进程无法访问文件”。

private void UnPackLegacyStats()
{
  DirectoryInfo oDirectory;
  XmlDocument oStatsXml;

  //Get the directory
  oDirectory = new DirectoryInfo(msLegacyStatZipsPath);

  //Check if the directory exists
  if (oDirectory.Exists)
  {
    //Loop files
    foreach (FileInfo oFile in oDirectory.GetFiles())
    {
      //Check if file is a zip file
      if (C1ZipFile.IsZipFile(oFile.FullName))
      {
        //Open the zip file
        using (C1ZipFile oZipFile = new C1ZipFile(oFile.FullName, false))
        {
          //Check if the zip contains the stats
          if (oZipFile.Entries.Contains("Stats.xml"))
          {
            //Get the stats as a stream
            using (Stream oStatsStream = oZipFile.Entries["Stats.xml"].OpenReader())
            {
              //Load the stats as xml
              oStatsXml = new XmlDocument();
              oStatsXml.Load(oStatsStream);

              //Close the stream
              oStatsStream.Close();
            }

            //Loop hit elements
            foreach (XmlElement oHitElement in oStatsXml.SelectNodes("/*/hits"))
            {
              //Do stuff
            }                
          }

          //Close the file
          oZipFile.Close();
        }
      }

      //Delete the file
      oFile.Delete();
    }
  }
}

我正在努力查看文件仍然可以锁定的位置。所有可能持有文件句柄的对象都在使用块中并且被显式关闭。

它与使用 FileInfo 对象而不是静态 GetFiles 方法返回的字符串有关吗？

有任何想法吗？

4

5 回答 5

2

我在您的代码中没有看到问题，一切看起来都很好。要检查问题出在 C1ZipFile 我建议您从流初始化 zip，而不是从文件初始化，因此您明确关闭流：

//Open the zip file
using (Stream ZipStream = oFile.OpenRead())
using (C1ZipFile oZipFile = new C1ZipFile(ZipStream, false))
{
    // ...

其他几个建议：

  • 您不需要调用 Close() 方法，使用 (...)，删除它们。
  • 移动 xml 处理（循环命中元素）超大 zip 处理，即在 zip 文件关闭之后，因此您尽可能少地保持文件打开。
于 2009-07-06T18:35:53.547 回答
1

我只是在猜测：你确定 oZipFile.Close() 就足够了吗？也许您必须调用 oZipFile.Dispose() 或 oZipFile.Finalize() 以确保它确实释放了资源。

于 2009-07-06T16:10:16.630 回答
1

我假设您在 oFile.Delete 调用中遇到错误。我能够重现此错误。有趣的是，该错误仅在文件不是zip 文件时发生。这是您看到的行为吗？

当文件不是 zip 文件时，C1ZipFile.IsZipFile 调用似乎没有释放该文件。我能够通过使用 FileStream 而不是将文件路径作为字符串传递（IsZipFile 函数接受任何一种）来避免这个问题。

因此，对您的代码进行以下修改似乎有效：

if (oDirectory.Exists)
{
    //Loop files
    foreach (FileInfo oFile in oDirectory.GetFiles())
    {
        using (FileStream oStream = new FileStream(oFile.FullName, FileMode.Open))
        {
            //Check if file is a zip file
            if (C1ZipFile.IsZipFile(oStream))
            {
            // ...
            }
        }
        //Delete the file
        oFile.Delete();
    }
}

回答主题中的原始问题：我不知道是否可以知道是否可以在不尝试删除文件的情况下删除文件。您总是可以编写一个函数来尝试删除文件并在无法删除时捕获错误，然后返回一个布尔值，指示删除是否成功。

于 2009-07-06T19:25:09.093 回答
1

更何况它很可能没有被处置，每当您访问托管代码（流、文件等）之外的东西时，您必须处置它们。我学会了使用 Asp.NET 和图像文件的艰难方法，它会填满你的内存，让你的服务器崩溃等等。

于 2009-07-06T20:04:24.397 回答
1

为了完整起见，我提出了我的工作代码，因为更改来自多个来源。

private void UnPackLegacyStats()
{
  DirectoryInfo oDirectory;
  XmlDocument oStatsXml;

  //Get the directory
  oDirectory = new DirectoryInfo(msLegacyStatZipsPath);

  //Check if the directory exists
  if (oDirectory.Exists)
  {
    //Loop files
    foreach (FileInfo oFile in oDirectory.GetFiles())
    {
      //Set empty xml
      oStatsXml = null;

      //Load file into a stream
      using (Stream oFileStream = oFile.OpenRead())
      {
        //Check if file is a zip file
        if (C1ZipFile.IsZipFile(oFileStream))
        {
          //Open the zip file
          using (C1ZipFile oZipFile = new C1ZipFile(oFileStream, false))
          {
            //Check if the zip contains the stats
            if (oZipFile.Entries.Contains("Stats.xml"))
            {
              //Get the stats as a stream
              using (Stream oStatsStream = oZipFile.Entries["Stats.xml"].OpenReader())
              {
                //Load the stats as xml
                oStatsXml = new XmlDocument();
                oStatsXml.Load(oStatsStream);
              }
            }
          }
        }
      }

      //Check if we have stats
      if (oStatsXml != null)
      {
        //Process XML here
      }

      //Delete the file
      oFile.Delete();
    }
  }
}

我从中学到的主要教训是在调用代码的一个地方管理文件访问，而不是让其他组件管理自己的文件访问。当您想在其他组件完成任务后再次使用该文件时，这是最合适的。

尽管这需要更多的代码，但您可以清楚地看到流的处理位置（在使用结束时），而不是必须相信组件已正确处理流。

于 2009-07-07T08:32:54.377 回答