C言語 ラベリングについて

Question

##質問内容
4近傍のラベリングを実装したいと思っているのですが，0と1しか表示されません．

本来は，

0,1,2,3,4,・・・

といったように，順次，ラベル数を足していきたいです．

##コード

//-------------------------------------------
//r_in,g_in,b_in・・・入力画像
//r_out_labeling,g_out_labeling,b_out_labeling・・・出力画像
//
//-------------------------------------------
//ラベリングを行うための関数(https://gist.github.com/masazdream/8630805)
void labeling(unsigned char r_in[Y_SIZE][X_SIZE], unsigned char g_in[Y_SIZE][X_SIZE], unsigned char b_in[Y_SIZE][X_SIZE], unsigned char r_out_labeling[Y_SIZE][X_SIZE], unsigned char g_out_labeling[Y_SIZE][X_SIZE], unsigned char b_out_labeling[Y_SIZE][X_SIZE])
{
	int i, j;

	int up_r, up_g, up_b;
	int left_r, left_g, left_b;

	int up_r_label, up_g_label, up_b_label;
	int left_r_label, left_g_label, left_b_label;

	unsigned char r_out[Y_SIZE][X_SIZE];
	unsigned char g_out[Y_SIZE][X_SIZE];
	unsigned char b_out[Y_SIZE][X_SIZE];


	//バッファを作成
	for (j = 0; j < Y_SIZE; j++)
	{
		for (i = 0; i < X_SIZE; i++)
		{
			//出力用のrgb値を用意
			r_out[j][i] = r_in[j][i];
			g_out[j][i] = g_in[j][i];
			b_out[j][i] = b_in[j][i];
		}
	}

	for (j = 0; j < Y_SIZE; j++)
	{
		for (i = 0; i < X_SIZE; i++)
		{
			//上の画素（元画像）
			up_r = r_out[j - 1][i];
			up_g = g_out[j - 1][i];
			up_b = b_out[j - 1][i];


			//左の画素（元画像）
			left_r = r_out[j][i - 1];
			left_g = g_out[j][i - 1];
			left_b = b_out[j][i - 1];
		}
	}


	for (j = 0; j < Y_SIZE; j++)
	{
		for (i = 0; i < X_SIZE; i++)
		{

			//ラベル用
			r_out_labeling[j][i] = r_out[j][i];
			g_out_labeling[j][i] = g_out[j][i];
			b_out_labeling[j][i] = b_out[j][i];


			//(i,j)が127以下の時・・・
			if (r_out[j][i] < 127 && g_out[j][i] < 127 && b_out[j][i] < 127)
			{
				//ラベルの値を0にする
				r_out_labeling[j][i] = 0;
				g_out_labeling[j][i] = 0;
				b_out_labeling[j][i] = 0;
			}
			//(i,j)が127より大きい時・・・
			else if (r_out[j][i] >= 127 && g_out[j][i] >= 127 && b_out[j][i] >= 127)
			{
				//ラベルの値を1にする
				r_out_labeling[j][i] = 1;
				g_out_labeling[j][i] = 1;
				b_out_labeling[j][i] = 1;
			}
		}
	}

	for (j = 0; j < Y_SIZE; j++)
	{
		for (i = 0; i < X_SIZE; i++)
		{
			//上の画素（ラベル）
			up_r_label = r_out_labeling[j - 1][i];
			up_g_label = g_out_labeling[j - 1][i];
			up_b_label = b_out_labeling[j - 1][i];


			//左の画素（ラベル）
			left_r_label = r_out_labeling[j][i - 1];
			left_g_label = g_out_labeling[j][i - 1];
			left_b_label = b_out_labeling[j][i - 1];


			//注目画素の上と左の値が0の時・・・
			if (r_out_labeling[j][i] == 0 && g_out_labeling[j][i] == 0 && b_out_labeling[j][i] == 0)
			{
				//続ける
				continue;
			}

			//注目画素のラベルが1以上なら・・・
			else if (r_out_labeling[j][i] >= 1 && g_out_labeling[j][i] >= 1 && b_out_labeling[j][i] >= 1)
			{
				//上と左のラベルが0の時・・・
				if ((up_r_label == 0 && up_g_label == 0 && up_b_label == 0) && (left_r_label == 0 && left_g_label == 0 && left_b_label == 0))
				{
					//ラベルに+1する
					r_out_labeling[j][i] = r_out_labeling[j][i] + 1;
					g_out_labeling[j][i] = g_out_labeling[j][i] + 1;
					b_out_labeling[j][i] = b_out_labeling[j][i] + 1;
				}

				//上か左のラベルが0と0以上の時・・・
				if ((up_r_label >= 1 && up_g_label >= 1 && up_b_label >= 1) && (left_r_label == 0 && left_g_label == 0 && left_b_label == 0))
				{
					//0ではない方の値をとる
					r_out_labeling[j][i] = up_r_label;
					g_out_labeling[j][i] = up_g_label;
					b_out_labeling[j][i] = up_b_label;

				}
				else if ((up_r_label == 0 && up_g_label == 0 && up_b_label == 0) && (left_r_label >= 1 && left_g_label >= 1 && left_b_label >= 1))
				{
					r_out_labeling[j][i] = left_r_label;
					g_out_labeling[j][i] = left_g_label;
					b_out_labeling[j][i] = left_b_label;
				}

				//0以外でラベルが付いている時・・・
				if ((up_r_label >= 1 && up_g_label >= 1 && up_b_label >= 1) && (left_r_label >= 1 && left_g_label >= 1 && left_b_label >= 1))
				{
					//もし，上のラベルの方が小さい時・・・
					if ((up_r_label < left_r_label) && (up_g_label < left_g_label) && (up_b_label < left_b_label))
					{
						//0ではない方の値をとる
						r_out_labeling[j][i] = up_r_label;
						g_out_labeling[j][i] = up_g_label;
						b_out_labeling[j][i] = up_b_label;

						//大きい方の値を小さい方の値にする
						left_r_label = up_r_label;
						left_g_label = up_g_label;
						left_b_label = up_b_label;
					}
					else if ((up_r_label > left_r_label) && (up_g_label > left_g_label) && (up_b_label > left_b_label))
					{
						r_out_labeling[j][i] = left_r_label;
						g_out_labeling[j][i] = left_g_label;
						b_out_labeling[j][i] = left_b_label;

						//大きい方の値を小さい方の値にする
						up_r_label = left_r_label;
						up_g_label = left_g_label;
						up_b_label = left_b_label;
					}
					//label_upとlabel_leftが同じ値の時・・・
					else if ((up_r_label == left_r_label) && (up_g_label == left_g_label) && (up_b_label == left_b_label))
					{
						r_out_labeling[j][i] = up_r_label = left_r_label;
						g_out_labeling[j][i] = up_g_label = left_g_label;
						b_out_labeling[j][i] = up_b_label = left_b_label;
					}
				}
			}
		}
	}


}

