bmpファイルの指定範囲を塗りつぶすプログラム

Question

bmpファイルを読み込み、コマンドライン引数で指定した範囲のみを黒く塗りつぶすプログラムを分割コンパイルで実装することを想定しています。
ですが、gcc main.c read.c write.c erase.c -o mainでコンパイルし、./main 50 200 250 350で実行した結果、bmpファイルが黒塗りの画像となって出力されてしまいました。

またerase.cのout[i][j][0] = 0、out[i][j][1] = 0、out[i][j][2] = 0の値を全て255に変更し、白色で塗りつぶすプログラムにしてみた所、黒塗りの画像にコマンドライン引数で指定した範囲のみが白色で塗りつぶされた画像が出力されました。

上記の試行から、プログラム内のどこかで一度画像全体を黒で塗りつぶす操作が行われており、そこを修正すれば指定した範囲のみが塗りつぶされ、指定範囲外は元のbmpファイルを維持した状態で出力される、と考えられるのですが、件の問題があるであろう箇所を特定することができませんでした。

どこに不備があるのか教えていただけないでしょうか。

/*main.c*/

#include <stdio.h>
#include <stdlib.h>
#include "imagedata.h"

extern int erase( int x1, int x2, int y1, int y2 );

unsigned char img[HEIGHT][WIDTH][3];
unsigned char out[HEIGHT][WIDTH][3];
unsigned char header[54];

int main(int argc, char** argv)
{
    int x[2];
    int y[2];
    int i;

    for( i = 0; i < 2; i++)
    {
        x[i] = atoi(argv[i+1]);
        y[i] = atoi(argv[i+3]);
        if( x[i] > HEIGHT ) x[i] = 400;
        if( y[i] > WIDTH ) y[i] = 400;
    }
    
    read_image("in.bmp");

    erase((int)x[0], 
         (int)x[1], 
         (int)y[0],
         (int)y[1]);

    write_image("out.bmp");
    
    return 0;
}

/*imagedata.h*/

#define HEIGHT 400
#define WIDTH 400

extern unsigned char img[HEIGHT][WIDTH][3];
extern unsigned char out[HEIGHT][WIDTH][3];
extern unsigned char header[54];

extern int read_image(char* filename);
extern int write_image(char* filename);

/*erase.c*/

#include <stdio.h>
#include "imagedata.h"

int erase( int x1, int x2, int y1, int y2 )
{
    int i, j;

    for( i = y1; i < y2; i++ )
    {
        for( j = x1; j < x2; j++)
        {
            out[i][j][0] = 0;
            out[i][j][1] = 0;
            out[i][j][2] = 0;
        }
    }
    
    return 0;
}

/*write.c*/

#include <stdio.h>
#include "imagedata.h"

int write_image(char* filename)
{
    FILE *fw;
    int i, j;

    fw = fopen(filename, "wb");

    if( fw == NULL ){
        printf("Cannot open file\n");
        return 1;
    }

    fwrite(header, sizeof(unsigned char), 54, fw);

    for( i = 0; i < HEIGHT; i++ ){
        for( j = 0; j < WIDTH; j++ ){
            fwrite(out[i][j], sizeof(unsigned char), 3, fw);
        }
    }

    fclose(fw);

    return 0;
}

/*read.c*/

#include <stdio.h>
#include "imagedata.h"

int read_image(char* filename)
{
    FILE *fr;
    int i, j;

    fr = fopen(filename, "rb");

    if( fr == NULL ){
        printf("Cannot open file\n");
        return 1;
    }

    fread(header, sizeof(unsigned char), 54, fr);

    for( i = 0; i < HEIGHT; i++ ){
        for( j = 0; j < WIDTH; j++ ){
            fread(img[i][j], sizeof(unsigned char), 3, fr);
        }
    }

    fclose(fr);

    return 0;
}

Comments

[episteme]

read_image("in.bmp");
// erace(...) しない
write_image("out.bmp");

ってやったら元の画像がコピーできますか？

[sigure666]

確認してみましたがそもそも元の画像がコピーできてませんでした。
これは読み込みの時点に問題があるということでしょうか？

[dodox86]

Windowsのビットマップファイルは、ビットマップファイルと一口に言っても様々なフォーマットがあります。色数、パレット、一画素あたりのビット幅、透明度、などなど。加えて画像の幅、高さです。質問者さんが扱っているin.bmpファイルのフォーマット、画像の幅、高さと、プログラムで意識しているものが一致しているのが大前提です。それは大丈夫でしょうか。過去の質問でも、フォーマットを意識していない故の問題が何度も投稿されています。

[sigure666]

その点に関しては問題ないはずです。
99%プログラムに不備があると思われます…。

[jbpb0]

そもそも、何でimgとは別にoutを用意しているのですか？
画像から読んだデータをimgに入れたのだから、そのimgを触って(部分的に消すとか)、その後もimgをファイルに書き込めばいいのでは？？

触ってない元のデータを残したい、という意図ですか？
もしそうなら、imgの内容をoutにまずコピーしないと
outを触るのは、それから

[sigure666]

>> そもそも、何でimgとは別にoutを用意しているのですか？

ご想像の通り、元のデータを残したい、という意図があるからです。

imgの内容をoutにコピーですか。
試してみます。

[jbpb0]

コピーしなくてもoutにimgと同じものが入ってる、なんて都合のいいことは起きません
(outとimgは実は同じものでした、ってことをしない限り)

[sigure666]

確かに言われてみればそうですね。
大変勉強になりました。
ありがとうございます。

Answer 1

C言語のコードを書くなら、デバッグできる環境を整えましょう。
Eclipseや、WindowsならVisualStudioなど。
コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります
そうすれば、アテズッポでコードを書かなくて済むようになります。

Answer 2

それぞれの関数で何をしているか(入力は何、出力は何)、ちゃんと把握してみてください。

read_image("in.bmp");
imgに画像を読み込む
erase(...)
outの一定範囲を黒く塗りつぶす
write_image("out.bmp");
outを書き出す

これで「imgの一部が黒く塗り潰される」ことが期待できるでしょうか...

Comments

[sigure666]

すみませんどういう意味でしょうか…？

[thkana]

imgという紙(変数)とoutという紙(変数)を用意しました
imgに絵を描きました
outの真ん中を黒く塗りました
outの紙をみてもimgに描いてある絵が見えません

あたりまえでしょう? imgとoutは別の紙(変数)なんだから。

[thkana]

グローバル変数は特に指定しないと初期値0になるので紙と違って何も描かない状態で「真っ黒」というのはありますけれど。

[sigure666]

なるほど…何となく意味は掴めた気がします。
ということはeraseの処理にimgとoutを重ねるような処理を行えば良いのですね。