質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.49%

  • C

    4639questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

C言語について

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 553

berry

score 52

今、C言語の勉強中なのですが
数値を読み込んでその数の分だけ数値を読み込み合計と平均を出すプログラムを組みました。

include<stdio.h>

int main(void)
{
int i,j;
int v[i];
int sum;
printf("j:"); scanf("%d",&j);
for(i=0;i<j;i++){
printf("v[%d]:",i); scanf("%d",&v[i]);
sum = sum + v[i];
}
printf("sum:%d\n",sum);
printf("ave:%.2f\n",(double)sum / j);
return 0;

これを実行したところjの値を6以上読み込むと
j:11
v[0]:1
v[1]:1
v[2]:1
v[3]:1
v[4]:1
v[5]:1
sum:6
ave:6.00
となってしまいます。この原因はなんでしょうか?回答のほどよろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • raccy

    2016/07/31 18:46

    コードはコードブロックで囲んでいただけませんか? ```(バッククオート3つ)で囲み、前後に改行をいれるか、コードを選択して「</>」ボタンを押すとコードブロックになります。

    キャンセル

回答 4

checkベストアンサー

+1

すでに指摘されていますが、

int v[i]; 

これが問題ですね。初期化されていないiには何が入っているのか判らないので、配列vのサイズがいくつになるか予想が付きません。プログラムが正しく動かない原因となります。

おそらく、main関数実行時にiに6が入っていて、配列vのサイズが6で確保されたのだと思います。その後、jに11を入力してforループを回して配列vに1を入力していますが、v[6]に入力しようとしてすでに配列のサイズを超えており、たまたま'j'に1が入ってしまったのだと思われます。そのため、その時点でループを抜け、aveが6になってしまったのでしょう。

配列はサイズを入力してから確保するか、上限を決めて確保し、j:では上限の範囲内で入力するようにした方が良いです。


ちょっと違ったかも。最初のiは5が入っていて、v[5]を入力した時点でjに1が入った、というのが正確かもしれません。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/31 19:51

    ありがとうございました。とても、勉強になりました。
    他の回答してくださった方々も大変参考になりました。

    キャンセル

0

このプログラムにはいくつか問題があります。

  • int v[i];となっていますが、このような不定個の配列宣言は(いちおう規格化されているとはいえ)あまり一般的なものではありませんし、それ以前にこの宣言の時点ではiの値が不定なので、正常に動作しない危険が大きいです。別に配列を使う必要もない場面なので、一時変数を使いまわして配列は削ってしまいましょう。
  • scanf("%d",&v[i]);で整数を入力させると、改行コードが残ってしまいます。読み捨てて処分する必要があります。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

配列 v を
int v[i]; 
と定義していますが、i が設定されていません。
ここで十分な大きさの配列を確保する必要があります。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/31 17:59

    今回の件とは関係ありませんが、 sum は最初に0に初期化しておいたほうが良いです。

    キャンセル

0

こんにちは。

他の方も言っているように、int v[i];では意図した通りになりません。
これはscanf()でjを読み込んだ後にint v[j];(iではなくj)とすれば意図通りになると思います。

またsumに初期値を設定していないため、不定値になってます。
そこにv[i]の値を加えても最初が不定なので、やはり不定なままです。
for()ループの前で初期設定しましょう。

上記2点を修正すれば動作しますよ。頑張って下さい。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • C

    4639questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。