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

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

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

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

Q&A

解決済

2回答

16795閲覧

エラーが出てしまいました❗

yu-ri1177

総合スコア13

C

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

0グッド

0クリップ

投稿2016/11/28 10:50

編集2016/11/28 11:14

c言語でオセロプログラムを作っていますが、
Runtime error(Exit status:139(Invalid memory reference))
が実行したら出てきてしまいました。

まだc言語がわからないので教えてください。

ソースコード........
#include <stdio.h>

int put, turn, all, done, pass, count, value, i,
map[90] = {0}, dir[]={-10, -9, -8, -1, 1, 8, 9, 10};

void check()
{
if (map[put] == 0)
for (i=0; i<8; i++) {
for (count = 0, value = put+dir[i];
map[value] == 3-turn; value += dir[i])
count++;

if (count && map[value] == turn) { all += count; value = put; if (done) do map[value] = turn, value += dir[i]; while (map[value] != turn); } }

}
char *h=" - o x\n";

int main()
{
for(i=1, map[41] = map[49] = 2; i<10; map[i++*9] = 3)
map[40] = map[50] = turn = pass = 1;

for (;; all = done = 0) { for(put = 9; put<82; ++put) check(), printf("%.2s",&h[map[put]*2]); if(all) for(done = all = pass = put = 8; all==8; check()) turn - 2 ? (scanf("%d %d",&put,&i), put+=i*9): ++put; else if(pass) pass=0,printf("pass"); else break; turn = 3 - turn; } return 0;

}

です。

すみません

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

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

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

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

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

kpiyohiko

2016/11/28 10:56

エラーがでたソースコードとデバッグした際にどの部分でエラーが発生しているか、どのような環境で実行しているのかが不明です。
yu-ri1177

2016/11/28 11:14

すみません。よろしくお願い致します
guest

回答2

0

int put, turn, all, done, pass, count, value, i

これのどれかがオーバーフローしてないでしょうか
C言語のintの最大値は32767です。
デバッグはされましたでしょうか?
また、想定される最大値はいくつでしょうか。

投稿2016/11/28 11:40

kpiyohiko

総合スコア658

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

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

yu-ri1177

2016/11/28 13:17

回答ありがとうございました。 最大値と最低値は char のサイズ: 8 bit char の最大値: 127 char の最小値: -128 int の最大値: 2147483647 int の最小値: -2147483648 long の最大値: 9223372036854775807 long の最小値: -9223372036854775808 signed char の最大値: 127 shigned char の最小値: -128 short の最大値: 32767 short の最小値: -32768 unsigned char の最大値: 255 unsigned int の最大値: 4294967295 unsigned long の最大値: 18446744073709551615 unsigned short の最大値: 65535 でした。 分かりやすい解説ありがとうございました。m(__)m
guest

0

ベストアンサー

たぶんですが、printf("%.2s",&h[map[put]*2] のところでmap[put]*2の値が ポインタ h が指し示す文字列の領域を超えているのでは無いでしょうか?

以下のように範囲チェックの処理を入れて確認してください。

C

1// ・・・略・・・ 2char *h=" - o x\n"; 3 4int main() 5{ 6// ・・・略・・・ 7for (;; all = done = 0) { 8 for(put = 9; put<82; ++put) { 9 if (map[put]*2 < strlen(h)) { 10 check(), printf("%.2s",&h[map[put]*2]); 11 } else { 12 printf("system error. map[put]*2 hの長さを超えています。\n"); 13 exit(1); 14 } 15// ・・・略・・・ 16

もし上記が当たりであれば根本的な原因としては
map配列を0で初期化していないため、初期化していない配列要素をint値として使用しているため h の指す領域を超えてアクセス(バッファオーバーフロー)しているのだと思います。

投稿2016/11/28 12:40

Y.H.

総合スコア7914

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

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

yu-ri1177

2016/11/28 14:22

とても分かりやすく解説していただきありがとうございました。 教えて下さったことをやってみます‼ ありがとうございました_(._.)_
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問