OpenCVを利用したC++の画像アニメーションで動作しない

Question

###前提・実現したいこと
OpenCVを利用して、画像を横へ移動（アニメーション)したいです。
また、画像が一定の位置まで言った場合戻ってくるアニメーションにしたいです。

###発生している問題・エラーメッセージ

エラーは発生しないのですが、結果の画面が出力されない（動作停止になってしまう）です。
なので、移動しているかも確認できていません。

###該当のソースコード

#include <iostream>
#include "opencv2\opencv.hpp"
using namespace cv;

int x;


int main(int argc, char* argv[])
{
     x=0;
	cv::Mat baseimage(cv::Size(1000, 600), CV_8UC3);
	cv::Mat img1 = cv::imread("pa_dot.png");
	cv::Mat img4 = cv::imread("test4.jpg");

	cv::Mat Roi4(baseimage, cv::Rect(0, 0, img4.cols, img4.rows));
	img4.copyTo(Roi4);

	cv::namedWindow("test", CV_WINDOW_AUTOSIZE);



	while (1){
		x += 1;


		if (x >= 500){

			x = 0;
		}

		
		cv::Mat Roi1(baseimage, cv::Rect(x, 0, img1.cols, img1.rows)); //画像の座標 Player1
		img1.copyTo(Roi1);


				cv::imshow("test", baseimage);		/* 画像の表示 */

	}

	waitKey(0);						/* 入力待機 */
	return 0;
}

###試したこと
while文以外もfor文でも試しましたが結果は同じく動作が停止してしまいます。

###補足情報(言語/FW/ツール等のバージョンなど)
繰り返しの文が間違っているのか、また画像を表示させるが間違っているのか
C++初心者なので、ぜんぜんわからない状態です。
改善点、修正点お願いします。

Answer 1

OpenCVはあまり使ったことがないので、正確に理解しているわけではないのですが、

cv::namedWindow("test", CV_WINDOW_AUTOSIZE);

これはWhileループの手前に書くべきですね。リファレンスによると、すでに同じ名前のウィンドウがある場合は何もしないようなのでループの中に書いても問題にはなりませんが、無駄なので。

cv::imshow("test", baseimage); /* 画像の表示 */

おそらく、これだけではWM_PAINTが処理されないために画像が表示されないのだと思われます。
というのも、cv::waitKeyの説明のところに、

注意： この関数は，HighGUI においてイベントを取得して扱うための唯一の手段です．よって，実行環境がイベント処理を扱ってくれる場合を除けば，通常のイベント処理ではこの関数を周期的に呼び出す必要があります．

と書いてあります。したがって、

cv::imshow("test", baseimage);
cv::waitKey(1);

このように、cv::imshowの後で必ずcv::waitKey(1)を呼ぶようにすれば表示されると思います。

Comments

[kyousuke]

ありがとうございます、解決できました。

Answer 2

こんにちは。

とりあえずxを初期化してないようです。運良く500を超えていたら0クリアされますが、符の値だったりすると異常動作しそうです。

他にOpenCVの質問でたいへんよくあるのですが、img1やimg4をちゃんと読み出せていない可能性があります。
imread()の直後で何か適当にWindowをnamedWindow()で開いてimshow()してみましょう。
もし、表示されずに落ちるようでしたら、画像ファイルを読めていません。
間違いなくするため、'C:'当たりに画像ファイルをおいて、それをフル・パスで指定してみると良いです。

Comments

[catsforepaw]

> とりあえずxを初期化してないようです。

xはグローバル変数なので、よほど特殊な処理系でない限り0で初期化されますよ。

[kyousuke]

画像はwhile文なしなら表示できましたが、mainの中whileの前あたりにx=0;と初期化しても動作してくれないです・・

[Chironian]

catsforepawさん。フォローありがとうです。確かにそうでした。
大昔のC言語の記憶が更新されてませんでした;;

kyousukeさん。
catsforepawさんの指摘通り、xの初期化の件は私の記憶違いでした。(でも、特に理由がない限り、明示的に初期化する習慣を付けておいた方が安全ですよ。)

もう少し現象を教えて下さい。
while文有りの時は1回も画像が表示されないのでしょうか?
それとも1回だけ表示されて2回目が表示されないのでしょうか?

あ、cv::namedWindow()はwhile()文の前に移動する必要があるような気がします。
2回以上、同じウィンドウを作るとおかしなことになりそうです。

[kyousuke]

cv::namedWindow()はwhileの前に移動させました。実行されたときに、画面が灰色（読み込み中）のままになってしまい、応答なしになってしまいます。

[kyousuke]

色々な回答ありがとうございました。waitkeyの場所を変更したら解決しました。