Ｃ言語課題！Ｃ言語初心者です。お願いします。

質問

課題内容
要素数１０の（０～９の順）配列Ａの値を２倍して配列Ｂに格納して
　その後配列Ａの値を配列Ｂに、配列Ｂの逆順を配列Ａに格納し
　入れ替える前と入れ替えた後の配列Ａ，Ｂの値を表示するプログラム

書いてみたプログラム

#include <stdio.h>

int main(void)
{

int i;
int va[10] = {0,1,2,3,4,5,6,7,8,9}; /* 初期化　*/
int vb[10];

puts("入れ替え前");
puts("-----------");
for(i=0 , i<10 , i++){
printf("va[%d] = %d\n" , i , va[i]); /* 入れ替え前の配列Ａ */
}

for(i=0 , i<10 , i++){
vb[10] = 2*va[10];
}
for(i=0 , i<10 , i++){
printf("vb[%d] = %d\n" , i ,vb[i]); /* 入れ替え前の配列Ｂ */
}

puts("入れ替え後");
puts("-----------");

for(i=0 , i<10 , i++){
vb[10] = va[10];
}
for(i=0 , i<10 , i++){
printf("vb[%d] = %d\n" , i ,vb[i]); /* 入れ替え後の配列Ｂ */
}

for(i = 0 , i < 5 , i++){ /* 配列Ｂの逆順を配列Ａに格納 */
int temp = va[i];
va[i] = vb[4-i];
vb[4-i] = temp;
}

for(i=0 , i<10 , i++){
printf("va[%d] = %d\n" , i ,va[i]); /* 入れ替え後の配列Ａ */
}

return(0);

}

コンパイルできません。
教えてください。お願いします。

回答No.4

asuncion

＞for(i=0 , i<10 , i++){
＞vb[10] = 2*va[10];
＞}

そもそも、[10]という、配列定義範囲外の領域を参照しようとしているのが間違いなのは、
別の回答者さんからのご指摘どおりです。

せっかく i というループ制御変数を使っているのに、
2倍するところにそれが出てこないことが不自然である、ということに
気づくべきでありましょう。

回答No.3

jjon-com

後半だけ提示すれば，前半は大丈夫でしょう。

puts("入れ替え後");
puts("-----------");

for(i=0 ; i<10 ; i++){
int temp = va[i];
va[i] = vb[i];
vb[i] = temp;
}
for(i=0 ; i<10 ; i++){
printf("vb[%d] = %d\n" , i ,vb[i]); /* 入れ替え後の配列Ｂ */
}

for(i = 0 ; i < 5 ; i++){ /* 配列Ｂの逆順を配列Ａに格納 */
int temp = va[i];
va[i] = va[9-i];
va[9-i] = temp;
}
for(i=0 ; i<10 ; i++){
printf("va[%d] = %d\n" , i ,va[i]); /* 入れ替え後の配列Ａ */
}

お礼

前半２倍のところも間違っているぽいのですが、どうすればよいですかね？

回答No.2

asuncion

＞for(i=0 , i<10 , i++){

すべてのfor文において、書き方が間違っています。
区切りはカンマで、本当に正しいですか？

お礼

ありがとうございます。助かりました！

回答No.1

maiko0318

コンパイルならどの行か出ているでしょ。

vb[10] = 2*va[10];
vb[10] = va[10];
要素番号１０はない。

for(i = 0 , i < 5 , i++){ /* 配列Ｂの逆順を配列Ａに格納 */
int temp = va[i];
va[i] = vb[4-i];
vb[4-i] = temp;
}

コンパイルには引っかからないが、間違っている。（動かしたらわかります。）

お礼

ありがとうございます。その通りでした。２倍のところはどうするべきでしょうか？