(クラス分けを伴う)BGRをHSVに変換したい。

Question

###実現したいこと
クラス分けをする
opencvを利用し、BGRをHSVに変換したい。

###利用している環境
opencv3.1.0，visualstudio2015

###発生している問題
ハンドルエラーの発生

###原因と思われる箇所

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

	ImgInfo imginfo;
	ImgProcess imgprocess;

	std::ofstream LValueFile("BValueFile.csv");

	for (int i = 0; i < 125; i++) {

		sprintf_s(imgname, "img_%d.jpg", i);

		imginfo.SetImg(cv::imread(imgname, CV_LOAD_IMAGE_COLOR));
		if (imginfo.GetImg().empty()) return -1;

		imgprocess.ImgInput(imginfo.GetImg());　　//←ここが原因だと思われる

		cv::namedWindow("hsvimg", cv::WINDOW_AUTOSIZE);
		cv::imshow("hsvimg",imginfo.GetHsvImg());
		cv::waitKey(30);
	}
	return 0;
}

###関連ソースコード

class ImgInfo
{
public:
	//Ctor/Dtor
	ImgInfo();
	~ImgInfo();

	//Setter/Getter
	cv::Mat GetImg();
	void SetImg(cv::Mat _img);

	cv::Mat GetHsvImg();
	void SetHsvImg(cv::Mat _hsv_img);

private:
	cv::Mat img;
	cv::Mat hsvImg;
};


cv::Mat ImgInfo::GetImg() {
	return this->img;
}
void ImgInfo::SetImg(cv::Mat _img) {
	this->img = _img;
}

cv::Mat ImgInfo::GetHsvImg() {
	return this->hsvImg;
}
void ImgInfo::SetHsvImg(cv::Mat _hsv_img) {
	this->hsvImg = _hsv_img;
}

class ImgProcess
{
public:
	//Ctor/Dtor
	ImgProcess();
	~ImgProcess();

	//public method
	void ImgInput(cv::Mat img);

private:
	ImgInfo imginfo;
};

void ImgProcess::ImgInput(cv::Mat img) {
	cv::Mat hsv_img;
	cv::cvtColor(img, hsv_img, CV_BGR2HSV);
	imginfo.SetHsvImg(hsv_img);
}

###追記
試した見たところ、以下の状態では正常に動きました。

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

	ImgInfo imginfo;
	ImgProcess imgprocess;

	std::ofstream LValueFile("BValueFile.csv");

	for (int i = 0; i < 125; i++) {

		sprintf_s(imgname, "c:\Users\osumi\Documents\Yuki Tsuchiya\programs\ObjectHeight\ObjectHeight\img_%d.jpg", i);

		imginfo.SetImg(cv::imread(imgname, CV_LOAD_IMAGE_COLOR));
		if (imginfo.GetImg().empty()) return -1;

		imgprocess.ImgInput(imginfo.GetImg());

		cv::namedWindow("hsvimg", cv::WINDOW_AUTOSIZE);
		cv::imshow("hsvimg",imginfo.GetImg());　///←GetHsvImg()を変更
		cv::waitKey(30);
	}
	return 0;
}

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

	ImgInfo imginfo;
	ImgProcess imgprocess;

	std::ofstream LValueFile("BValueFile.csv");

	for (int i = 0; i < 125; i++) {

		sprintf_s(imgname, "c:\Users\osumi\Documents\Yuki Tsuchiya\programs\ObjectHeight\ObjectHeight\img_%d.jpg", i);

		imginfo.SetImg(cv::imread(imgname, CV_LOAD_IMAGE_COLOR));
		if (imginfo.GetImg().empty()) return -1;

		cv::Mat hsvimg;
		cvtColor(imginfo.GetImg(), hsvimg, CV_BGR2HSV);　　///←imgperocess.Imginput(imginfo.GetImg())の内容を表示

		cv::namedWindow("hsvimg", cv::WINDOW_AUTOSIZE);
		cv::imshow("hsvimg",hsvimg);
		cv::waitKey(30);
	}
	return 0;
}

Answer 1

ImgProcess::ImgInput()では，
ImgProcessクラスのメンバであるimginfoに対して
imginfo.SetHsvImg(hsv_img);
を行っています．

他方，main関数の末尾では，
cv::imshow("hsvimg",imginfo.GetHsvImg());
として，関数内のローカルなオブジェクトであるimginfoからHSV画像を取得しようとしています．
その結果は空のMatでしょうから，imshow()で例外が発生しているのでしょう．

Comments

[fana]

> クラス分けをする

とされているので，「想定しているクラスの分け方」があるのだと思うのですが，
各クラスの役割がいまいち読み取れませんので，どこをどう直すと良いのか（＝想定されている形になるのか）の判断がつきません．
そのため，問題の原因と思われる事柄のみを述べました．

[nigg_1513]

回答ありがとうございます。
現状、学習のため、「とりあえず」という形でクラスを分けております。そのため、全くできてはいませんがオブジェクト指向型プログラミングで構築できたらと考えております。

現在、想定している形としては、ImgProcessにおいても、main関数においても、共通のImgInfo内のプライベート関数から情報を取得したいと考えております。
そのような場合、どのように対処を行えば良いのでしょうか。お時間がありましたら、回答していただけると助かります。

[fana]

件の例外に対処するだけならまぁどうとでもやれば良いと思うのですが…　例えば…

現コードではImgInfoのインスタンスがmain内のとImgProcessのメンバとで２つあるわけですが，
ImgProcessがメンバとして持っている側を廃止し，ImgProcess::ImgInput()は「ImgInfoの参照を引数に受けて，そのImgInfoが持つMat(HSV側)を更新する」という話にするとか…？
（構造体とそれを扱う関数　の形ですね．「クラスを使ってみる」的な話として見ると非常に面白くない形に見える）

[nigg_1513]

ありがとうございます。
そのような形で、構築し直してみようと思います！

まずは、原因が特定できて一安心です。ありがとうございました！！！

Answer 2

まず、デバッグモードで

        imginfo.SetImg(cv::imread(imgname, CV_LOAD_IMAGE_COLOR));

にブレークポイントをセットして動作。
F10キーで次に移る際にエラーならばファイルの位置（パス）が問題。
できれば

 sprintf_s(imgname, "img_%d.jpg", i);

の中にパスも加えれ記述

 sprintf_s(imgname, "c:\hoge\img_%d.jpg", i);

してみてください。

Comments

[nigg_1513]

回答ありがとうございます。返信がおそくなり、申し訳ございません。
パスを加え実行してみましたが、改善されませんでした。

imshowにおいて、対象をimginfo.GetImg()で行った場合はしっかりと画面に表示されます。