配列を一つずつずらすコード

Question

各配列に格納されている数値を一つずつずらすコード（以下に示してます）を作成したのですがうまく動作しません．
数値入力を3,4,5と入力すると，rotation関数を通した後は0,0,0と表示されます．
また、rotation関数で，for文の条件をj<2　に変更すると，0,0,3と表示されます．
配列と関数とポインタの表記も理解度は微妙です…

void rotation(int *a);

int main(void)
{
int a[3] = { 0 };
int i;
int k;

printf("数値入力\n");
for (i = 0; i < 3; i++)
{
	scanf("%d", &a[i]);
}

printf("入力は以下です\n");
for (i = 0; i < 3; i++)
{
	printf("%d\n", a[i]);
}


printf("rotation後は以下です\n");


do
{
	rotation(a);
	printf("rotationする場合は1、しない場合は0を入力：");
	scanf("%d", &k);

} while (k == 0);

return 0;

}

void rotation(int *a) //値が格納されている配列を一つずつずらす関数
{
int j;
int b[3] = { 0 };

for (j = 0; j < 3; j++);
{
	if (j != 2)
	{
		b[j] = a[j+1];  //配列aが1ずつズレて配列bに格納
	}

	else
	{
		b[j] = a[0];  // 配列aの先頭は配列bの最後尾
	}
}

for (j = 0; j < 3; j++)
{
	printf("%d\n", b[j]);
}

}

Comments

[yumetodo]

ソースコードはそれを書くための記法(```で上下を囲む)があるのでそれを使ってください。インデントが消えてしまいます。

Answer 1

for (j = 0; j < 3; j++);

末尾に;があるのでその下のブロックはループになっていません。

b[j] = a[0]; // 配列aの先頭は配列bの最後尾

正しくループしたとしてもこれは動作しません。
なぜなら、この文が実行される時、すでに最初にa[0]にあった要素はa[1]にあった要素に置き換わったあとだからです。

Comments

[DreamTheater]

配列a自身は更新されないので、yumetodoさんが指摘されたfor文のセミコロンを削除すれば動きますよ。
ただ、コードが読みにくいのでせめてインデントしてください。 > suga.ruiさん

[yumetodo]

あ、bに入れてんの見落としてたわ、指摘ありがとうございます。

Answer 2

答えになっているかどうか？

usr ~/Project/test % ./a.out
0 1 2 3 4 5 6 7 8 9
1 2 3 4 5 6 7 8 9 0

#include <stdio.h>

static void rotation(int ary[], size_t siz)
{
    int tmp = ary[0];
    for(size_t i = 0; i < siz - 1; i++) {
        ary[i] = ary[i+1];
    }
    ary[siz-1] = tmp;
}

static void Print(int ary[], size_t siz)
{
    for(size_t i = 0; i < siz; i++) {
        printf("%d ", ary[i]);
    }
    putchar('\n');
}

int main(void)
{
    int ary[] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
    //
    Print(ary, sizeof(ary)/sizeof(int));
    rotation(ary, sizeof(ary)/sizeof(int));
    Print(ary, sizeof(ary)/sizeof(int));
    //
    return 0;
}