C++の関数等を使った書き方

質問

C++OpenCVで関数等を使ってプログラムをみやすくまとめたいのですが、教えていただけないでしょうか。

プログラムの内容はHSV変換をしています。
これをmain文より外に出したい場合どのような書き方をすればいいでしょうか。
返り値はHSVの３つがほしいです。

struct _HSV
{
int H;
int S;
int V;
}HSV[2000][2000];

main(void){

宣言
src_img = cvLoadImage (filename, CV_LOAD_IMAGE_COLOR);
for(y = 0; y < src_img->height; y++)
{
for(x = 0; x < src_img->width; x++)
{
p[0] = src_img->imageData[src_img->widthStep * y + x * 3]; // B
p[1] = src_img->imageData[src_img->widthStep * y + x * 3 + 1]; // G
p[2] = src_img->imageData[src_img->widthStep * y + x * 3 + 2]; // R

if (p[0] >= p[1] && p[0] >= p[2]) {max = p[0];} //Bがmax
else if (p[1] >= p[0] && p[1] >= p[2]) {max = p[1];} //Gがmax
else {max = p[2];} //Rがmax

if (p[0] <= p[1] && p[0] <= p[2]) {min = p[0];} //Bがmin
else if (p[1] <= p[0] && p[1] <= p[2]) {min = p[1];} //Gがmin
else {min = p[2];} //Rがmin

if(max==min){
HSV[y][x].H = 0;
HSV[y][x].S = 0;
HSV[y][x].V = p[0]; //R
}
else
{
if(max==p[2]){ //Rがmax
HSV[y][x].H = 60 * (p[1]-p[0])/(max-min);
}
else if(max==p[1]) //Gがmax
{
HSV[y][x].H = 60 * (p[0]-p[2])/(max-min);
HSV[y][x].H += 120;
}
else //Bがmax
{
HSV[y][x].H = 60 * (p[2]-p[1])/(max-min);
HSV[y][x].H += 240;
}
if(HSV[y][x].H < 0)
{
HSV[y][x].H += 360;
}

}

if(x % 60== 0){
fprintf(fp, "%d ,%d\n", HSV[y][x].H, HSV[y][x].S);
}

}

}
}

初歩的な質問ですみません。
できれば、関数以外でもこうすれば見やすい等ご指摘頂けると助かります。
よろしくお願いします。

回答No.1

麻野 なぎ（@AsanoNagi） · ベストアンサー

こんな感じですかね。

あと、この場合のように、使う必要のないところで配列を使うのは、たとえば、

HSV[y][x].V = p[0]; //R
else{min = p[2];} //Rがmin

で、どっちが red ？

というように混乱のもとかと思います。
一応そのあたり振り直したつもりですが、間違っているかもしれません。
あと、処理自体がどうも不自然な感じがしますが、そのまま書き直しました（と思います）

また、ここでは使ってないですが、本当に、HSV[y][x].H = 0; のような添え字付きで同じものを書く必要があるときは、

struct _HSV *ptr = &HSV[y][x];
とするだけで、
ptr->H のように（あと、ptr->V, ptr->S も同じ）アクセスできるので見かけは変わりますね。

struct colorSetType
{
　int red;
　int green;
　int blue;
};

struct _HSV
{
int H;
int S;
int V;
};

int minColor(struct colorSetType color)
{
　int result = color.red;
　if (reuslt > color.green) result = color.green;
　if (result > color.blue ) result = color.blue;
　return result;
}

int maxColor(struct colorSetType color)
{
　int result = color.red;
　if (reuslt < color.green) result = color.green;
　if (result < color.blue ) result = color.blue;
　return result;
}

strcut _HSV getHSV(struct colorSetType color)
{
　struct _HSV resut;
　int max = maxColor(color);
　int min = minColor(color);
　int diff = max - min;

　if (min == max)
　{
　 result.H = 0;
　 result.S = 0;
　 result.V = color.blue;
　}
　else if (color.red == max)
　{
　 result.H = 60 * (color.green - color.blue) / diff;
　}
　else if (color.green == max)
　{
　 result.H = 60 * (color.blue - color.red) / diff;
　 result.V += 120;
　}
　else
　{
　 result.H = 60 * (color.red - color.green) / diff;
　 result.V += 240;
　}

　if (result.H < 0) result.H += 360;

　return result;
}

int main()
{
　struct _HSV HSV[2000][2000];
　struct colorSetType p;
　int x;
　int y;

　src_img = cvLoadImage (filename, CV_LOAD_IMAGE_COLOR);
　for(y = 0; y < src_img->height; y++)
　{
　 for(x = 0; x < src_img->width; x++)
　 {
　 p.blue = src_img->imageData[src_img->widthStep * y + x * 3]; // B
　 p.green = src_img->imageData[src_img->widthStep * y + x * 3 + 1]; // G
　 p.red = src_img->imageData[src_img->widthStep * y + x * 3 + 2]; // R

　 HSV[y][x] = getHSV(p);
　 if(x % 60== 0)
　 {
　 fprintf(fp, "%d ,%d\n", HSV[y][x].H, HSV[y][x].S);
　 }

　 }
}
return 0;
}

お礼

迅速かつ、わかりやすい説明ありがとうございます。
大変勉強になります！
これをもとにプログラムを組んでみようと思います。
ありがとうございました。