画像ファイルの書き込みができないです。

Question

下から上に青から赤に変わっていくグラデーション画像をつくりたいのですが、全部緑の色になってしまいました。どこが間違っていますか？

コード
#define HEIGHT 400
#define WIDTH 400

#include <stdio.h>
int main()
{
    FILE *fr, *fw;
    int i, j;

    unsigned char header[54];

    fr = fopen("red.bmp", "rb");
    fw = fopen("out1.bmp", "wb");

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

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

    for( i = 0; i < HEIGHT; i++ )
    {
        for( j = 0; j < WIDTH; j++)
        {
            unsigned char img[3] = {225 * HEIGHT / 400, 0, 225 - 225 * HEIGHT / 400 };

            fwrite(img, sizeof(unsigned char), 3, fw);
        }
    }

    fclose(fr);
    fclose(fw);
    
    return 0;
}

Comments

[thkana]

「全部緑の色になってしまいました」という現象についての質問のタイトルが「画像ファイルの書き込みができないです」なのは正しいですか?

[majiponi]

画像ファイルのフォーマットは、無圧縮、400x400px、24bitカラー、ヘッダはV4形式、ヘッダと画像データ間にパディングなし、以上の条件は満たしていますか? ヘッダを解析しない(エラーチェックしない)ということは、前提に誤りがあっても気づけないということです。

Answer 1

unsigned char img[3] = {225 * HEIGHT / 400, 0, 225 - 225 * HEIGHT / 400 };

HEIGHT のナカミはループ中で変わりませんよ
もしかして、ここは i にする、とかなんとかかな。

ここは初期化式になってるのでiが使えないって話ですな
この変数の定義は関数冒頭で行って、ここでは代入のみを行いましょう

img[0] = 225 * i / 400;
img[1] = 0;
img[2] = 225 - 225 * i / 400 ;