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

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

ただいまの
回答率

91.33%

  • C

    2540questions

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

ダウンロードしたプログラムでエラーが出て実行できない

解決済

回答 1

投稿 2017/12/03 11:51 ・編集 2017/12/03 11:52

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

sanchu52

score 58

ダウンロードしたプログラムでエラーが出て実行できない。
URLはcodepad.org/B83f1FSq です。セグメンテーション違反が出ています。
ダウンロードしたプログラムに名前を付けて保存して実行しています。
debug結果(デバッグとは言えないと言われそうですが)は以下のとおりです。ここから前に進まない状態です。どこが悪いのかおしえていただければありがたいのですが。
お願いいたします。
naka@naka ~/kadai
$ gcc -g kad9-8-2c.c -Wall

naka@naka ~/kadai
$ gdb a.exe
GNU gdb (GDB) 7.6.1
Copyright (C) 2013 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "mingw32".
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>...
Reading symbols from C:\MinGW\msys\1.0\home\nakamura\kadai\a.exe...done.

(gdb) b main
Breakpoint 1 at 0x401b34: file kad9-8-2c.c, line 184.
(gdb) r
Starting program: C:\MinGW\msys\1.0\home\nakamura\kadai/a.exe
[New Thread 5180.0x32f4]
[New Thread 5180.0x28dc]
[New Thread 5180.0x12d0]
[New Thread 5180.0x4c8]
[New Thread 5180.0x33b0]
[New Thread 5180.0x1dd8]
[New Thread 5180.0xf14]

Breakpoint 1, main () at kad9-8-2c.c:184
184       char *token=",";
(gdb) step
187       head = 0;
(gdb)
188       if ((fp = fopen(FILENAME,"r")) != 0) {
(gdb)
189         while(fgets(buff, N, fp) != 0){
(gdb)
190           chop(buff);
(gdb)
chop (p=0x408120 <buff.2666> "naka") at kad9-8-2c.c:26
26        for (; *p; p++)
(gdb)
28        p--;
(gdb)
29        while (*p == '\r' || *p == '\n')
(gdb)
31      }
(gdb)
main () at kad9-8-2c.c:191
191           strcpy(name, strtok(buff, token));
(gdb)
192           strcpy(address, strtok(NULL, token));
(gdb)

Program received signal SIGSEGV, Segmentation fault.
0x74245623 in strcat () from C:\WINDOWS\SysWOW64\msvcrt.dll
(gdb)
Single stepping until exit from function strcat,
which has no line number information.

Program received signal SIGSEGV, Segmentation fault.
0x74245623 in strcat () from C:\WINDOWS\SysWOW64\msvcrt.dll
(gdb)
Single stepping until exit from function strcat,
which has no line number information.
[Inferior 1 (process 5180) exited with code 030000000005]
(gdb)
The program is not being run.
(gdb)

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

192           strcpy(address, strtok(NULL, token));

のstrtok()関数の戻り値がNULLになってませんか?
buffの中身または、"address.csv"の内容が正しいか確認してください。

追記

if ((fp = fopen(FILENAME,"r")) != 0) {
  while(fgets(buff, N, fp) != 0){
    char *p;
    chop(buff);
    printf( "ファイルから読んだ文字列:%s\n", buff );
    p = strtok(buff, token);
    if ( p != NULL ) {
      strcpy(name, p);
    } else {
      printf( "nameの切り出しに失敗しました。\n");
      continue;
    }
    p = strtok(NULL, token);
    if ( p != NULL ) {
      strcpy(address, p);
    } else {
      printf( "addressの切り出しに失敗しました。\n");
      continue;
    }
    p = strtok(NULL, token);
    if ( p != NULL ) {
      strcpy(number, p);
    } else {
      printf( "numberの切り出しに失敗しました。\n");
      continue;
    }
    p = strtok(NULL, token);
    if ( p != NULL ) {
      strcpy(mail, p);
    } else {
      printf( "mailの切り出しに失敗しました。\n");
      continue;
    } 
    list_add(&head, name, address, number, mail);
  }
  fclose(fp);
}  

投稿 2017/12/03 12:18

編集 2017/12/03 15:50

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/03 13:28

    "address.csv"の内容が間違っていました。修正して実行するとlist_add()はすべて実行されて、while{}から抜けるはずですが、抜けないでじっこうがはじまるみたいです。そしてまた同じようなエラーになります。
    つぎにメニューを選択してwhile{}に実行に移ることにならなければいけないと思います。よろしくおねがいいたします。判断材料が足りなければおしゃってください。

    main () at kad9-8-2a.c:244
    244 while(fgets(buff, N, fp) != 0){
    (gdb)
    245 chop(buff);
    (gdb)
    chop (
    p=0x408120 <buff.2666> ' ' <repeats 141 times>, "P 5432", ' ' <repeats 51 times>...) at kad9-8-2a.c:37
    37 for (; *p; p++)
    (gdb)
    58 p--;
    (gdb)
    59 while (*p == '\r' || *p == '\n')
    (gdb)
    61 }
    (gdb)
    main () at kad9-8-2a.c:247
    247 strcpy(number, strtok(buff, token));
    (gdb)
    248 strcpy(name, strtok(NULL, token));
    (gdb)

    Program received signal SIGSEGV, Segmentation fault.
    0x74245623 in strcat () from C:\WINDOWS\SysWOW64\msvcrt.dll
    (gdb)

    キャンセル

  • 2017/12/03 15:49 編集

    セグメンテーションフォルトが発生している直接原因は理解していますか?
    追記のようにコードを直して、期待通りに文字列の切り出しが成功しているか確認してください。

    キャンセル

  • 2017/12/03 16:24

    ありがとうございます。一応メニューが表示されました。途中に以下の表示が40個ぐらい表示された後に
    メニューが表示されます。なんででしょうか。
    naka@naka ~/kadai
    $ kad9-8-3a
    ファイルから読んだ文字列:naka,kamikosaka,080-4444,mail1-1
    ファイルから読んだ文字列:kato,kanagawa,080-8888,mail1-2
    ファイルから読んだ文字列:saito,yamanashi,080-6666,mail1-3
    ファイルから読んだ文字列:sato,tokyo ,090-3333,mail1-4
    ファイルから読んだ文字列:suzuki,saitama,090-2222,mail1-5
    ファイルから読んだ文字列: P 5432 naka
    addressの切り出しに失敗しました。
    ファイルから読んだ文字列: P 5432
    addressの切り出しに失敗しました。
    ファイルから読んだ文字列:
    ......
    addressの切り出しに失敗しました。
    ファイルから読んだ文字列:

    addressの切り出しに失敗しました。
    1:sort 2:delete 3:add 4:dump 0:quit
    select menu: 3
    input data to add.
    name: nasi
    address: oosaka
    tel: 6789
    mail: h-mail

    キャンセル

  • 2017/12/03 16:31

    できました。continueをbreakに直させていただきました。1歩前進しました。
    これから住所プログラム読みたいと思います。ありがとうございました。

    キャンセル

  • 2017/12/03 23:46

    breakに直しただけだとファイルの読み込みを途中でやめただけで本質的には改善してないと思いますが・・・。

    キャンセル

  • 2017/12/04 10:11

    ありがとうございます。一応すべてのメニューを試して正常に動いたのですが、間違っていますか。
    同じようなコードでエラーが出て実行できない状態なので、質問を新たにしたいと思います。そちらでまた教えてください。

    キャンセル

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

ただいまの回答率

91.33%

関連した質問

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

  • C

    2540questions

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