elle さんが書きました:＞h2so5さん
ご指摘ありがとうございます。1点を除いていずれも修正しました。

h2so5 さんが書きました:

• 正規表現よりはbasic_path::extensionをつかって比較したほうが良いような気がする

これについてなのですが、string_typeが返るようなので

コード:

// 疑似コード
string ext = extension();
if(ext == ".jpg" || ext == ".png" || ...)

のような形になってしまうように思うのですが、やはり正規表現よりbasic_pathを使った方がいいのでしょうか…

	static const char* extensions[] = { ".bmp", ".jpeg", ".jpg", ".png", ".dds", ".argb", ".tga" };
	string ext = extension();
	transform(ext.begin(), ext.end(), ext.begin(), tolower);
	if (find(begin(extensions), end(extensions), ext) != end(extensions)) {
		cout << "found";
	}

h2so5 さんが書きました:

• 正規表現よりはbasic_path::extensionをつかって比較したほうが良いような気がする

// 疑似コード
string ext = extension();
if(ext == ".jpg" || ext == ".png" || ...)

みけCAT さんが書きました:移動音を指定していない状態だと、移動するたびに「move soundの再生に失敗しました」というダイアログが出ました。
わざわざ移動音を用意しなくても、無音で普通にプレイできるようになるとさらにいいと思います。
あと、矢印キーでも移動操作ができると嬉しいです。

みけCAT さんが書きました: さらに、終了してもう一度プレイしようとしたところ、OS標準の「動作を停止しました」という表示が出て強制終了になりました。
画像ファイルをexeファイルにドロップした場合は
「clear soundが見つかりません」「shuffle soundが見つかりません」と出たあと、強制終了します。
一度解凍した15puzzleフォルダを消して再解凍しても、強制終了を回避できませんでした。

• Puzzle15クラスはNoncopyableというよりSingletonでは？
• sealed はC++/CLIのキーワードなので、C++11なら final を使うべき
• 正規表現よりはbasic_path::extensionをつかって比較したほうが良いような気がする
• セーブデータはテキストにしたほうが普通のエディタでもそのまま閲覧できて便利
• NaturalPow関数に再帰を使う理由がよくわからない、負の指数を入れると無限ループになる