8X8のドット文字を回転して返す関数がわからないです。

Question

前提・実現したいこと

C言語で8X8のドット文字「A」を回転して返す関数を作成したいです。
回転方向と回転角度を入力してそれに沿った処理をするといった形です。

発生している問題・エラーメッセージ

コンパイルエラーは起きないのですが実行をかけるとデバッグエラーが起きたり
そもそも何も表示されません。何が問題なのか全く分からないので、
どうすれば正しく表示する方法を教えてほしいです。

該当のソースコード

#include <stdio.h>

/*
	名称	rotation
	引数	const char in_buf[8][9]：8X8ドット文字
			char out_buf[8][9]：回転後の文字
			int direc：回転方向(0:左へ,1:右へ)
			int angle：回転角度(0:90度,1:180度,2:270度)
	戻り値	無し
*/
/*----- ここより下に解答を記述してください -----*/
void rotation(const char in_buf[8][9], char out_buf[8][9], int direc, int angle)
{
	int i, j;

	if(direc == 1){
		switch(angle){
			case 0 : for(i = 0; i < 8; i++){
						for(j = 0; j < 9; j++){
							out_buf[j][8 - i] = in_buf[i][j];
						}
					 }
					 break;

			case 1 : for(i = 0; i < 8; i++){
						for(j = 0; j < 9; j++){
							out_buf[j][8 - i] = in_buf[i][j];
							out_buf[j][8 - i] = out_buf[i][j];
						}
					 }
					 break;

			case 2 : for(i = 0; i < 8; i++){
						for(j = 0; j < 9; j++){
							out_buf[j][8 - i] = in_buf[i][j];
							out_buf[j][8 - i] = out_buf[i][j];
							out_buf[j][8 - i] = out_buf[i][j];
						}
					 }
					 break;
		}
																		
	} else {
		switch (angle) {							
			case 0 : for(i = 0; i < 8; i++){
						for(j = 0; j < 9; j++){
							out_buf[j][8 - i] = in_buf[i][j];
							out_buf[j][8 - i] = out_buf[i][j];
							out_buf[j][8 - i] = out_buf[i][j];
						}
					 }
					 break;

			case 1 : for(i = 0; i < 8; i++){
						for(j = 0; j < 9; j++){
							out_buf[i][j] = out_buf[j][8 - i];
							out_buf[i][j] = out_buf[j][8 - i];
						}
					 }
					 break;

			case 2 : for(i = 0; i < 8; i++){
						for(j = 0; j < 9; j++){
							out_buf[i][j] = out_buf[j][8 - i];
						}
					 }
					 break;
		}
	}

	return;
}
/*----- ここより上に解答を記述してください -----*/

int main(void)
{
	char in_buf[8][9] = {	{"   A    "},
							{"   A    "},
							{"  A A   "},
							{"  A A   "},
							{" A   A  "},
							{" AAAAA  "},
							{"A     A "},
							{"A     A "}	};
	char out_buf[8][9] = { { 0 } };
	int i, direc, angle;
	
	printf("回転方向(0:左へ,1:右へ)=");
	scanf("%d", &direc);
	
	printf("回転角度(0:90度,1:180度,2:270度)=");
	scanf("%d", &angle);
	
	/*----- ここより下に解答を記述してください -----*/
	rotation(in_buf, out_buf, direc, angle);
	/*----- ここより上に解答を記述してください -----*/
	
	for(i = 0; i < 8; i++)
		printf("%s\n", out_buf[i]);
	
	return 0;
}

試したこと

関数内での処理は一応書けました。90度(90度1回転),180度(90度2回転),270度(90度3回転)。
右回りでも左回りでも90度何回回転するかによって処理を分けています。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

左90度と右270度は同じ結果になるでしょう。
i や j が 0～7 の時、8 - i は 1～8 になるのでダメ。7 - i でしょう。

void rotation(const char in_buf[8][9], char out_buf[8][9], int direc, int angle)
{
	switch (direc * 10 + angle) {
	case 0: case 12: // 左90度、右270度
		for (int i = 0; i < 8; i++)
			for (int j = 0; j < 8; j++)
				out_buf[7 - j][i] = in_buf[i][j];
		break;
	case 1: case 11: // 左180度、右180度
		for (int i = 0; i < 8; i++)
			for (int j = 0; j < 8; j++)
				out_buf[7 - i][7 - j] = in_buf[i][j];
		break;
	case 2: case 10: // 左270度、右90度
		for (int i = 0; i < 8; i++)
			for (int j = 0; j < 8; j++)
				out_buf[j][7 - i] = in_buf[i][j];
		break;
	}
}

