2012-01-25 8 views
0

イメージの行または列をチェックし、すべての白いピクセルが含まれている場合は、その行または列をトリミングします。 理由はわかりませんが、このコードスニペットを実行すると、TrimLeftのimg.Widthは1で、その前に1が減ります。 実際の画像幅は175です。私は最終的にArgumentExceptionで終わります。私は右サイドをトリミングするための同様の方法があり、それはうまく動作します。それは非常に非効率的になるようにC#ビットマップの問題

class ImageHandler 
{ 

    public Bitmap img; 
    public List<int[]> pixels = new List<int[]>(); 

    public ImageHandler(String path) 
    { 
     img = new Bitmap(path); 
     GetPixels(); 
    } 

    public void TrimLeft() 
    { 
     while (CheckColIfWhite(0, 0)) 
     { 
      Rectangle cropBox = new Rectangle(1, 0, (img.Width-1), img.Height); 
      Bitmap cropdImg = CropImage(img, cropBox); 
      img = cropdImg; 
     } 
    } 

    public bool CheckColIfWhite(int colx, int starty) 
    { 
     bool allPixelsWhite = false; 
     int whitePixels = 0; 

     for (int y = starty; y < img.Height; y++) 
     { 
      if (pixels[y][colx] >= 200) { whitePixels++; } 
      else { return false; } 
      if (whitePixels == img.Height) { allPixelsWhite = true; } 
     } 
     return allPixelsWhite; 
    } 

    public void GetPixels() 
    { 
     for (int y = 0; y < img.Height; y++) 
     { 
      int[] line = new int[img.Width]; 
      for (int x = 0; x < img.Width; x++) 
      { 
       line[x] = (int) img.GetPixel(x, y).R; 
      } 
      pixels.Add(line); 
     } 
    } 

    public Bitmap CropImage(Bitmap tImg, Rectangle area) 
    { 
     Bitmap bmp = new Bitmap(tImg); 
     Bitmap bmpCrop = bmp.Clone(area, bmp.PixelFormat); 
     return bmpCrop; 
    } 
} 

答えて

1

あなたの方法は思える - 画像は、175個のピクセルの幅、および完全に白であるならば、あなたは(おそらく)の前に、それの175のコピーを作成しようとして作成しようとすると失敗しています0ピクセル幅の画像。

白以外の列が見つかるまで各列を順番に調べてから、その時点で1つのクロップを実行してみてください。未テストコード、そして、できれば明らかに他の変更を伴う:

public Bitmap CropImage (Bitmap image) 
{ 
    int top = 0; 
    int bottom = image.Height-1; 
    int left = 0; 
    int right = image.Width-1; 
    while(left < right && CheckColIfWhite(image,left)) 
     left++; 
    if(left==right) return null; //Entirely white 
    while(CheckColIfWhite(image,right)) //Because left stopped, we know right will also 
     right--; 
    while(CheckRowIfWhite(image,top)) 
     top++; 
    while(CheckRowIfWhite(image,bottom)) 
     bottom--; 
    return CropImage(image,new Rectangle(left,top,right-left+1,bottom-top+1)); 
} 

(例:私は今、私はまた、画像を撮影するためにCheckColIfWhiteCheckRowIfWhiteを変更した、と1つのパラメータが常に固定されていることを前提とし、周辺の画像を渡しています

public bool CheckColIfWhite(Bitmap image,int colx) 
{ 
    for (int y = 0; y < image.Height; y++) 
    { 
     if (image.GetPixel(colx,y).R < 200) 
      return false; 
    } 
    return true; 
} 
+0

これはVだった:0)


また

、ないなぜあなたは、事前に画素配列を抽出しているので、私はあまりにも私の再書かれCheckColIfWhiteを入れていることを確認でええ、ありがとうございました。私の方法は非効率的であることは分かっていましたが、それについてもっと良い方法は見当たりませんでした。私はもともと、ファイルの複数の読み込みを保存するために、画像からすべてのピクセルを取得しました。 – sjensen85