优化“补丁”流程

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;
        }
    }