Illegal instructionの原因について

Question

前提・実現したいこと

C言語で名前と数字を入力していき1番小さい数字と2番目に小さい数字の人の名前を表示させるプログラムを作りたいです。

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

現在名前と数字を入力することができるのですが2人目の名前と数字を入力した後にIllegal instructionというようなエラーが出てきてしまいます。
Illegal instructionについて調べてみたのですが情報が少なくあまりわからなかったのでIllegal instructionについても説明していただけるとありがたいです。

エラーメッセージ
Illegal instruction

ソースコードです。

#include <stdio.h>
#include <string.h>

int main (void) {
    int i;
    char name[1000];
    int number[1000];
    char min_name_1[1000];
    int min_number_1[1000];
    char min_name_2[1000];
    int min_number_2[1000];
    char ans_name_1;
    char ans_name_2;

    for(i = 0; i < 1000; i++) {
        if((scanf("%s",&name[i])) == EOF) break;
        scanf("%d", &number[i]);
        if(i == 0) {
            min_name_1[i] = name[i];
            min_number_1[i] = number[i];
            min_name_2[i] = name[i];
            min_number_2[i] = number[i];
        } else {
            if(number[i] < min_number_1[i]) {
                min_name_2[i] = min_name_1[i];
                min_number_2[i] = min_number_1[i];
                min_name_1[i] = name[i];
                min_number_1[i] = number[i];
                strcpy(&ans_name_1,&min_name_1[i]);
                strcpy(&ans_name_2,&min_name_2[i]);
            }
        }
    }
    printf("%c\n",ans_name_1);
    printf("%c\n",ans_name_2);
    return 0;
}

Comments

[cateye]

min_name_1[i] = name[i];って何をしているんでしょう？・・・ほかも・・・
文字列のコピーはstrcpy()を使いましょう。→http://www.c-tipsref.com/reference/string/strcpy.html

[Daregada]

別の質問でこの問題見た覚えが……どこの課題なんだろう。
それはともかく、「1000人分の名前を覚えられるようにした」つもりなんだろうけど、実際には「最大1000文字分の名前ひとつしか覚える場所を作ってない」ように見えます。

[thkana]

既視感の原因はこれですね
https://teratail.com/questions/265979

[Daregada]

私の既視感はこっちでした
https://teratail.com/questions/268235

[hatena00]

最大1000文字分の名前ひとつしか覚える場所を作ってないとありますがどのように書けば1000人分の名前を覚えられるようにできるのですか、、？

Answer 1

エラーメッセージは、主要な単語だけでなく、文として書きましょう。
Illegal instructionだけでは、単に不正な命令を意味するだけ。まあ、そうなんでしょうが、それ以上の情報は無く、手掛かりになりません。

2人目の名前と数字を入力した後に

との事なので、 i=1の場合を考えると、 if(i == 0) {が偽となり、} else {の方に分岐する。最初の if の判定 number[i] < min_number_1[i]の
min_number_1[i] ... i=1 とした時、値ははいっているでしょうか?
.. と続き、 strcpy(&ans_name_1,&min_name_1[i]); ここで、ans_name_1 の宣言を見ると、char ans_name_1; 一文字分のサイズしかありません。 (その前も怪しいですが、) これは、確実に他の領域を破壊(上書き)するので、その後の動作は不明になります。
この辺でしょうか。

Comments

[hatena00]

1文字分のサイズしかないとありますがどのようにすれば複数文字入れられるのでしょうか？

[pepperleaf]

10文字(バイト)分だと、 char ans_name_1[10]; ただし、C言語は、'\0'で終わるというルールがあるので、+1 して、char ans_name_1[11];
ここは、これでも良いですが、 char name[1000]; とかで宣言されている変数は、(この場合)1000文字分 の文字しか用意されません。これを name[i] で参照する場合、 i 番目(正確には i+1)の文字(1文字)を見る事になります。scanf()の引数としては、 &でアドレス取ってるのでエラーにはなりませんが、ループを回しても 1バイト毎にずらした位置への読み込みとなります。
全般的に、C言語の文字の扱いに問題があると思います。

[cateye]

Cの、文字と文字列の違いをはっきりさせましょう。
C言語での文字列とは、終端記号('\0')で終わる文字の配列のことです。

[pepperleaf]

C言語の文字の扱いじゃなくて、文字列の扱いですね。
文字列が 文字の配列なので、複数の文字列を扱うには、配列の配列、2次元配列(char name[N][M])にするが一般的でしょうか。