OpenCV 色の検出

[aaaa]

こんにちは。
今、わたしは赤色と青色だけを抽出することができるプログラムを作りました。(赤色のみ黒で塗りつぶした画像、青色のみ画像を塗りつぶした画像を表示)

次に私はこの画像が赤色か青色かを判断するプログラム（printfで赤、青を表示する）を作りたいと考えています。

しかしどうのようにしたらよいか全くわからず、ご助言をお願いした次第でございます。

コード:

using namespace cv;

int main(int argc, const char *argv[])
{
	/*genga*/
	Mat src_image = imread("big.jpg");
	namedWindow("genga");
	imshow("genga", src_image);
	
	int width = src_image.cols;
	int height = src_image.rows;
	
	/*HSV*/
	Mat hsv_image;
	cvtColor(src_image, hsv_image, CV_BGR2HSV);
	
	uchar hue, sat, val;
	Mat red_image = Mat(Size(width, height),CV_8UC1);
	Mat blue_image = Mat(Size(width, height),CV_8UC1);
	
	for(int y=0; y<height; y++)
		for(int x=0; x<width; x++)
		{
			/*H,S,V*/
			hue = hsv_image.at<Vec3b>(y, x)[0];
			sat = hsv_image.at<Vec3b>(y, x)[1];
			val = hsv_image.at<Vec3b>(y, x)[2];
			
			/*red_detect*/
			if((hue<10 || hue>170) && sat>45)
				red_image.at<uchar>(y, x) = 0;
			
			else
				red_image.at<uchar>(y, x) = 255;
				
			/*blue_detect*/
			if((hue>75 && hue<90) && sat>60)
				blue_image.at<uchar>(y, x) = 0;
			
			else
				blue_image.at<uchar>(y, x) = 255;
		
	}
	
	namedWindow("red_detect");
	imshow("red_detect", red_image);
	
	namedWindow("blue_detect");
	imshow("blue_detect", blue_image);
	
	if(red_image)
	
	
	
	waitKey(0);
	destroyAllWindows();
	return 0;
}

[aaaa]

度々すいません。
自分では、「red_image.at<uchar>(y, x) = 0　ならばflagを1にする」→「flagが1ならばprintfで赤と言う」
とすればできると思ったのですがうまく行きません

どうしたらよいのでしょうか。。。
お願いしたします

[asd]

次に私はこの画像が赤色か青色かを判断するプログラム（printfで赤、青を表示する）を作りたいと考えています。

自分では、「red_image.at<uchar>(y, x) = 0　ならばflagを1にする」→「flagが1ならばprintfで赤と言う」
とすればできると思ったのですがうまく行きません

プログラム内部の話をする前に判定方法について素朴な質問なのですが、
与えられる画像は「赤一色」「青一色」「赤でも青でもない」の3種類の画像と考えてよいでしょうか？
というのも、与えられた画像を「赤」、「青」と判別する基準が不明確なのでそこを明確にしたいなと思いまして。

例えば「赤一色」の画像であれば赤成分が1ピクセルでもあれば「赤画像である」と判別できるので
赤成分の有無をフラグで保持すれば判別できますが、
「赤も青も混在した画像」が渡された場合には赤成分も青成分もあるのに「赤画像である」と誤判別される可能性も出てきます。

この場合には赤成分と青成分のより多い方を優勢とみなす方法もありますが、
さらに厄介なのが「紫色一色」の画像のように赤成分も青成分も含まれているものの人間が見ると赤でも青でもない画像を与えた場合など
判別方法をどのようにするのか、与える画像は制限して考えてよいのかを教えてください。

一旦ここでは「赤一色」「青一色」のみの画像しか与えない前提で提示されているプログラムを見てみました。
提示されているプログラムでは赤と青のフラグはないようですが、
どのような処理を書かれて、どのような画像を与えたらどう判定されると考え、実際はどうだったのでしょうか？

