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

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

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

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

Q&A

解決済

3回答

940閲覧

do文ループ中のfor文の繰り返しができない

snake_men

総合スコア7

C

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

0グッド

0クリップ

投稿2018/05/24 12:52

編集2018/05/24 14:38

do文ループでのバグ

あるサイトからお借りしたソースコードを改造したのですが、コンパイルするとdo文中のfor文が繰り返されず、do文直後の文と最後の文しか実行されないバグが起きています。
家にある本を見てもわからず、どうすればいいのでしょうか。
※試しに1つ目のfor文の}の場所を調整してみましたが、どうしてもfor文の繰り返しができませんでした。
あるサイトのURLはhttp://saeki-ce.xsrv.jp/C_src/soinsuu01.htmlです。
そのままのソースコードでは問題なく動作します。

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

★素因数分解★ 分解する整数: 12 素因数:2 乗数:2 素因数:3 乗数:1 1:やめる 0:もう一回:0 ★素因数分解★ 分解する整数: 1:やめる 0:もう一回:

該当のソースコード

C

1/****************/ 2/* 素因数分解 */ 3/****************/ 4#include <stdio.h> 5#include <stdlib.h> 6 7int main(void) 8{ 9 char c[128]; 10 int n0,n; // 整数 11 int ns=2; // 素因数 12 int j; // 乗数 13 int retry; // 繰り返し 14 15 do{ 16 printf("\n★素因数分解★\n"); 17 printf("\n分解する整数: "); fflush(stdout); 18 gets(c); n0=atoi(c); // 整数を入力 19 20 for( n=n0,ns=2; n>=ns; ns++ ) 21 { 22 for( j=0; n%ns==0; j++ ) // 割り切れる間繰り返す 23 { 24 n/=ns; 25 } 26 27 if ( j==0 ) continue; // 1回も割り切れなかった 28 29 printf("素因数:%d 乗数:%d\n",ns,j); 30 } 31 printf("1:やめる 0:もう一回:"); scanf("%d", &retry); 32 } while(retry == 0); 33 34 return 0; 35}

試したこと

}の位置を変えた

補足情報(FW/ツールのバージョンなど)

gcc 6.3.0

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

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

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

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

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

Orlofsky

2018/05/24 13:00

差支えない限り、「あるサイト」のURLを載せては?そのサイトのコードをそのまま実行したら正常に動作しますか?
snake_men

2018/05/24 14:39

orlofskyさん、ご指摘をいただきありがとうございます。
guest

回答3

0

ベストアンサー

惜しい。
期待通りに動作しないのは、scanfの使い方が間違っているせいです。
scanfは初心者が使ってはいけない関数です(正しい使い方が非常に難しい)。
また、getsは誰であれ使ってはいけない関数です(入力最大長(=変数サイズ)を指定できないため)。

★★★★★の2行だけ修正です。

C

1/****************/ 2/* 素因数分解 */ 3/****************/ 4#include <stdio.h> 5#include <stdlib.h> 6 7int main(void) 8{ 9 char c[128]; 10 int n0,n; // 整数 11 int ns=2; // 素因数 12 int j; // 乗数 13 int retry; // 繰り返し 14 15 do{ 16 printf("\n★素因数分解★\n"); 17 printf("\n分解する整数: "); fflush(stdout); 18 fgets(c,sizeof c,stdin); n0=atoi(c); // 整数を入力 ★★★★★ 19 20 for( n=n0,ns=2; n>=ns; ns++ ) 21 { 22 for( j=0; n%ns==0; j++ ) // 割り切れる間繰り返す 23 { 24 n/=ns; 25 } 26 27 if ( j==0 ) continue; // 1回も割り切れなかった 28 29 printf("素因数:%d 乗数:%d\n",ns,j); 30 } 31 printf("1:やめる 0:もう一回:"); 32 fgets(c,sizeof c,stdin); retry=atoi(c); // 整数を入力 ★★★★★ 33 } while(retry == 0); 34 35 return 0; 36}

投稿2018/05/24 13:14

otn

総合スコア84423

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

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

snake_men

2018/05/24 14:59

ご回答ありがとうございます。おかげさまでしっかりループするようになりました。 scanf関数ではなく、fgets関数を使えばいいのですね。ありがとうございます。
guest

0

原因

C

printf("1:やめる 0:もう一回:"); scanf("%d", &retry);

この行で、改行文字がバッファに残ってしまい、

C

gets(c); n0=atoi(c);

入力を待たずにそれを読み取ってしまっています。

解消するには

次の三つの方法が考えられます。

  • 入力を scanf に統一する
  • 入力を gets で受け取り、 atoi で数値化する方法に統一する
  • do-while文末尾に、バッファのゴミを回収するコードを書く

簡単に対応できるのは最初でしょうね。

投稿2018/05/24 13:00

LouiS0616

総合スコア35658

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

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

0

こんにちは、
原因は入力の際scanfを使っているからだと考えれます。
(scanf関数は色んな意味で危険です。)
入力の時はfgetsを使うことをおすすめします。

また、不要な変数があったため、少し手を加えました。

C

1#include <stdio.h> 2#include <stdlib.h> 3 4#define BUFSIZE 256 5 6int main(void) 7{ 8 char buf[BUFSIZE]; 9 int number; // 整数 10 int i; // 素因数 11 int j; // 乗数 12 int retry; // 繰り返し 13 14 do{ 15 printf("\n★素因数分解★\n"); 16 printf("\n分解する整数: "); 17 18 fgets(buf,sizeof(buf),stdin); 19 number = atoi(buf); 20 21 for(i = 2; number >= i; i++ ){ 22 for(j = 0; number % i == 0; j++){ 23 number /= i; 24 } 25 if (j == 0){ 26 continue; 27 } 28 printf("素因数:%d 乗数:%d\n",i,j); 29 } 30 printf("1:やめる 0:もう一回:"); 31 fgets(buf,sizeof(buf),stdin); 32 retry = atoi(buf); 33 } while(retry == 0); 34 35 return 0; 36}

投稿2018/05/24 13:18

編集2018/05/24 13:54
退会済みユーザー

退会済みユーザー

総合スコア0

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

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

退会済みユーザー

退会済みユーザー

2018/05/24 13:19

あっ、すでにほぼ同じ内容の回答がある......
snake_men

2018/05/24 15:11

こちらの方でも無事コンパイルできました。 自分がもともと改造していたソースコードは、マジックナンバーが残っていたので、 このソースコードも大変勉強になりました。ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問