Comments

[退会済みユーザー]

教えてくださりありがとうございます!
配列周りをもうちょっと復習しておきます・・・

Answer 2

とりあえずout_bufのサイズは[8][9]ですが、以下のコードではj=8の場合に範囲外をアクセスしてしまいます。
まずは単純な２×２や３×３の場合でin/outがどういう対応関係になるか紙に描くなりして考えてみてはどうでしょうか？
ちなみにin/outともに行方向の末尾の\0終端文字は回転する必要はありません（してはいけません）。

                     for(j = 0; j < 9; j++){
                            out_buf[j][8 - i] = in_buf[i][j];
                        }

ついでに

以下のように座標値変換（回転後の座標値を求める方法）でもできます。

#include <stdio.h>
#include <string.h>

#define SIZE 3

// 座標値を時計回りに90度回転
void rot90( int *px, int *py)
{
	int x(*px), y(*py);
	*px = SIZE - y - 1;
	*py = x;
}

// 座標値を回転
// direc : 0=反時計, 1=時計回り
// angle : x90度
void rot( int *px, int *py, int direc, int angle)
{
	int lop = direc == 1 ? (angle % 4) : (4 - (angle % 4));
	for( int i = 0; i < lop; i++){
		rot90( px, py);
	}
}

// 図形を回転
void rotate( char src[][SIZE+1], char dst[][SIZE+1], int direc, int angle)
{
	for( int y = 0; y < SIZE; y++){
		for( int x = 0; x < SIZE; x++){
			char c = src[y][x];
			int dx(x), dy(y);
			rot( &dx, &dy, direc, angle);
			dst[dy][dx] = c;
		}
	}
}

// 図形をprint
void print( char s[][SIZE+1])
{
	for( int y = 0; y < SIZE; y++){
		printf("%s\n", s[y]);
	}
}

int main(void)
{
	char src[SIZE][SIZE+1] = {	"-+-",
								"+++",
								"---"};
	char dst[SIZE][SIZE+1];
	
	for( int direc = 0; direc < 2; direc++){
		for( int angle = 0; angle < 4; angle++){
			memset( dst, 0, sizeof(dst));
			printf("direc=%d, angle=%d\n", direc, angle);
			printf("src=====\n");
			print( src);
			rotate( src, dst, direc, angle);
			printf("dst=====\n");
			print( dst);
		}
	}
	return 0;
}

Comments

[退会済みユーザー]

終端文字は回転しちゃだめだから、0～7の添え字の部分を回転させると
out_buf[j][7 - i] = in_buf[i][j];
行と列の数が違う場合範囲外をアクセスしないようにする方法が分かりません。
行と列が同じなら上のコードが使えるのですが・・・

[8524ba23]

>終端文字は回転しちゃだめだから、0～7の添え字の部分を回転させると
その通りです。
>行と列が同じなら上のコードが使えるのですが・・・
終端文字の領域はアクセスする必要はありません。
よってin/outどちらも正方形の領域だけアクセスすればよいです。
あ、ただしout側はあらかじめ全体を\0で埋めておく必要はあります。

[退会済みユーザー]

教えてくださりありがとうございました!
おかげでなんとか全パターン表示させることができました!

Answer 3

動かしてみたわけではないのですがぱっと見て、同じミスだと思うものが何か所かあります。

void rotation(const char in_buf[8][9], char out_buf[8][9], int direc, int angle)
:
 for(j = 0; j < 9; j++){
     out_buf[j][8 - i] = in_buf[i][j];
:
:

out_bu[8][9]なので、forループの中はout[8 - 1][j]じゃないといけないと思います。

Comments

[退会済みユーザー]

for文のループ 部分はこう書き直しました
for(i = 0; i <= 7; i++){
for(j = 0; j <= 7; j++){
左に90度、右に270度はできたのですが他が何も表示されません・・・

[退会済みユーザー]

教えてくださりありがとうございました!
おかげでなんとか全パターン表示させることができました!