for文を正常に動作させたい

Question

forループが継続しない

OpenCVを利用して画像から特徴点の抽出をおこなうプログラムを作成しています．利用する画像サンプルが多いので，stringを利用して上手く自動化できればと考えていましたが，うまく動作できませんでした．

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

想定している分のforループが実行されない．画像の読み込み，特徴抽出，書き出しといった動作はしてくれるものの，ループ自体が7回目の段階で終了してしまう．

ソースコード

#include <string.h>
#include <iostream>
#include <fstream>
#include <opencv2/opencv.hpp>
#include <opencv2/core.hpp>
#include <opencv2/features2d/features2d.hpp>
#include <opencv2/highgui/highgui.hpp>
#include <opencv2/xfeatures2d/nonfree.hpp>
#include <opencv2/imgcodecs.hpp>

using namespace cv;
using namespace std;

int main() {

	auto algorithm = KAZE::create();//auto型:変数の型を推測?

	vector<KeyPoint> keypoint1;
	Mat color_image, gray_image, descriptor1, dest1;
	string filename, exportname, writename;
	int count = 0;

	for (int j = 1; j < 6; j++) {
		for (int i = 1; i < 9; i++) {

			filename = "./true/test" + to_string(j) + "-" + to_string(i) + ".bmp";
			exportname = "./true_res/test" + to_string(j) + "-" + to_string(i) + "_result.bmp";


			//画像指定
			color_image = imread(filename, 1);
			if (color_image.empty()) {
				return -1;
			}

			//resize(color_image, color_image, Size(), 0.5, 0.5);

			cvtColor(color_image, gray_image, COLOR_RGB2GRAY);
			normalize(gray_image, gray_image, 0, 255, NORM_MINMAX);


			algorithm->detect(gray_image, keypoint1);
			algorithm->compute(gray_image, keypoint1, descriptor1);

			drawKeypoints(color_image, keypoint1, dest1);

			imshow("result2", dest1);
			imwrite(exportname, dest1);
			waitKey(50);

			writename = "./true_kp_res/test" + to_string(j) + "-" + to_string(i) + "_kp.csv";
			ofstream ofs(writename);

			for (const auto& p : keypoint1) {
				ofs << p.pt.x << "," << p.pt.y << "," << p.angle << endl;
				count++;
			}


		}
	}
	return 0;
}

Comments

[int32_t]

color_image.empty() で return -1 しているかどうかは調べましたか?

[0sashimi]

今確認してきましたが，おっしゃる通りでした．この部分でreturn -1していました．

[int32_t]

回答書いておきます。

Answer 1

            color_image = imread(filename, 1);
            if (color_image.empty()) {
                return -1;
            }

この return -1 で関数を抜けているのでしょう。そのときの filename を調べて、対応するファイルがなかったり壊れてたりしないか確認しましょう。

Comments

[0sashimi]

使用していたサンプルリストを確認したところ，xxx_1-7.bmpが欠落していました．
回答ありがとうございました．

Answer 2

for (int j = 1; j < 6; j++) {

CとかC++の入門サイトを見て、for文がどういう動作をするのか調べてみよう。
この文は、j が1から始まって５まで（６未満）、５回のループを回るってことになってます