4

画像から支配的な色を取得するこの JavaScript コードを変換しようとしてきましたが、これまでのところ成功していません。colorCount&color変数でエラーが発生します。これらの変数に使用する適切で同等のデータ型がわかりません。これが私のコードです：

public string dominantColor(Bitmap img)
{
       int[] colorCount = new int[0];
       int maxCount = 0;
       string dominantColor = "";

       // data is an array of a series of 4 one-byte values representing the rgba values of each pixel
       Bitmap Bmp = new Bitmap(img);

       BitmapData BmpData = Bmp.LockBits(new Rectangle(0, 0, Bmp.Width, Bmp.Height), ImageLockMode.ReadOnly, Bmp.PixelFormat);
       byte[] data = new byte[BmpData.Stride * Bmp.Height];


       for (int i = 0; i < data.Length; i += 4) 
       {
            // ignore transparent pixels
            if (data[i+3] == 0)
                continue;

            string color = data[i] + "." + data[i+1] + "," + data[i+2];
            // ignore white
            if (color == "255,255,255")
               continue;

            if (colorCount[color] != 0)
               colorCount[color] = colorCount[color] + 1;
            else
               colorCount[color] = 0;

            // keep track of the color that appears the most times
            if (colorCount[color] > maxCount)
            {
                 maxCount = colorCount[color];
                 dominantColor = color.ToString;
            }
       }

       string rgb = dominantColor.Split(",");
       return rgb;   
}
4

1 に答える 1

4

コードの完全な管理バージョンを提供します。

static Color dominantColor(Bitmap img)
{
    Hashtable colorCount = new Hashtable();
    int maxCount = 0;
    Color dominantColor = Color.White;

    for (int i = 0; i < img.Width; i++)
    {
        for (int j = 0; j < img.Height; j++)
        {
            var color = img.GetPixel(i, j);

            if (color.A == 0)
                continue;

            // ignore white
            if (color.Equals(Color.White))
                continue;

            if (colorCount[color] != null)
                colorCount[color] = (int)colorCount[color] + 1;
            else
                colorCount.Add(color, 0);

            // keep track of the color that appears the most times
            if ((int)colorCount[color] > maxCount)
            {
                maxCount = (int)colorCount[color];
                dominantColor = color;
            }
        }
    }

    return dominantColor;
}

では、ここでの違いは何ですか？-配列の代わりにハッシュテーブルを使用します（配列の次元を再定義することはありません-JavaScriptから拡張可能なオブジェクトを使用する最良の方法はハッシュテーブルです）-すでに含まれている構造体Color（Alpha用に4バイト節約）を使用することを好みます、赤、緑、青）-私も比較を行い、この構造を返します（その後、好きなことを自由に行うことができます-JavaScriptでこれらの文字列を使用することは、ブラウザーがそのようなRGB（a ）文字列）

コードのもう1つの問題は、byte [] data = new byte [BmpData.Stride*Bmp.Height];を含む行です。-配列は作成および初期化されますが、データはありません（.NETは以前のすべてのデータを消去し、多くのゼロになります）。したがって、あなたはどこにもなりません。

私のバージョンの欠点は、それが実際に非常に小さいことです（これはあなたのロックビットが作用するところです）。必要に応じて、管理されていないバージョン（ロックビットと安全でないブロックを使用）を提供できます。パフォーマンスがあなたにとって非常に重要であるかどうか、そしてあなたが興味を持っているかどうかによって異なります！

于 2012-05-13T21:09:42.007 に答える