ちなみに，ピクセル数をヒストグラムで表すと，以下のような結果になり，0と1しか表示されません．

##参考資料
ラベリングのアルゴリズムについて

Answer 1

単独で動くコードではないようなので、見たところ、、、

まず、 RGB (3色?) を処理しようとしてますが、分かりにくくなってません?
その一方、127 を閾値にして、判定を行っていますが、全てがどちらかの場合と、混じった場合の処理が混在。　まず、この辺の意図は?

2つ目のfor()で、up_r, up_g, up_b に値を設定してますが、ループ最後の値しか設定されません。なんの意味があるでしょう? もっともその後、使われていないようですが。
.. おっと、ここで、j - 1 の参照。 j=0の場合、不正な値の参照となります。その後の for()でも同じようなところがあります。 (i-iもある)

*_out_labeling に初期値を与えるループがあるようですが、初期値とは思えません。何を設定してるのでしょうか? ここでは、0/1 しか設定してないようなので、それが出力を 0/1 にしてる理由と思われます。 (他の値も設定してるが、それは消える?)

と、、色々と見直し箇所が多そう、、と言うより、単色にしてまずは、0/1のモノクロデータでの検証をお勧めします。

Answer 2

こういう規模のコードを書くなら、デバッグできる環境を整えましょう
WindowsならVisualStudioやEclipseなど。
コードを途中で止めて、変数のナカミを見ることができます。
また、１行づつ実行させてコードの実行順や変数の変化を見れるようになります

そうすれば、コードを当てずっぽで組むことがなくなり、どこがまずいのかわからない、と丸投げするようなことはしなくて済むようになります