画像処理のモザイクの仕方について

Question

in.bmp の画像の左半分に 16×16 画素のモザイクをかけて out.bmp に出力するプログラム image1mosaic.c を作成せよ. 16×16 画素毎に (その 256 画素の) 平均値を計算し, その値をその 16×16 画素すべてに代入すればよい

という問題なのですが、現在かけているプログラムは下記に貼っているものです。
これだと左半分に斑点のようなものが書けるだけでモザイクになりません。
どこが違うかご指摘いただけないでしょうか？

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

void bmp_mosaic(bmp_header_t *hd, pixel_t img[MAX_Y][MAX_X]);

int main(void){

bmp_header_t hd;
pixel_t img[MAX_Y][MAX_X];

bmp_read("in.bmp", &hd, img);		//読み込み
bmp_mosaic(&hd, img);			//呼び出し
bmp_write("out.bmp", &hd, img);		//書き出し

return 0;

}

//画像処理の関数の本体

void bmp_mosaic(bmp_header_t *hd, pixel_t img[MAX_Y][MAX_X]){

int i, j, x, y;
unsigned int rr, gg, bb;
int size = 16;

for(i = 0; i < hd -> biHeight; i += size){

	for(j = 0; j < hd -> biWidth / 2; j += size){
	
		rr = 0;
		gg = 0;
		bb = 0;
		
		for(x = 0; x < size; x++){

			if(hd -> biHeight < (i + x)){break;}
			
			for(y = 0; y < size; y++){
		
				if(hd -> biWidth < (j + y)){break;}
				
				rr += img[i + x][j + y].r;
				gg += img[i + x][j + y].g;
				bb += img[i + x][j + y].b;
			}
		}

		rr = rr / size * size;
		gg = gg / size * size;
		bb = bb / size * size ;
		

		for(x = 0; x < size; x++){
			
			for(y = 0; y < size; y++){
				
				img[i + x][j + y].r = rr;
				img[i + x][j + y].g = gg;
				img[i + x][j + y].b = bb;
				
			}
		}
	}
}

}

Answer 1

rr = rr / size * size;

これじゃ平均にならない

Comments

[fana]

おおっと！
「リファクタリング」なるコードで問題点が自然消滅してるｗ

Answer 2

左半分

なのは

for(j = 0; j < hd -> biWidth / 2; j += size){

幅を２で割っているからでは？

また、

rr = rr / size * size;

gg = rr / size * size;
bb = rr / size * size;

ggとbbでrrを使っているのはたぶん間違いでしょう。

それと平均値を求めている時に使っている座標↓と

rr += img[i + x][j + y].r;

書き込んでいる座標が違うのものおかしいです。

img[i][j].r = rr;

---

リファクタリングしたコードを載せておきます。

void bmp_mosaic(bmp_header_t *hd, pixel_t img[MAX_Y][MAX_X]) {

	int x, y, dx, dy;
	unsigned int r, g, b;
	int size = 16;
	int pixelCount;

	for (y = 0; y < hd->biHeight; y += size) {
		for (x = 0; x < hd->biWidth / 2; x += size) {

			r = 0;
			g = 0;
			b = 0;
			pixelCount = 0;

			for (dy = 0; dy < size && y + dy < hd->biHeight; dy++) {
				for (dx = 0; dx < size && x + dx < hd->biWidth; dx++) {
					r += img[y + dy][x + dx].r;
					g += img[y + dy][x + dx].g;
					b += img[y + dy][x + dx].b;
					pixelCount++;
				}
			}

			r /= pixelCount;
			g /= pixelCount;
			b /= pixelCount;

			for (dy = 0; dy < size && y + dy < hd->biHeight; dy++) {
				for (dx = 0; dx < size && x + dx < hd->biWidth; dx++) {
					img[y + dy][x + dx].r = r;
					img[y + dy][x + dx].g = g;
					img[y + dy][x + dx].b = b;
				}
			}

		}
	}
}

Comments

[hikaru_love_n]

画像の左半分にモザイクをかけたいので幅を2で割っていたのですが、間違いでしょうか？

書き込んでいる座標が違うという点についてもっと詳しく教えていただけますか？？

[takabosoft]

ああ、左半分は意図したものでしたか、失礼しました。

> 書き込んでいる座標が違うという点についてもっと詳しく教えていただけますか？？

ヒントだけですが、forでx,yで回しているのに中でx,yを使っていないのは不思議だと思いませんか？

[hikaru_love_n]

確かにそうですね！それを踏まえて書き直したところ
img[i + x][j + y].r = rr;
img[i + x][j + y].g = gg;
img[i + x][j + y].b = bb;

カラフルなモザイクになったのですが、どのようにしたら通常のモザイクになるでしょうか？

[takabosoft]

とりあえず現在のところまでを質問欄に追記してください。
ソースコードは

```cpp
（ここにコードを貼る）
```

というように記述してもらうと、ページ上で見やすくなります。

あと、カラフルってなんでしょう？
もし元画像と変換後の画像が載せられるようでしたらそれも追加してください。

[hikaru_love_n]

現在のところまでは追記させていただきました。
ただ画像の貼り方が分からなくて、、、
言葉で説明させていただくと、通常のモザイクではなく、色がついたモザイクになっています。

[takabosoft]

別にモザイクなので、色が付いているのは問題ないですけどねえ。
グレースケール処理なら灰色とかになっていないといけませんが。

[takabosoft]

本題とはあまり関係ありませんが、変数i, j, x, yが何を指しているかかなりわかりにくいですし、xとyは意味合い的には逆に使っている（実行結果はたぶん間違いない）ので、大変よみづらいです。

iとjをy, xに、xやyをdx,dyなどとすると、読みやすくなるかもしれません。

[hikaru_love_n]

実行結果が課題の本文に貼っていて、グレースケールにしないといけないんです、、、

なるほどです！

[takabosoft]

コンパイル通るかわかりませんが、リファクタリングしたコードを回答欄に追記しておきました。

＞ 実行結果が課題の本文に貼っていて、グレースケールにしないといけないんです、、、

少なくともこの質問欄には「左半分にモザイクを掛けろ」としかかいてありませんので、
グレースケールが必要なのかどうかはこちらはわからんです。

[hikaru_love_n]

ありがとうございます。解決しました。

すいません、記述不足でしたね、、、