フォルダ内の画像を一つ一つに対して処理をしたい。

Question

openFrameworksにて特定フォルダ内の画像全ての、各ピクセルのRGB値を取得し、平均の値を取るようなプログラムを作ろうとしました。
以下のようにofAPP.h, ofAPP.cpp, main.cppを作成し、実行しようとしたところ、ofAPP.cppの14行目image.load(p.filename());にて、「外側の関数のローカル変数は、キャプチャ リストに含まれていない限り、lambda 本体で参照できません」とエラーが出てしまいました。
ラムダ式の定義の問題だと思うのですが、C++、openFrameworksどちらにも触れ始めたばかりで上手に理解できませんでした。
こちら、どのように修正すればよいかご教授いただけないでしょうか。

当方の環境、Windows10で、Visual Studio 2015を用いてopenFrameworksを動かしております。

main.cpp

#include "ofMain.h"
#include "ofApp.h"

//========================================================================
int main( ){

	ofSetupOpenGL(1024,768, OF_WINDOW);			// <-------- setup the GL context

	// this kicks off the running of my app
	// can be OF_WINDOW or OF_FULLSCREEN
	// pass in width and height too:
	ofRunApp( new ofApp());

}

ofAPP.h

#pragma once

#include "ofMain.h"

class ofApp : public ofBaseApp{
	private:
		ofImage image;
	public:
		void setup();
		void update();
		void draw();
		
		void keyPressed(int key);
		void keyReleased(int key);
		void mouseMoved(int x, int y);
		void mouseDragged(int x, int y, int button);
		void mousePressed(int x, int y, int button);
		void mouseReleased(int x, int y, int button);
		void mouseEntered(int x, int y);
		void mouseExited(int x, int y);
		void windowResized(int w, int h);
		void dragEvent(ofDragInfo dragInfo);
		void gotMessage(ofMessage msg);
};

ofAPP.cpp

void ofApp::setup(){
	namespace sys = std::tr2::sys;
	sys::path p("./data"); // 列挙の起点
	std::for_each(sys::directory_iterator(p), sys::directory_iterator(),
		[](const sys::path& p) {
		if (sys::is_regular_file(p)) { // ファイルなら...
			cout << "file: " << p.filename() << endl;
			//  640x640, 30 frames/sec
			ofSetWindowShape(640, 640);
			ofSetFrameRate(30);
			//  set background
			ofSetBackgroundColor(0, 80, 0);
			//  load image
			image.load(p.filename());
			ofPixels pixels = image.getPixelsRef();
			int w = pixels.getWidth();
			int h = pixels.getHeight();
			for (int y = 0; y<h; y++) {
				for (int x = 0; x<w; x++) {
					ofColor color = pixels.getColor(x, y);
					totalr = totalr + color.r;
					totalg = totalg + color.g;
					totalb = totalb + color.b;
				}
			}
			totalr = totalr / 409600;
			totalg = totalg / 409600;
			totalb = totalb / 409600;
			printf("r: %d \n", totalr);
			printf("g: %d \n", totalg);
			printf("b: %d \n", totalb);
		}
		else if (sys::is_directory(p)) { // ディレクトリなら...
			cout << "dir.: " << p.string() << endl;
		}
	});
}

Comments

[moredeep]

どこでエラーがでているのでしょうか。

Answer 1

std::tr2::sys

filesystemはその頃とC++17とでそれなりに仕様が変わっているので、せめてexperimental版の方を使ってあげてください。

オーバーロードされた関数 "ofImage_<PixelType>::load [代入PixelType=unsigned char]" のインスタンスが引数リストと一致しません

たぶん

image.load(p.filename().generic_string());

か

image.load(p.filename().generic_u8string());

のどっちかで行けるはず・・・(OpenFrameWorksで文字コードを何を規定にしているのか忘れた)。

Comments

[bolnox]

ありがとうございました！
`image.load(p.filename().generic_string());`
にて無事解決いたしました。

Answer 2

ラムダ式内でラムダ式の外側の変数を使用するにはキャプチャリスト([]の中)に記載する必要があります。
メンバ変数のimageを使用しているのでthisをキャプチャしてみてください。

std::for_each(sys::directory_iterator(p), sys::directory_iterator(),
        [this](const sys::path& p) {

[&]で必要なもの全部参照してもOKです。

ついでですが、同じ変数名(p)の使用は避けたほうがいいと思います。
規格上内側が使用されますが読みにくくなるので。

Comments

[bolnox]

ご返答ありがとうございます。キャプチャリストについてようやく理解できました。
こちらの修正をしたことにより、ラムダ式でのエラーは出なくなったのですが、代わりに以下のようなエラーメッセージが出るようになってしまいました。
ofImageにはunsigned char型を渡す必要があるのに、p.filename()の型が違うからエラーが出ているのかな、と思うのですが、どのように修正すればよいかこちらについてもご存知でしょうか？
もしご存知でしたらお手数ですがご教授いただけますとありがたいです。

エラーメッセージ
>>オーバーロードされた関数 "ofImage_<PixelType>::load [代入PixelType=unsigned char]" のインスタンスが引数リストと一致しません
>>引数の型: (std::experimental::filesystem::v1::path)
>>オブジェクト型は ofImage です