0

下記のプログラムをコンパイルし、実行してみたところ下の方にある数字を入力すると、Segmentation fault(コアダンプ)が表示されます。
どこか間違っているでしょうか?
よろしければ教えてください。

#include <stdio.h>

#define N 10
#define NUM_SCORE 50

int main ( void ) 
{

    int i,j,n,x;
    int score[ NUM_SCORE ] = {
        1,  4,  9,  9,  8, 10, 10,  9,  5, 10, 
        2,  9,  6,  4,  0,  7,  3,  5,  6,  6, 
        7,  4,  2,  9,  2,  5,  5,  3,  1,  9, 
        5,  7,  3,  2,  7,  9,  1,  7,  6,  6, 
        5,  8,  2,  5,  3, 10,  6,  2,  2,  5, 
    };

    int histogram[ N+1 ]; 

    for( i = 0; i <= N; i++ )
    {   
        histogram[ i ] = 0;
    }
    
    for( j = 0; j < NUM_SCORE; j++ )
    {
        histogram[ score[ j ] ]++;
    }


    printf("点数の分布は以下の通りです.\n");
    for ( i = 0; i <= N; i++ ) 
    {
        printf(" %2d点: %d\n ", i, histogram[i] );
    }
    
    printf( "整数を入力してください>>>" );
    scanf( "%d", x );
    n = 0;
    for ( i = x; i <= N+1; i++ )
    {
        n = n + histogram[ i ];
    }
    printf( "%d点以上の人は%d人います", x, n );
    return 0;

}
0

2 件の回答 2

1

@Jogenara さんも指摘されていますが、Segmentation fault(コアダンプ)の原因は追加されたscanf( "%d", x );で、入力された値を格納する変数へのポインタを指定すべきところを、変数そのもの(しかも初期化されていない)が指定されているためです。

入力された数値文字列をintに変換した結果が、初期化されていない変数の値をポインタと見立てて、その示す先の領域に格納しようとするため、Segmentation fault(コアダンプ)が発生しているのでしょう。
まあ(たいがいは0で)初期化されていたとしても、それはポインタ(アドレス)としては無効な値なのでSegmentation fault(コアダンプ)は発生しますが。

前の回答で@metropolisさんが紹介されていたcppcheckだと「警告」ですが以下が表示されます。
(WindowsのGUI画面でチェックしているので細かい表示内容は違っているかもしれませんが)

CWE: 686
%d in format string (no. 1) requires 'int *' but the argument type is 'signed int'.

@Jogenara さん指摘のようにxの前に&を付けてscanf( "%d", &x );としてください。


また前の回答へのコメントでprintfとscanfを追加しただけと書かれていましたが、それ以外のところ(ループの継続条件)も変更されていてそれも問題となります。
元:

for ( i = 7; i <= N; i++ )

新:

for ( i = x; i <= N+1; i++ )

i <= Ni <= N+1になっています。
それが原因でループ内の以下で問題(Segmentation fault(コアダンプ)か、そうでなくても結果の合計人数の違い)が発生するでしょう。

    n = n + histogram[ i ];

上記同様cppcheckでは上記n = n + histogram[ i ];の行で以下の「エラー」が表示されます。

CWE: 788
Array 'histogram[11]' accessed at index 11, which is out of bounds.

こちらは元のi <= Nに戻してください。

0
for ( i = x; i <= N+1; i++ )
{
    n = n + histogram[ i ];
} 

ここですね。
配列外を参照してます。
他の個所ではちゃんと注意しているようなので、
気を付ければ気付けたバグですね。

よくみたらscanfも間違ってましたね。
scanf("%d",&x);が正しい。

1
  • この質問は回答者が試されてたんですかね?ちゃんと実行確認するかどうか?考えすぎ?
    – Jogenara
    2020年11月21日 23:00

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。