実際に動かなかったというプログラムも提示されておらず、与えた画像と期待した結果、実際の結果も提示されていないので、
「うまく行きません」だけでは何に困っているのかこちらでもよくわかりません。

赤フラグと青フラグを追加して、どのような結果を期待し、実際はどうだったのかを提示していただけると
より詳細な助言がもらえるかと思いますよ。

[aaaa]

ご返信ありがとうございます。

とても納得させられました。
自分の中でもまとまっていないので、まとまり次第追記させていただきます。

また、私は信号の色を判断したいと考えています。
画像は限りなく信号機のみを写している画像を使う予定なので、画像は「上半分が赤で下半分が黒」、「上半分が黒、下半分が青」というような画像のつもりです。

[aaaa]

先ほど、信号機の画像を使うと宣言しましたが、まず第一歩として、赤色単色の画像を「赤色」と言わせたいです。
そこで、このようなコードを作りました。
赤であればflag_redが1になり、赤色というというイメージです

コード:

using namespace cv;

int main(int argc, const char *argv[])
{
	/*genga*/
	Mat src_image = imread("red.jpg");
/*	namedWindow("genga");
	imshow("genga", src_image);
*/	
	int width = src_image.cols;
	int height = src_image.rows;
	
	/*HSV*/
	Mat hsv_image;
	cvtColor(src_image, hsv_image, CV_BGR2HSV);
	
	uchar hue, sat, val;
	Mat red_image = Mat(Size(width, height),CV_8UC1);
	Mat blue_image = Mat(Size(width, height),CV_8UC1);
	
	int flag_red = 0;
	int flag_blue = 0;
	
	for(int y=0; y<height; y++)
		for(int x=0; x<width; x++)
		{
			/*H,S,V*/
			hue = hsv_image.at<Vec3b>(y, x)[0];
			sat = hsv_image.at<Vec3b>(y, x)[1];
			val = hsv_image.at<Vec3b>(y, x)[2];
			
			/*red_detect*/
			if((hue<10 || hue>170) && sat>45)
				red_image.at<uchar>(y, x) = 0;
				
				flag_red = 1;
				
			else
				red_image.at<uchar>(y, x) = 255;
				
		/*		flag_blue = 1;
		*/		
			/*blue_detect*/
		/*	if((hue>75 && hue<90) && sat>60)
				blue_image.at<uchar>(y, x) = 0;
			
			else
				blue_image.at<uchar>(y, x) = 255;
				/*flag = 1;*/
	}
	
/*	namedWindow("red_detect");
	imshow("red_detect", red_image);
	
	namedWindow("blue_detect");
	imshow("blue_detect", blue_image);
*/
	if(flag_red=1){
	printf("red\n");
	}
	
	
	waitKey(0);
	destroyAllWindows();
	return 0;
}

しかし、こちらはこのようなエラーが出てしまいました。

コード:

redbluedetect.cpp:55:4: error: ‘else’ without a previous ‘if’
    else
    ^

私はこのエラーを調べたところ意味が理解できませんでした。
また、elseの下ならば、flagが1になることは確認できたのですが、この違いがよくわからないのです。

このプログラムでも、赤じゃないから青です。
とは言えそうなのですが、できれば赤だから赤と言わせたいです。

[asd]

また、私は信号の色を判断したいと考えています。
画像は限りなく信号機のみを写している画像を使う予定なので、画像は「上半分が赤で下半分が黒」、「上半分が黒、下半分が青」というような画像のつもりです。

追記ありがとうございました。なるほど信号機でしたか。
そういうことであれば、背景やら信号機の灯火以外の余計な部分は一旦考えないものとして、
与えられた画像の赤成分、青成分を数えて多い方をその画像の色だと判定してもよいかもですね。
(赤と青が同時に点灯することはないものとして)

