Dessiner un bitmap sur un autre produit une image déformée

Question

J'utilise une classe personnalisée pour écrire une image sur une autre image de plus grande taille. Voici le plein source code dans DotNetFiddle.

Ma personnalisation GetPixel() fonctionne correctement. Mais le SetPixel() suivant est incapable de produire la sortie appropriée. Probablement, il y a eu quelques problèmes avec le calcul des adresses. Mais, je suis incapable de le détecter.

public void SetPixel(int x, int y, Color color) 
    { 
     // Get color components count 
     int cCount = ColorDepth/8; 
     // Get start index of the specified pixel 
     int i = ((y * Width) + x) * cCount; 
     //int i = ((x * Width) + y) * cCount; 
     if (ColorDepth == 32) // For 32 bpp set Red, Green, Blue and Alpha 
     { 
      _imageData[i] = color.B; 
      _imageData[i + 1] = color.G; 
      _imageData[i + 2] = color.R; 
      _imageData[i + 3] = color.A; 
     } 

     if (ColorDepth == 24) // For 24 bpp set Red, Green and Blue 
     { 
      _imageData[i] = color.B; 
      _imageData[i + 1] = color.G; 
      _imageData[i + 2] = color.R; 
     } 
     if (ColorDepth == 8) 
     { 
      // For 8 bpp set color value (Red, Green and Blue values are the same) 
      _imageData[i] = color.B; 

      string str = string.Empty; 
     } 
    } 

C'est génère une image déformée:

.

P.S. Voici l'image d'entrée:

.

.

Answer 1

// Get start index of the specified pixel 
    int i = ((y * Width) + x) * cCount; 

Cela n'est pas correct, ni dans GetPixel ni dans SetPixel. Vous obtenez le biais parce que vous ignorez Stride. Quels sont le nombre d'octets dans une seule ligne de balayage de l'image. C'est un multiple de 4 à aligner les données de pixel dans la mémoire, aide le processeur à lire les données plus rapidement. Correction:

int i = y * Stride + x * cCount; 

Un autre bogue est caché dans votre code, les lignes de balayage sont stockées à l'envers. En d'autres termes, les données de la dernière ligne de balayage sont stockées en premier. Mais seulement si BitmapData.Height n'est pas négatif. Puisque ce bug se produit à la fois dans vos méthodes GetPixel et SetPixel, ils s'annulent mutuellement. Le code correct serait (Hauteur - y - 1) * Stride + x * cCount.

Ce code n'est pas plus rapide que Graphics.DrawImage(), la méthode que vous devriez toujours préférer.