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

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

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

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

Q&A

解決済

2回答

5469閲覧

Abort trap 6とはなにか、またその対処法について[C言語]

K-tom

総合スコア7

C

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

0グッド

0クリップ

投稿2018/02/22 16:57

前提・実現したいこと

勉強として調べながらビンゴのプログラムを作成していると、コンパイルは出来たのですがたまに「Abort trap 6」と表示されます。
この文字が表示されたときは、20個の内1つの数値がバグっている状態です。

質問したいことは2つありまして、
0. そもそもAbort trap 6とはなにか?
0. この問題を解決するにはどうすればいいか?

これらの点について、よろしくお願いします。

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

Abort trap 6

該当のソースコード

#include <stdio.h> #include <stdlib.h> #define N 20 #define size 76 void shuffle(int ary[],int x){ for(int i=0;i<size;i++){ int j = rand()%x; int t = ary[i]; ary[i] = ary[j]; ary[j] = t; } } int main(void){ int val[size]; for(int i=0;i<size;i++){ val[i] = i + 1; } srand(time(NULL)); int y = rand() % 100 + 1; shuffle(val,y); for(int j = 0;j<20;j++){ printf("%2d回目:%2d\n", j+1, val[j]); } return 0; }

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

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

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

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

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

guest

回答2

0

ベストアンサー

c

1 shuffle(val,y); 2 3```の y は 4```c 5 int y = rand() % 100 + 1; 6 7```なので 1100 ですが shuffle 内で 8```c 9 int j = rand()%x; 10 11```によって j は 099 になる可能性があります。 12この j はその後に 13```c 14 ary[i] = ary[j]; 15 ary[j] = t; 16 17```としているので ary[j] とアクセスされているため ary[]100(099)の配列が必要です。 18しかし ary[] は val[] の事なので size で定義された 76 個の配列です。 19乱数の出方によって 76-99 へのアクセスが成立した時には確保された配列の外に不正にアクセスする事になってしまいます。 20 21 221.そもそもAbort trap 6とはなにか? 23 24おそらく確保されていない不正な領域への不正なアクセスによって発生しているエラーだと思います。 25 262.この問題を解決するにはどうすればいいか? 27 28確保しているメモリ(配列)の範囲をキチンと把握して不正な未確保領域へのアクセスが発生しないように対策すれば解決すると思われます。

投稿2018/02/23 01:15

HiroshiWatanabe

総合スコア2160

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

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

K-tom

2018/02/24 15:13

わざわざ回答ありがとうございました! 配列の要素数のミスでしたか… 以後気をつけます!ありがとうございました!
guest

0

c

1 int y = rand() % 100 + 1; 2 shuffle(val,y);

yは1~100の値となりますのでshuffle関数内で配列の要素数を超えたアクセスが発生してしまいます。

せっかく冒頭でsizeおよびNをdefineしているので
main関数内の直接数値が打ち込まれてる個所(マジックナンバー)をsizeおよびNで置き換える事をオススメします。
1と0以外は置き換える事が可能なはずです

投稿2018/02/22 17:36

asm

総合スコア15147

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問