その場合だとフラグ方式よりも、赤と青の各成分量(ピクセル数)をカウントするようにした方がよいのかもです。
いずれにせよ判別方法がまとまるのをお待ちします(*´ヮ｀)

…と書いていたら追記があったのでそちらにも返信します。

しかし、こちらはこのようなエラーが出てしまいました。

コード:

redbluedetect.cpp:55:4: error: ‘else’ without a previous ‘if’
    else
    ^

提示されているプログラムのif文を見るとブロック{}で囲まれていないため、
判定分の直後の処理は1文しか書けない状態になっています。
それにもかかわらずif文の後に複数の文を書いた後でelse節を書いたために、
「elseに対応するifが見つからないよ！」とエラーを出されています。

当該部分だけを抜き出すと以下の部分です。
if((hue<10 || hue>170) && sat>45) の後にflag_red=1;が追記されたために、
if文がtrueだった時に処理する内容が複数の文になってしまいました。
{}で囲わない場合にはifやelseの後には1文しか書けないので、
エラーになっているものと思います。

コード:

			/*red_detect*/
			if((hue<10 || hue>170) && sat>45)
				red_image.at<uchar>(y, x) = 0;
				
				flag_red = 1;
				
			else
				red_image.at<uchar>(y, x) = 255;
				

ですので解決策としては以下のようにifやelseの処理部分を{}で囲んでブロックにしてあげるとよいかと思います。

コード:

			/*red_detect*/
			if((hue<10 || hue>170) && sat>45)
			{
				red_image.at<uchar>(y, x) = 0;
				flag_red = 1;
			}
			else
				red_image.at<uchar>(y, x) = 255;

これで期待した動きになったでしょうか？
不明な点があれば遠慮なく聞いてくださいね(*´ヮ｀)

このプログラムでも、赤じゃないから青です。
とは言えそうなのですが、できれば赤だから赤と言わせたいです。

確かに消去法で「赤じゃないから青」と判定してしまうと、
「黄色信号」の場合や「停電で赤も青も消灯している信号」に確実に誤判定しますので、間違いなく赤だ(or青だ)という判定にしたいですよね。

[aaaa]

素早い変身体へなりがとうございます。

まずは、ifのあとに{}の説明とご指示のため、解決することができました！
わかりやすい説明でとても感謝しています。
ありがとうございます。

また、ピクセルをカウントしたほうが良いという助言にも大変有り難みを感じています。
実際に赤信号の時に別のもの写ったらどうしようと悩んでいたので、この助言を元に一度プログラムを作ってい見たいと思います
本当にありがとうございます

[asd]

素早い変身体へなりがとうございます。

まずは、ifのあとに{}の説明とご指示のため、解決することができました！
わかりやすい説明でとても感謝しています。
ありがとうございます。

また、ピクセルをカウントしたほうが良いという助言にも大変有り難みを感じています。
実際に赤信号の時に別のもの写ったらどうしようと悩んでいたので、この助言を元に一度プログラムを作ってい見たいと思います
本当にありがとうございます

本題については無事に解決したようでよかったです(*´ヮ｀)

ちなみに解決した場合には送信ボタンの横に「解決！」というチェックボックスがありますので、
そこにチェックを入れた状態で解決報告を投稿すると解決したことが表示されますので、
解決した場合には「解決！」へのチェックも忘れないようお願いします。

オフトピック

一旦は信号機のみが写っている画像としているので問題ないとは思いますが、
提示されているように信号機以外のものが写っている場合(たとえが背景に赤い風船が写ってたりとか)には
単純なピクセルカウントだけですと誤判定の可能性も出てきます。

第一弾の判別プログラムが完成したら、実用的な条件(別のものが写っているとか)を付与していき
そのような場合にどう対応するのかを考えて精度を上げていくのかなと思います。
そのあたりを考えるのが面白味だと思うので、引き続き頑張ってください(*´ヮ｀)

[aaaa]

失礼いたしました。
おかげさまで解決することができました。
ありがとうございました。