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

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

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

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

Q&A

解決済

4回答

996閲覧

入力で格納した値が0になってします。

apeirogon0813

総合スコア117

C

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

0グッド

1クリップ

投稿2019/01/22 14:16

前提・実現したいこと

以下のプログラムにおいて入力として
3
0:1:10:2.000
0:2:10:4.000
1:2:10:3.000

と与えるのですが,3をeという変数に格納するのですが
直後のprintf文では3と出力されますが
最後の方のprint文では0と出力され値が変更されてしまっている原因がわかりません.
ご教示願います.

該当のソースコード

C

1#include<stdio.h> 2#include<stdlib.h> 3char buf[256]; 4 5int main() { 6 int e; 7 scanf("%d ", &e); 8 printf("e = %d\n",e); //ここでは読み込んだ値(3)が出力されます。 9 int adjmat[e][e]; 10 int eki1, eki2, i, j; 11 int a; 12 float b; 13 /* 0で初期化 */ 14 for(i=0; i<e; i++) 15 for(j=0; i<e; j++) 16 adjmat[i][j] = 0; 17 18 while(fgets(buf,sizeof(buf),stdin)!=NULL) { 19 sscanf(buf, "%d:%d:%d:%f␣", &eki1, &eki2, &a, &b); 20 adjmat[eki1][eki2] = adjmat[eki1][eki2] = 1; 21 } 22 printf("e = %d\n",e); //ここで0と出力されてしまいます。 23 for(i=0; i<e; i++) 24 for(j=0; i<e; j++) 25 printf("%d\n", adjmat[i][j]); 26 27 return 0; 28} 29 30

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

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

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

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

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

guest

回答4

0

daisuke7さんの指摘の通りで、配列の添え字をオーバーして書き込んでしまうために、他の変数の値を破壊して、それ以降めちゃくちゃになってしまっています。

投稿2019/01/22 14:43

otn

総合スコア84423

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

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

apeirogon0813

2019/01/22 14:51

ありがとうございます。気づきませんでした。。
guest

0

ベストアンサー

何をやっているかまったく読んでいませんが、とりあえず、

C

1 for(i=0; i<e; i++) 2 for(j=0; i<e; j++) 3 adjmat[i][j] = 0; 4

C

1 for(i=0; i<e; i++) 2 for(j=0; i<e; j++) 3 printf("%d\n", adjmat[i][j]); 4

どちらも内側のループの条件が j<e ではなく i<e になっています。

投稿2019/01/22 14:24

daisuke7

総合スコア1563

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

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

apeirogon0813

2019/01/22 14:51

初歩的なミスでしたありがとうございます。
guest

0

既に回答が付いていますが、

C

1 for(i=0; i<e; i++) 2 for(j=0; i<e; j++) 3 adjmat[i][j] = 0;

内側のループの終了条件が、 i<eなので、adjmatの範囲外をアクセスしている可能性大です。 ..... というより、エラー終了しないのが不思議。

この時に、eの値も壊している(0を設定している)と思われます。

投稿2019/01/22 14:32

pepperleaf

総合スコア6383

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

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

apeirogon0813

2019/01/22 14:52

ありがとうございます。見過ごしていました。
pepperleaf

2019/01/22 14:55

追記したら、エラーになりました。 多分、e を壊した時点でループが終了するので、そこで停止ですね。(多分)
guest

0

とりあえず 2箇所ありますが、

c

1 for(i=0; i<e; i++) 2 for(j=0; i<e; j++)

c

1 for(i=0; i<e; i++) 2 for(j=0; j<e; j++)

ですね。

投稿2019/01/22 15:09

fu7mu4

総合スコア1088

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問