fscanfで取り込んだ日本語を一次元配列に保存して正しくprintfで出力する方法

前提

各市町村区の名前と人口が記されているテキストファイルを読み込み，標準偏差と平均値に最も近い都市名をディスプレイに表示するというものを作りたいのですが，都市名の表示がうまくされません．

実現したいこと

正しい都市名の出力

発生している問題・エラーメッセージ

平均値に最も近い都市は��東京都八丈支庁青ヶ島村
標準偏差は190728.0
*** stack smashing detected ***: terminated
Aborted (core dumped)

該当のソースコード

``C
ソースコード

#include<stdio.h>
#include<math.h>
#define N 1740
double average(int number[])/*平均値を求める*/
{
    double ave,all = 0; int j;
    for(j = 0 ; j < N; j++)
    all += number[j];
    ave = (all)/N;
    return(ave);
}
double SD(int number[])/*標準偏差を求める*/
{
    int k,sd; double Dif[N], Sum = 0.0;
    for(k = 0 ; k < N ; k++)
    {
        Dif[k] = number[k] - average(number);
        Sum += ((Dif[k]*Dif[k])/N);
    }
    sd = sqrt(Sum);
    return(sd);
}
int main(void)
{
    FILE *fp = fopen("data11.txt", "r");
    int number[N],i = 0,min,l,num; char letter[N]; double Dif[N];
    while(i < N)
    {
        fscanf(fp, "%s%d", &letter[i], &number[i]);
        Dif[i] = number[i] - average(number);
        if(min > fabs(Dif[i])){/*最小値を求めてそのときの添え字の保存*/
            min = fabs(Dif[i]);
            num = i;
        }
        i++;
    }
    printf("平均値に最も近い都市は%s\n標準偏差は%.1lf\n", &letter[num],SD(number));
    fclose( fp );
    return 0;
}

### 試したこと
試しにwhlie(i < N)の中にprintf("%s", &letter[i]);をいれたら綺麗な日本語が出力されたのですが，下から4行目のprintf("平均値に最も近い都市は%s\n標準偏差は%.1lf\n", &letter[num],SD(number));ではnumがうまく求まってないのかと思い，適当な数字をいれたりしてもうまく表示されませんでした．

###補足情報
data11.txtは以下のように都市名が並んでおり，都市名と人口の間には空白があります．
神奈川県横浜市 3777491
大阪府大阪市 2752412
愛知県名古屋市 2332176
北海道札幌市 1973395
福岡県福岡市 1612392
神奈川県川崎市 1538262
兵庫県神戸市 1525152
京都府京都市 1463723
埼玉県さいたま市 1324025
広島県広島市 1200754
宮城県仙台市 1096704
千葉県千葉市 974951
東京都世田谷区 943664
福岡県北九州市 939029
．
．（省略）
．
高知県土佐郡大川村 366
奈良県吉野郡野迫川村 357
新潟県岩船郡粟島浦村 353
沖縄県島尻郡渡名喜村 346
東京都大島支庁利島村 327
東京都三宅支庁御蔵島村 323
東京都八丈支庁青ヶ島村 169

melian

2022/07/17 01:32

letter を2次元の配列に。しかし、その方法で平均値を調べても意味がない様な(変則的な移動平均？) char letter[N] => char letter[N][128] fscanf(fp, "%s%d", &letter[i], &number[i]); => fscanf(fp, "%s%d", letter[i], &number[i]); printf("平均値に最も近い都市は%s\n標準偏差は%.1lf\n", &letter[num],SD(number)); => printf("平均値に最も近い都市は%s\n標準偏差は%.1lf\n", letter[num],SD(number));

jimbe

2022/07/17 03:40 編集

コードのマークダウン内には画面表示, コードやテキストファイル(data11.txt)毎に分けて、それぞれの内容のみを入れて頂いたほうが分かり易くコピペし易いです。

kakaka-014

2022/07/17 05:21

ご指摘ありがとうございます．書式の変更と，ソースコードの変更をします．

kakaka-014

2022/07/17 05:30

質問編集の更新でエラーになってしまうので以後ご指摘いただいたことを気を付けます．

行動規範の内容に同意します

回答2件

ベストアンサー

平均は全てデータを読み込んだ後で、求めないといけません。
そのあとで、全データをチェックし、平均に一番近いデータを算出します。
letter[N]は１件につき256バイト必要とする前提なら、letter[N][256]のように定義します。
minの初期値が与えられていません。（もしくは１件目にminを無条件にセットする処理が必要）
sd,minの型をdouble型へ変更すべきです。
average関数を毎回呼び出しているが、本来は１回呼び出せば済むはずです。
（この件は遅くなるだけなので、このままにしておきます）
上記を考慮した結果は、以下のようになります。

C
1#include<stdio.h>
2#include<math.h>
3#define N 1740
4double average(int number[])/*平均値を求める*/
5{
6    double ave,all = 0; int j;
7    for(j = 0 ; j < N; j++)
8    all += number[j];
9    ave = (all)/N;
10    return(ave);
11}
12double SD(int number[])/*標準偏差を求める*/
13{
14    int k; double Dif[N], Sum = 0.0,sd;
15    for(k = 0 ; k < N ; k++)
16    {
17        Dif[k] = number[k] - average(number);
18        Sum += ((Dif[k]*Dif[k])/N);
19    }
20    sd = sqrt(Sum);
21    return(sd);
22}
23int main(void)
24{
25    FILE *fp = fopen("data11.txt", "r");
26    int number[N],i = 0,l,num; char letter[N][256]; double Dif[N],min;
27    while(i < N)
28    {
29        fscanf(fp, "%s%d", letter[i], &number[i]);
30        i++;
31    }
32    i = 0;
33    min = 9999999999999;
34    while(i < N)
35    {
36        Dif[i] = number[i] - average(number);
37        if(min > fabs(Dif[i])){/*最小値を求めてそのときの添え字の保存*/
38            min = fabs(Dif[i]);
39            num = i;
40        }
41        i++;
42    }
43    printf("平均値に最も近い都市は%s\n標準偏差は%.1lf\n", letter[num],SD(number));
44    fclose( fp );
45    return 0;
46}
47

投稿2022/07/17 02:24

tatsu99

総合スコア5438

usekay

2022/07/17 04:15

letterはたぶんletterではなく文字ではなく文字名を入れる想定かと… 変数名が不適切ですね。

行動規範の内容に同意します

処理系が一切不明のため確実なことは言えませんがたぶん「�」の羅列はコアダンプですね。
文字コードに存在しないコードかフォントに入ってないかで化けてんでしょう。

引用テキスト*** stack smashing detected ***: terminated
引用テキストAborted (core dumped)

なんてでちゃってますし。
原因はスタック不足なんでスタックを増やせばいいが処理系が不明なので対応不能っぽい。
あと『 int number[N],i = 0,min,l,num; char letter[N]; double Dif[N];』でminが未初期化なんで不定値を参照して意図しない動作になりそう。
アルゴリズムがそもそもあってるかは見てないのでわかりません。

投稿2022/07/17 04:41

usekay

総合スコア395