c言語/sizeof関数の使い方

Question

###前提・実現したいこと
１０回数字を打ってそれを反対から出力させたいです
例）
入力
1
2
3
4
6
7
8
9
10
出力
10
9
8
7
6
5
4
3
2
1
sizeof関数を使うために自分で作った問題なのでsizeofを使って解きたいです。

###発生している問題・エラーメッセージ
出力が
10
10
9
8
7
6
5
4
3
2
1
となってしまいます。

エラーメッセージ

###該当のソースコード

ここにご自身が実行したソースコードを書いてください

#include<stdio.h>

int main(void)
{
int i;
int a[10];

for(i = 0;i < sizeof(a)/sizeof(a[0]);i++)
{
	scanf("%d",&a[i]);
}
printf("\n\n\n");
for(i=sizeof(a)/sizeof(a[0]);i >= 0;i--)
{
	printf("%d\n",a[i]);
}

return 0;

}

###試したこと
課題に対してアプローチしたことを記載してください
反対に出力させる所のループの最終値を色々いじってみたのですがダメでした＞＜
###補足情報(言語/FW/ツール等のバージョンなど)

Answer 1

＞二番目のループの初期値がおかしかったようです

はっきり理解できませんか？

２番めのループは１０から０まで表示することになっています。
a[10]と定義したらa[0]からa[9]まで定義され、a[10]は存在しません。

Answer 2

単純な添字範囲のミスのようですね。2番目のforループについて、iが配列aの添字範囲内(0〜9)におさまっているかご確認ください。

Comments

[Rivia]

二番目のループででiの初期値が10になっているので入力するときにi=0から始めたので
配列の大きさが１１というおかしいことになっているということなのでしょうか

[Rivia]

#include<stdio.h>

int main(void)
{
int i;
int a[10];

for(i = 0;i < sizeof(a)/sizeof(a[0]);i++)
{
scanf("%d",&a[i]);
}
printf("\n\n\n");
for(i=sizeof(a)/sizeof(a[0])-1;i>0;i--)
{
printf("%d\n",a[i]);
}

return 0;
}
というプログラムに書き直したら正常に動きました！やはり二番目のループの初期値がおかしかったようです

[Bongo]

そうなりますね。配列の要素が10個しかないのに、11番目から値を取り出そうとしていたようです。

GasTさんのアドバイスのように、ループ内でa[i - 1]とするのも一つの手ですが、iの初期値を1減らして配列の末尾、つまり9になるようにする手もあるかと思います。、

for ( i = sizeof(a) / sizeof(a[0]) - 1; i >= 0; i--)
{
printf("%d\n", a[i]);
}

[追記]
すみません、投稿タイミングの問題で追記コメントを見逃してしまいました。
コメントの通り、初期値を修正すると意図通りになりますね。

[Rivia]

ありがとうございます！
何が悪いか分かったので一安心しました。

Answer 3

配列の要素の大きさは1オリジンですから、indexで要素を指定するときマイナス1するといいのでは？
動くサンプル

#include<stdio.h>

int main(void) {
    int i;
    int a[10];
    
    for(i = 0; i < sizeof(a) / sizeof(a[0]); i++) {
        scanf("%d", &a[i]);
    }
    
    for(i = sizeof(a) / sizeof(a[0]); i > 0; i--) {
        printf("%d\n", a[i - 1]);
    }
    return 0;
}

Comments

[Rivia]

要素を指定するときマイナス１したら出来ました！
ありがとうございました！

Answer 4

#include<stdio.h>

int main(void)
{
int i;
int a[10];

for(i = 0;i < sizeof(a)/sizeof(a[0]);i++)
{
	scanf("%d",&a[i]);
}
printf("\n\n\n");
for(i=sizeof(a)/sizeof(a[0])-1;i>0;i--)
{
	printf("%d\n",a[i]);
}

return 0;

}
で正常に動きました！二番目のループの初期値がおかしかったようです
回答してくださりありがとうございました。

Comments

[Bongo]

おっと、よく見ると修正後のループの条件が、この記述ですと今度はa[0]が抜けてしまいませんか?投稿時のミスでしょうか。

[Rivia]

ホントでした＞＜よく見ると出力が10～2までで1が抜けてました！
二番目のループの終了値をi=>0にすることで正常に動きました（今度こそ）
重ね重ね指摘をありがとうございます！