Opencv画像処理 bmp処理

質問

常々自分の理解力の無さに呆れるyuki-yutoです。

今回はOpencvで作ったプログラムと別のプロジェクトで作ったプログラムをソースをコピーして一つにしたところ、オバーフロー？してしまうらしくプログラムが実行できません。
二つ別々なら動くので関数の受け渡しが悪いのでしょうか？

今回の処理の目的は　Opencvでキャプチャ→bmpで保存→（書いていませんが）for文でRGBの量を求める→一定量のRが検出されればショートカットを起動する　というものです。
RGBに関してはfor(i=0;i<width;i++)とかでまわしているソースを何処かで見た気がするのですが、bmp処理は疎いので正直RGBをどうやって出すかまだ良くわかっていません。
widthまでを使っているあたり1pixel毎配列にでも記録させているのでしょうか・・・？

#include <stdio.h>
#include <Windows.h> //bmp読み込み,ショトカ
#include<iostream>

#include <C:\OpenCV2.1\include\opencv\cv.h>
#include <C:\OpenCV2.1\include\opencv\highgui.h>

#ifdef _DEBUG
//Debugモードの場合
#pragma comment(lib,"C:\\OpenCV2.1\\lib\\cv210d.lib")
#pragma comment(lib,"C:\\OpenCV2.1\\lib\\cxcore210d.lib")
#pragma comment(lib,"C:\\OpenCV2.1\\lib\\cvaux210d.lib")
#pragma comment(lib,"C:\\OpenCV2.1\\lib\\highgui210d.lib")
#else
//Releaseモードの場合
#pragma comment(lib,"C:\\OpenCV2.1\\lib\\cv210.lib")
#pragma comment(lib,"C:\\OpenCV2.1\\lib\\cxcore210.lib")
#pragma comment(lib,"C:\\OpenCV2.1\\lib\\cvaux210.lib")
#pragma comment(lib,"C:\\OpenCV2.1\\lib\\highgui210.lib")
#endif

#include <math.h>
#include <stdlib.h>

int main(void){

//画像保存用
char str[32];//大きさは多く取っておいたほうが良いかも？640*480対応できるように。

//キー入力用
int key;

//カメラの用意
CvCapture *capture;
IplImage *frame = NULL; //カメラ画像用

//カメラ・デバイスの先頭ポインタを取得
capture = cvCaptureFromCAM(-1);

//カメラ画像のサイズ
double height = 240;
double width = 320;

//カメラ初期化
if(capture == NULL){
//カメラが見つからなかった場合
printf("カメラが見つかりません\n");
return (0);
}

//カメラ画像を表示するウィンドウの生成
cvSetCaptureProperty(capture, CV_CAP_PROP_FRAME_HEIGHT, height);
cvSetCaptureProperty(capture, CV_CAP_PROP_FRAME_WIDTH, width);
cvNamedWindow("CaptureFromCamera", CV_WINDOW_AUTOSIZE);

//キー入力されたら画像をキャプチャ
while(1){

//カメラ画像からフレーム画像を取り込む
frame = cvQueryFrame(capture);
if(frame == NULL)
break;

//画像の表示
cvShowImage("CaptureFromCamera", frame);

//sキー入力で画像を保存、Escキー入力でループから抜ける
key = cvWaitKey(10);
if(key == '\x1b')
break;
else if(key == 's'){
sprintf_s(str, "Capture_Bitmap.bmp");
cvSaveImage(str, frame);
}
}
//ウィンドウの破棄
cvDestroyWindow("CaptureFromCamera");
}

//ここからキャプチャしたbmp処理。

int main( int argc, char *argv[] ){

FILE *fp; //ポインタ、読み込み専用
BITMAPFILEHEADER bfh; //ファイルヘッダ
BITMAPINFOHEADER bmpinfo; //情報ヘッダ

//BMPファイルがあるかどうかチェック
if((fp=fopen("Capture_Bitmap.bmp","rb"))==NULL){
printf("ファイルが見つかりません");
exit(1);
}
//BMPファイルか確認
fread(&bfh, 14, 1, fp); //ファイルヘッダ読み込み,ってかファイルヘッダの1~14を読み込む
if(memcmp(&bfh.bfType,"BM",2)) //BMPファイルかチェック、2バイト位置にBMがあるか
printf("BMPファイルではありません");

fread(&bmpinfo, 40, 1, fp); //情報ヘッダ読み込み

printf("データの大きさは%dMB\n",(bmpinfo.biSizeImage+bmpinfo.biSize)/1000000); //表示サイズ調節

//何色BMPファイルか返却値で判定
switch(bmpinfo.biBitCount){
case 1: printf("2色ビットマップ\n"); break;
case 4: printf("16色ビットマップ\n"); break;
case 8: printf("256色ビットマップ\n"); break;
case 24: printf("1677色ビットマップ\n"); break;
case 32: printf("1677色ビットマップ\n"); break;
}
//ココでRGBを抽出する、確かforを二回bmpの縦と横の最大値まで繰り返し、何か変数に入れるソースを見た気が・・・？

printf("インターネット　ショートカット　を起動します。Enterを押してね");
getchar();
//ショトカ起動文。Windows.hでデスクトップにショートカットを指定。

fclose(fp);
rewind(stdin);
getchar();
}

