0 
public byte[] GetFile(string filename)
{
    FileStream aStream = File.Open(filename, FileMode.Open, FileAccess.Read);
    BinaryReader binReader = new BinaryReader(aStream);
    binReader.BaseStream.Position = 0;
    byte[] binFile = binReader.ReadBytes(Convert.ToInt32(binReader.BaseStream.Length));
    binReader.Close();
    return binFile;
}

我对许多文件路径运行此方法，问题是每当无法使用 File.Open 访问文件时（因为它被另一个进程使用）我得到：

'aStream.Position' threw an exception of type 'System.ObjectDisposedException'

在以下行：

binReader.BaseStream.Position = 0;

我很少得到

{System.IO.IOException: The process can not access the file '\folder\file.txt' because it is being used by another process.}

这是我想要的例外。那么为什么对象大部分时间都被处理掉了呢？注意：我首先在 using 语句中添加了 FileStream 行，但删除了它，因为我认为这可能已经释放了该对象。但问题依然存在。

编辑：使用 Compact Framework，它没有ReadAllBytes.

4

3 回答 3

2

也许那部分时间你在使用你的文件时FileStream抛出IOException，而在其他时候，你可能会得到 ，ObjectDisposedException因为你的数组没有初始化。

显然，我无法检验这个理论。

看看你是否可以复制粘贴这个效果很好：

public byte[] GetFile(string filename)
{
  byte[] binFile = null;
  try
  {
    using (var aStream = File.Open(filename, FileMode.Open, FileAccess.Read))
    {
      BinaryReader binReader = new BinaryReader(aStream);
      binFile = new byte[binReader.BaseStream.Length];
      binReader.BaseStream.Position = 0; // <= this step should not be necessary
      binFile = binReader.ReadBytes(binReader.BaseStream.Length);
      binReader.Close();
    }
  } catch (IOException err) {
    // file is being used by another process.
  } catch (ObjectDisposedException err) {
    // I am guessing you would never see this because your binFile is not disposed
  }
  return binFile;
}

一定要检查空返回变量！

编辑：

我写了（我认为是）一个更简单的版本。我测试了它，它似乎工作正常。我也更喜欢Read()重载ReadBytes()，因为我知道有多少数据被拉入。

首先，是为 my Pictures 文件夹中的每个图像调用该方法的测试函数：

public void Test() {
  DirectoryInfo dir = new DirectoryInfo(Environment.GetFolderPath(Environment.SpecialFolder.Personal));
  foreach (var subDir in dir.GetDirectories()) {
    if (-1 < subDir.Name.ToLower().IndexOf("pictures")) {
      foreach (var file in subDir.GetFiles()) {
        byte[] data = GetFile(file.FullName);
        if (data != null) {
          Console.WriteLine(data.Length);
        }
      }
    }
  }
}

public byte[] GetFile(string filename) {
  byte[] result = null;
  try {
    if (File.Exists(filename)) {
      int len = 0;
      FileInfo file = new FileInfo(filename);
      byte[] data = new byte[file.Length];
      using (BinaryReader br = new BinaryReader(file.Open(FileMode.Open, FileAccess.Read))) {
        len = br.Read(data, 0, data.Length);
        br.Close();
      }
      if (0 < len) {
        if (len == data.Length) {
          return data;
        } else {
          // this section of code was never triggered in my tests;
          // however, it is good to keep it as a backup.
          byte[] dat2 = new byte[len];
          Array.Copy(data, dat2, len);
          return dat2;
        }
      }
    }
  } catch (IOException err) {
    // file is being used by another process.
  } catch (ObjectDisposedException err) {
    // I am guessing you would never see this because your binFile is not disposed
  }
  return result;
}

我看不出这些不起作用的任何原因 - 除非你有int溢出。

于 2012-04-18T13:18:21.137 回答
1

你为什么不简单地使用

public byte[] GetFile(string filename)
{
    try { return File.ReadAllBytes(filename); }
    catch { return null; }
}

只是为了好玩，你甚至可以定义一个扩展方法

public static class Extensions
{
    public static byte[] GetFile(this string filename)
    {
        try { return File.ReadAllBytes(filename); }
        catch { return null; }
    }
}

所以你可以这样做byte[] myfile = filename.GetFile();。
请记住，在继续之前，您必须检查 return 不为空：

if (myfile != null)
{
    // Do what you need
}
于 2012-04-17T09:58:58.080 回答
1

只需使用这个：

byte[] contents = File.ReadAllBytes(filename);
于 2012-04-17T10:03:55.097 回答