优化“补丁”过程

Question

对于某些项目，我需要覆盖文件，但由于用户可能同时使用其他程序编辑此文件，因此我不经常在运行时保留流，而是将所有数据保存在字节数组中。保存我的程序时，应该只保存它编辑的区域，而不是整个文件。我已经（非常糟糕）编写了一个例程来这样做，但它预计会很慢，我不知道如何在这里提高性能。我实际上只是在循环整个数组，并不像看起来那么聪明：

    public Boolean Patch(string path)
    {
        FileStream fs = new FileStream(path, FileMode.Open);
        BinaryReader br = new BinaryReader(fs);
        BinaryWriter bw = new BinaryWriter(fs);
        if (fs.Length != this.rawdata.Length)
            throw new ArgumentException();
        for (int i = 0; i < this.rawdata.Length; ++i )
        {
            if (br.ReadByte() != rawdata[i])
            {
                fs.Position--;
                bw.Write(rawdata[i]);
            }
        }
        fs.Close();
        return true;
    }

Answer 1

您对硬盘驱动器（或任何其他流）的每次访问都是昂贵的。 将您的代码转换为使用下一个要读取的X（例如，1024）字节和要写入的Y（例如，1024）字节的缓存副本。

我并不完全理解你的代码应该做什么，但是说你想在流之间复制一个文件，你的函数应该是：

private const int BUFFER_SIZE = 1024;

void copy(BinaryReader inStream, BinaryWriter outStream)
{
    byte[] cache = new byte[BUFFER_SIZE];
    int readCount = 0;
    while ((readCount = inStream.Read(cache, 0, BUFFER_SIZE)) != 0)
    {
        outStream.Write(cache, 0, readCount);
    }
}

在这个例子中，BUFFER_SIZE 不能太小（这样批量读写会很有效），也不能太大——溢出你的内存。
在您的代码示例中，您每次都读取一个字节（即BUFFER_SIZE = 1），因此这会减慢您的应用程序的速度。

编辑：添加了您需要编写的代码：

public Boolean Patch(string path)
    {
        const int BUFFER_SIZE = 512;

        // VERY IMPORTANT: The throw operation will cause the stream to remain open the function returns.
        using (FileStream fs = new FileStream(path, FileMode.Open))
        {
            BinaryReader br = new BinaryReader(fs);
            BinaryWriter bw = new BinaryWriter(fs);
            if (fs.Length != this.rawdata.Length)
                throw new ArgumentException();
            byte[] cache = new byte[BUFFER_SIZE];
            int readCount = 0, location = 0;
            while ((readCount = br.Read(cache, 0, BUFFER_SIZE)) != 0) 
            {
                int changeLength = 0;

                for (int j = 0; j < readCount; j++)
                {
                    if (cache[j] != rawdata[j + location])
                    {
                        changeLength++;
                    }
                    else if (changeLength > 0)
                    {
                        fs.Position = location + j - changeLength;
                        bw.Write(rawdata, location + j - changeLength, changeLength);
                        fs.Position = location + j;

                        changeLength = 0;
                    }
                }

                location += readCount;
            } 

            return true;
        }
    }