C# Draw image implementation

Question

Im пытается сделать вид Grahpics.DrawImage реализация с использованием небезопасный код и указатели конечно.

В этом случае я пытаюсь нарисовать небольшое растровое изображение на большей ширине (оба 32bppArgb).

Это мой код

private static unsafe void Draw(Bitmap bmp, Bitmap bmp2, int xPoint, int yPoint) 
    { 

     BitmapData bmData = bmp.LockBits(new Rectangle(0, 0, bmp.Width, bmp.Height), System.Drawing.Imaging.ImageLockMode.WriteOnly, bmp.PixelFormat); 
     BitmapData bmData2 = bmp2.LockBits(new Rectangle(0, 0, bmp2.Width, bmp2.Height), System.Drawing.Imaging.ImageLockMode.ReadOnly, bmp2.PixelFormat); 

     IntPtr scan0 = bmData.Scan0; 
     IntPtr scan02 = bmData2.Scan0; 

     int stride = bmData.Stride; 
     int stride2 = bmData2.Stride; 

     int nWidth = bmp2.Width; 
     int nHeight = bmp2.Height; 

     int sourceX = 0; 
     int sourceY = 0; 
     byte* p = (byte*)scan0.ToPointer(); 
     p += yPoint * stride + xPoint * 4; 
     byte* p2 = (byte*)scan02.ToPointer(); 
     p2 += sourceY * stride2 + sourceX * 4; 
     int bytes = nWidth * 4; 

     for (int y = 0; y < nHeight; y++) 
     { 

      for (int x = 0; x <nWidth; x++) 
      { 


       p[0] = p2[0]; 
       p[1] = p2[1]; 
       p[2] = p2[2]; 
       p[3] = p2[3]; 

      } 

      p += 4; 
      p2 += 4; 
     } 

     bmp.UnlockBits(bmData); 
     bmp2.UnlockBits(bmData2); 
    } 

это обновленный код

Answer 1

Я сделал некоторые изменения, чтобы сделать его работу:

• Использование ImageLockMode.WriteOnly в LockBits вызова для изображения, которое вам хочу писать.
• Не перемещайте p2 согласно xPoint и yPoint.

Я видел, что вы устанавливаете указатели внутри внешнего контура, поэтому вам не нужно будет перемещать указатели в конце цикла. Я рекомендую вам рассчитать начальные точки за пределами внешнего цикла и переместить указатели внутри него.

Я также рекомендую эти изменения, которые я сделал:

• Добавить чеки за пределы изображений, так что вы не случайно пытаются привлечь за пределы изображения.

• Сделайте способ void. Возврат растрового изображения предполагает, что он создает новое растровое изображение, а не заменяет одно из битового массива, которое вы передаете ему.

Я добавил параметр pixelBytes, чтобы сделать его пригодным для использования в разных форматах пикселей. (В основном потому, что случилось у JPEGs, не PNGs испытать с.)

Код:

private static unsafe void Draw(Bitmap bmp, Bitmap bmp2, int xPoint, int yPoint, int pixelBytes) { 

    BitmapData bmData = bmp.LockBits(new Rectangle(0, 0, bmp.Width, bmp.Height), System.Drawing.Imaging.ImageLockMode.WriteOnly, bmp.PixelFormat); 
    BitmapData bmData2 = bmp2.LockBits(new Rectangle(0, 0, bmp2.Width, bmp2.Height), System.Drawing.Imaging.ImageLockMode.ReadOnly, bmp2.PixelFormat); 

    IntPtr scan0 = bmData.Scan0; 
    IntPtr scan02 = bmData2.Scan0; 

    int stride = bmData.Stride; 
    int stride2 = bmData2.Stride; 

    int nWidth = bmp2.Width; 
    int nHeight = bmp2.Height; 

    int sourceX = 0; 
    int sourceY = 0; 

    if (xPoint < 0) { 
    sourceX = -xPoint; 
    nWidth -= sourceX; 
    xPoint = 0; 
    } 
    if (yPoint < 0) { 
    sourceY = -yPoint; 
    nHeight -= sourceY; 
    yPoint = 0; 
    } 
    if (xPoint + nWidth > bmp.Width) { 
    nWidth = bmp.Width - xPoint; 
    } 
    if (yPoint + nHeight > bmp.Height) { 
    nHeight = bmp.Height - yPoint; 
    } 

    if (nWidth > 0 && nHeight > 0) { 

    byte* p = (byte*)scan0.ToPointer(); 
    p += yPoint * stride + xPoint * pixelBytes; 
    byte* p2 = (byte*)scan02.ToPointer(); 
    p2 += sourceY * stride2 + sourceX * pixelBytes; 

    int bytes = nWidth * pixelBytes; 

    for (int y = 0; y < nHeight; y++) { 
     for (int x = 0; x < bytes; x++) { 
     p[0] = p2[0]; 
     p++; 
     p2++; 
     } 
     p += stride - nWidth * pixelBytes; 
     p2 += stride2 - nWidth * pixelBytes; 
    } 

    } 

    bmp.UnlockBits(bmData); 
    bmp2.UnlockBits(bmData2); 
}