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

优化“补丁”流程

1 个答案: