这个解码器出了什么问题？ （在C＃中） - 帮助填充额外的字节

Question

我在这里有一个用C＃编写的PCX解码器，用于返回一个IntPtr，它指向一个未压缩字节数组（PCX文件使用RLE压缩，但我的解码器应该能够解决这个问题）。我已经从文件中读取了宽度，尺寸和调色板，图像将文件渲染为大多数图像的位图，但有些图像无法正确渲染。图像在那里，颜色也是如此，但实际的位图看起来像是对角切片4或5次并重新排列。我已经检查了图像中的平面数量，而且bpp也很好。

我认为我的代码有问题，所以如果有人能看到错误，请告诉我。

---

编辑2：

正如Guffa指出的那样，我没有处理任何填充。有人能指出我正确的方向吗？

---

代码（对不起，这里有很多，但它是实际的像素处理器）：

IntPtr pBits;
Boolean bRepeat;
Int32 RepeatCount;
Byte ReadByte;
Int32 Row = 0;
Int32 Col = 0;

Byte[] PCXData = new Byte[BytesPerScanline * ScanLines]; //BytesPerScanline * ScanLines);

BinaryReader r = new BinaryReader(file);
r.BaseStream.Seek(128, SeekOrigin.Begin);

while (Row < ScanLines)
{
    ReadByte = r.ReadByte();
    bRepeat = (0xc0 == (ReadByte & 0xC0));
    RepeatCount = (ReadByte & 0x3f);

    if (!(Col >= BytesPerScanline))
    {
        if (bRepeat)
        {
            ReadByte = r.ReadByte();
            while (RepeatCount > 0)
            {
                PCXData[(Row * BytesPerScanline) + Col] = ReadByte;
                RepeatCount -= 1;
                Col += 1;
            }
        }
        else
        {
            PCXData[(Row * BytesPerScanline) + Col] = ReadByte;
            Col += 1;
        }
    }

    if (Col >= BytesPerScanline)
    {
        Col = 0;
        Row += 1;
    }
}

pBits = System.Runtime.InteropServices.Marshal.AllocHGlobal(PCXData.Length);
System.Runtime.InteropServices.Marshal.Copy(PCXData, 0, pBits, PCXData.Length);

return pBits;

Answer 1

首先，您没有正确处理非托管资源（例如BinaryReader）。 完成后调用r.Dispose()，或将其包装在using块中，如下所示：

using(BinaryReader r = new BinaryReader(file))
{
  ...
}

并且总是为实现IDisposable的任何对象执行此操作。