回答No.2

kmee · ベストアンサー

for(y=0;y<img->height;y++){
for(x=0;x<img->width;x++){
p[0]=img->imageData[img->widthStep*y+x*3];//B
p[1]=img->imageData[img->widthStep*y+x*3+1];//G
p[2]=img->imageData[img->widthStep*y+x*3+2];//R
}}

では、p[i] の内容が毎回上書きされます。
ループ終了時には、一番最後の座標(x,yが最大値)の値になります。
その後で
printf("%u 赤の多さ",p[2]);
printf("%u 緑の多さ",p[1]);
printf("%u 青の多さ",p[0]);
を出力すれば、その最後の座標の情報が出力されることになります。
全体が青っぽくても、右下の1点だけがノイズ等で赤くなっていたら、「赤」の情報が出力されます。

全体の傾向を調べるなら、適切な処理をする必要があります。例えば、平均値、中央値等。
色の傾向を見るなら、BGR→HSVに変換して、色相で調べる、という方法もあります。

補足

そうですか・・・自分でも色々考えた結果今後このプログラムをこんな風にしようと思います。

○画像を4分割して左上右上、左下右下に分ける。（ucharの関係上画像サイズは255*255にする）
○範囲ごとに赤色の値を合計して比較する（BGR）。
○またはこのプログラムのピクセルを読み込む位置を数個指定してそこのBGR値を取得する。(for文で回している部分を固定化)
自分の熟練度では赤の最後の位置を取得するなどできない（そもそもキャプチャなので綺麗にRGB取れるか解らない）ので範囲で分けてそれぞれの合計で判定できないかなと思います。

位置の固定化は自分で模索していますが、もし範囲ループでBGRの合計値が出そうなコードを組めそうならそうしてくれると大変助かります。
配列pの中のBGRの要素をどうやって足していくか、そこが一番の難関だと思います。

回答No.1

kmee

肝心の部分が省略されていて、どこが原因だか判断できません。

これがそのままなら、 main 関数が複数、という、普通ではない状態になりますが、それはいいのでしょうか?

あれこれ悩むなら、参考書を買ってはどうです?
IplImageの中にどんな順番で画素が入っているか、書いてあるはずですよ。

BMPファイルを直接操作するにしても、やりかたを書いたサイトはいくつもあります。
でも、OpenCV使っているなら、素直にOpenCVで読んだ方が楽でしょう。

あるいは、パフォーマンスを多少犠牲にして Get2Dを使うとか。
http://opencv.jp/opencv-2svn/c/core_operations_on_arrays.html#get-d

補足

なんとかコレで動くようになったのですが、ほぼ同じ写真を移しても何故かRの値が毎回大きく変動してしまいます・・・。

～～省略。

//sキー入力で画像を保存、Escキー入力でループから抜ける
key = cvWaitKey(10);
if(key == '\x1b')
break;
else if(key == 's'){
sprintf_s(str, "Capture_Bitmap.bmp");
cvSaveImage(str, frame);
}
}
//ウィンドウの破棄
cvDestroyWindow("CaptureFromCamera");
int x,y;
uchar p[3];
IplImage *img;
img = cvLoadImage ("Capture_Bitmap.bmp",CV_LOAD_IMAGE_COLOR);

for(y=0;y<img->height;y++){
for(x=0;x<img->width;x++){
p[0]=img->imageData[img->widthStep*y+x*3];//B
p[1]=img->imageData[img->widthStep*y+x*3+1];//G
p[2]=img->imageData[img->widthStep*y+x*3+2];//R
}}
printf("%u 赤の多さ",p[2]);
printf("%u 緑の多さ",p[1]);
printf("%u 青の多さ",p[0]);
return(0);
}
これでRの値が読み出せるはずが、何故か毎回変動幅が大きいです。
青いものを移しても普通にRの値が赤より多くなったりしてしまいます。
そもそもopencvでのRGBはどんな風にして決められているのでしょうか。ソレがわかれば改良の余地があるかも？