C++ Builderにおける画像データの取得について

Question

C++での画像処理プログラムの研究をしている者です。文字数制限がありますので簡潔になりますがよろしくお願いします。担当教官から基本となるプログラムをいただいたのですが実行するとエラーが出てしまいます。プログラム内容： void TImg::ImgMemToBmp(Graphics::TBitmap *bmp, BYTE *img) { GetBmpHdr(); // ビットマップヘッダー取得 String fileName = StartDir + "\tmp.bmp"; FILE *fp;if (NULL == (fp = fopen(fileName.c_str(), "wb"))) return; fwrite(BmpHeader, 54, 1, fp); // ヘッダー //******** 画像書き込み ******** BYTE *p, *img1 = (BYTE *)malloc(Pixel_X_Max*3); for (int y=0; yLoadFromFile(fileName); // BMPファイル読み込み DeleteFile(fileName); free(img1); // 不要ファイル、メモリ削除 } 問題点：下から３行目の bmp->LoadFromFile(fileName); でクラスの例外が発生しました。"ビットマップが不正です"とのエラーが出ます。私なりにプログラムを解析してみたところで、tmp.bmpというファイルを作りそこに白黒画素値を書き込んでいるように思うのですが途中で止めてtmp.bmpのサイズを確認してみたところ原画像が640×480の画像でtmp.bmpのサイズは20971840×15728640となっていました。これが原因でエラーと出たのではないかと考えたのですがいかがでしょうか？補足要求していただければ、適宜補足いたしますのでどうかご助力をよろしくお願いいたします。

noocyte · Accepted Answer

> 原画像が640×480の画像でtmp.bmpのサイズは20971840×15728640となっていました。

これはどうやって調べたんでしょうか？
信頼できる手段で確認したんでしょうか？
tmp.bmp の16進ダンプを見たんでしょうか？
それとも (バグがあるかもしれない) 自作プログラムですか？

ヘッダがちゃんと書けていないんだったら，
普通の画像表示・編集ソフトで読もうとしてもエラーになるはずですよね？

(1) tmp.bmp が正しく書けていないのか，それとも LoadFromFile() に
　　問題があるのかを切り分けるため，tmp.bmp が画像表示・編集ソフト
　　(ペイントなど) でちゃんと表示できるかどうか確認してください．

(2) tmp.bmp がちゃんと書けていないのであれば，GetBmpHdr() に問題が
　　ありそうなので，そのソースを示してください．それと，できれば
　　tmp.bmp のヘッダ部 (ファイルの先頭54バイト) の16進ダンプも．

> 白黒画素値を書き込んでいるように思うのですが

元画像は８ビットグレースケールですが，
RGB 24ビットに変換して書き込んでいますね．

画像書き込み部のコードにもバグがあります．
BMP ファイルの１本の走査線は，４バイトの倍数に切り上げなければなりません．
(担当の先生は BMP ファイルのフォーマットを十分理解されていないようです．)
しかし今回は原画像の幅が480ドット (４の倍数) なので，
たまたまうまく動くはずです．したがって今回のエラーの原因ではありません．

↓参考
BMPファイルのフォーマット
http://www5d.biglobe.ne.jp/~noocyte/Programming/Windows/BmpFileFormat.html

noocyte · Answer

画像書き込み部の修正版も作ってみました．これもコンパイルは通してません．


　//******** 画像書き込み ********
　const size_t bitsPerPixel = 24; // １画素のビット数
　const size_t bytesPerPixel = bitsPerPixel / 8; // １画素のバイト数

　// 走査線１本のバイト数 lineSize を計算する．
　size_t lineSize = bytesPerPixel * Pixel_X_Max; // 正味のバイト数

　// lineSize を４の倍数に切り上げる (BMPファイルの仕様)．
　lineSize = (lineSize + 3) & ~(size_t)3;

　// 走査線１本分のバッファを確保する．
　// calloc() を用いるのは，lineBuf の最後の
　// ０～３バイトを０クリアするのが目的．
　BYTE *lineBuf = (BYTE*)calloc(lineSize, 1);
　if(lineBuf == NULL) {
Error:
　　if(lineBuf != NULL) free(lineBuf);
　　fclose(fp);
　　DeleteFile(fileName); // 出来損ないのファイルを削除する．
　　return; // 本当は NG を返すべき．
　}

　// 画素データを書く．
　const BYTE *srcLine = img + lineSize * Pixel_Y_Max; // img[] の直後を指す．
　for(int y = 0;　y < Pixel_Y_Max;　y++) {
　　// srcLine, src ← img[] の現在の走査線の先頭
　　srcLine -= lineSize;
　　const BYTE *src = srcLine;
　　BYTE *dest = lineBuf;
　　for(int x = 0;　x < Pixel_X_Max;　x++) {
　　　// 元画像の画素 *src (グレースケール８ビット) を
　　　// RGB 24ビットに変換して lineBuf に格納する．
　　　dest[2] = dest[1] = dest[0] = *src++;
　　　dest += bytesPerPixel;
　　}
　　if(fwrite(lineBuf, 1, lineSize, fp) != lineSize) goto Error;
　}
　fclose(fp);
　free(lineBuf);
　return; // 本当は OK を返すべき．

noocyte · Answer

#1 です．

> int TImg::MakeBmpHeaderFile(int w, int h)

うわ～，このメンバ関数は意図不明なことをやってますね．
処理の内容はわかりますが，わざわざこういう複雑怪奇な
処理をする意図がさっぱりわかりません．

Graphics::TBitmap は C++ Builder のクラスライブラリが提供している
クラスでしょうか？　だとすれば，おそらくこのメソッドの作者
(JulesVerne さんの先生か先輩？) は，ヘッダの各メンバの計算方法が
よくわからなかったかサボりたかったので，出来合いの TBitmap::SaveToFile()
が書き出してくれるヘッダを流用しようとしたのではないでしょうか？
でもいくつかのメンバは簡単なので自分で設定したと…．(笑)

でも，できたヘッダの16進ダンプを見ると，ほとんどデタラメですね．
ソースを見る限り，設定内容は悪くなさそうなので，
ファイルの読み書きがちゃんとできていないのかもしれません．
(エラーチェックをほとんどしていないので，そうであっても報告されない．)
もしかしたら StartDir の設定が間違っているのではないでしょうか．


┌複雑怪奇な TImg::MakeBmpHeaderFile() の改良版を作ってみましたので
│ご参考まで．(コンパイルは通していません．またインデントが潰れるのを
│防ぐため，半角空白を全角空白に置き換えています．)
↓
int TImg::MakeBmpHeaderFile(int w, int h)
{
　BITMAPFILEHEADER bfHdr;
　BITMAPINFOHEADER biHdr;
　const size_t bitsPerPixel = 24; // １画素のビット数
　const size_t bytesPerPixel = bitsPerPixel / 8; // １画素のバイト数

　// 画像サイズ設定
　if (w == 0) { w = Pixel_X_Max; h = Pixel_Y_Max; }

　// 走査線１本のバイト数 lineSize を計算する．
　size_t lineSize = bytesPerPixel * w; // 正味のバイト数

　// lineSize を４の倍数に切り上げる (BMPファイルの仕様)．
　lineSize = (lineSize + 3) & ~(size_t)3;

　// 画像データ本体のバイト数
　size_t imageSize = lineSize * h;

　// ヘッダサイズの合計
　size_t headerSize = sizeof(bfHdr) + sizeof(biHdr);

　// ヘッダの内容を設定する．

　bfHdr.bfType = 'B' | ('M' << 8);
　bfHdr.bfSize = (DWORD)(headerSize + imageSize);
　bfHdr.bfReserved1 = 0;
　bfHdr.bfReserved2 = 0;
　bfHdr.bfOffBits = (DWORD)headerSize;

　biHdr.biSize = sizeof(biHdr);
　biHdr.biWidth = (LONG)w;
　biHdr.biHeight = (LONG)h;
　biHdr.biPlanes = 1;
　biHdr.biBitCount = (WORD)bitsPerPixel;
　biHdr.biCompression = BI_RGB;
　biHdr.biSizeImage = (DWORD)imageSize;
　biHdr.biXPelsPerMeter = 0;
　biHdr.biYPelsPerMeter = 0;
　biHdr.biClrUsed = 0;
　biHdr.biClrImportant = 0;

　// ヘッダを BmpHeader にコピーする．
　memcpy(&BmpHeader, &bfHdr, sizeof(bfHdr));
　memcpy((char*)&BmpHeader + sizeof(bfHdr), &biHdr, sizeof(biHdr));

　BmpHFlag = 1;
　return OK;
}

C++ Builderにおける画像データの取得について

> 原画像が640×480の画像でtmp.bmpのサイズは20971840×15728640となっていました。

この回答への補足

画像書き込み部の修正版も作ってみました．これもコンパイルは通してません．

#1 です．

似たような質問が見つかりました

このQ&Aを見た人はこんなQ&Aも見ています

関連するカテゴリからQ&Aを探す

このQ&Aを見た人がよく見るQ&A

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング