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

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

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

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

Q&A

解決済

2回答

4333閲覧

C言語で乱数を発生させて挿入ソートを行いたい

clematis

総合スコア13

C

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

0グッド

0クリップ

投稿2018/10/03 14:24

前提・実現したいこと

プログラミング初心者です。
学校の授業の実習で、乱数を発生させたのち挿入ソートを行って整列させるプログラミングを扱いました。
paiza.ioにて実行したのですが、エラーメッセージはなく、画面上部に以下のような文章が表示されます。

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

Runtime error(Exit status:139(Invalid memory reference))

該当のソースコード

C

1#include <stdio.h> 2#include <stdlib.h> 3int BubSort(int x[ ], int n); 4void ShowData(int x[ ], int n); 5 6#define NUM_DATA 100 7int count = 0; 8 9 /* 挿入ソート */ 10int InsertionSort(int a[ ], int n) 11{ 12 int i, j; 13 14 for (i=1; i<n; i++){ 15 int tmp = a[i]; 16 for (j=i; j>0&&a[j-1]>tmp; j--) 17 a[j]=a[j-1]; 18 a[j]=tmp; 19 } 20 /* ソートの途中経過を表示 */ 21 ShowData(a, NUM_DATA); 22 printf("\n"); 23} 24 25 /* ソート対象のデータを表示 */ 26void ShowData(int x[ ], int n) 27{ 28 int i; 29 30 for (i = 0; i < n ; i++){ 31 printf("%d\t", x[i]); 32 } 33} 34 35int main(void) 36{ 37 38 int i; 39 int x[100]; 40 for (i=0; i<150; i++){ 41 x[i]=rand()%99+1; 42 } 43 44 /* ソート前のデータを表示 */ 45 printf("ソート前:\n"); 46 ShowData(x, NUM_DATA); 47 printf("\n\n"); 48 49 InsertionSort(x, NUM_DATA); 50 51 /* ソート後のデータを表示 */ 52 printf("\n\nソート後:\n"); 53 ShowData(x, NUM_DATA); 54 55 printf("交換回数"); 56 printf("\n\n%d\n", count); 57 58 return 0; 59 60} 61 62

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

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

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

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

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

guest

回答2

0

ベストアンサー

main()のforループで範囲外へのアクセス。

int x[100]; for (i = 0; i < 150; i++) { // ←ここ配列は100なのに150まで回してる x[i] = rand() % 99 + 1; }

lint(cppcheck)のメッセージ
t0.c:41: error: Array 'x[100]' accessed at index 149, which is out of bounds.

投稿2018/10/03 14:38

編集2018/10/03 14:42
cateye

総合スコア6851

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

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

clematis

2018/10/03 14:44

こんな些細なミスにご丁寧なコメントいただいてありがとうございます、、、 うまくいきました!
cateye

2018/10/03 14:56 編集

あと気になったのが、int InsertionSort()で、戻り値がないことですかね ・・・たぶん交換回数 (・ω・)・・・countはグローバルにする必要はないかと・・・
clematis

2018/10/03 15:06

ありがとうございます????????
guest

0

C

int x[100];
for (i=0; i<150; i++){
x[i]=rand()%99+1;
}

いきなり範囲外踏んでません?iが100になったときに。

投稿2018/10/03 14:31

LouiS0616

総合スコア35660

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

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

clematis

2018/10/03 14:44

本当に些細なミスでした! ご回答いただいてありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問