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

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

新規登録して質問してみよう
ただいま回答率
85.50%
C

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

Q&A

解決済

2回答

2895閲覧

segmentation fault :11となる理由が不明。

ryu72639

総合スコア14

C

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

0グッド

0クリップ

投稿2018/06/15 14:53

c言語で関数を用いて指定された個数だけ整数を読み込んで、それらの合計を計算し表示するプログラムを作成せしてるのですが、実行しようとすると、segmentation fault: 11 と表示されてしまいます。理由を教えて下さい!

#include <stdio.h>

/* 合計の計算:要素数numberの配列valuesの,各要素の合計を計算し返却値として返す */
int sum ( const int values[], int number)
{
int i;
int total = 0;

for ( i = 0; i < number; number++ )
total += values[i]; /* 合計を計算*/

return total; /合計を返す/
}

int main(void)
{
int j, n, SUM;
int values[j];

printf("整数:"); /*何個の整数を足し合わせるか*/ scanf("%d", &n); for (j = 1; j <= n; j++){ scanf("%d", & values[j]); /*n個の整数がそれぞれ何か*/ } SUM = sum(values, n); /*sum関数で得た答え= SUM */ printf("合計:%d\n", SUM); return 0; }

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

C

1for ( i = 0; i < number; number++ ) 2

ここで配列を最後を越えてアクセスし続けてますね。

C

1for ( i = 0; i < number; i++ )

投稿2018/06/15 15:04

a_saitoh

総合スコア702

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

int values[j];

これで、大きさゼロの配列が宣言されています。
配列のサイズを、nの最大値以上にしてください。
例えば、int values[100];と宣言して、入力したnが100以上ならエラーにする。

あ、添え字が1始まりでnまでなので、

例えば、int values[101];と宣言して、入力したnが100以上ならエラーにする。

ですね。
Cの処理系によっては、nを入力した後で、int values[n+1];と宣言するという手もあります。

投稿2018/06/15 14:58

編集2018/06/15 15:06
otn

総合スコア84423

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ryu72639

2018/06/15 15:16 編集

なぜここでjという変数を中に入れるとダメなのですか? また、なぜn+1ならばokになるのだしょうか?
otn

2018/06/15 15:23

int values[j] の時点で、jの値が未代入だからです。jの値は規格上は不定です。 nに値を入力した後だと、そのnの値が必要なサイズ(-1)だからです。
ryu72639

2018/06/15 15:29

納得しました。丁寧な解説ありがとうございました。
otn

2018/06/15 15:42 編集

あと、入力のところで、 for (j = 1; j <= n; j++){ じゃなくて for (j = 0; j < n; j++){ にしないと、sum() の方と合わないです。 こうすれば、+1しないで int values[n]; でいいです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問