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

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

ただいまの
回答率

90.51%

  • C

    4514questions

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

  • UNIX

    429questions

    UNIXとは、AT&Tのベル研究所で開発されたコンピューター用のマルチユーザー・マルチタスクのオペレーションシステム(OS)です。政府や教育機関や研究所で広範囲に採用されています。

  • Emacs

    155questions

    GNU Emacsと拡張性の高い、高性能なテキストエディタです。豊富なライブラリの導入により、統合開発環境やWebブラウザとしても機能させる事が可能です。

C言語でwhileとif文を組み合わせたコードその2のエラー

解決済

回答 3

投稿

  • 評価
  • クリップ 1
  • VIEW 597

tenjin

score 222

前提・実現したいこと

文字を入力して、その文字が小文字アルファベットの場合は、その文字の次の大文字を返し、それ以外の場合は何もしないという処理を100回繰り返すコードを書いています。

実現したい結果

input a small character = z
next is = A
input a small character = 6
input a small character = a
next is = B

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

input a small character =input a small character =
のようにwhileループの2回目以降に2回連続でprintfされてしまうのと

for文の中で、小文字のアルファベットを大文字にする際にどのように処理したら良いのかわからず困っています。

該当のソースコード

#include<stdio.h>
#include <ctype.h>
int main(){
  char a;
  int i = 0;
  while(i<100){
    printf("input a small character =");
    scanf("%c", &a);
    if(a>='a' && a<='z'){
      char b = toupper(a);
      printf("capital is =%c\n", b);
    }
    i = i + 1;
  }
  return 0;
}

現状のコードの実行結果

% ./ex2_6
input a small character =a
capital is =A
input a small character =input a small character =1
input a small character =input a small character =9
input a small character =input a small character =
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

+2

入力として、aEnterキーを押すと、最初のscanfでは'a'が得られ、次のscanfでは\n(改行)が得られます。この改行に対しても、処理が進むために、一度のEnterに対して2回ループを回るので、そういう現象になります。

案1:scanfを使うのを止めて、fgetssscanfを使う
案2:改行を読み飛ばす

    do {
        scanf("%c",&a);
    } while(a=='\n');


案3:行頭の文字以降、改行まで全て読み飛ばす

    char x;
    scanf("%c",&a);
    x = a;
    while(x!='\n'){
       scanf("%c",&x);
    }


案4:scanfのパターンをちゃんと勉強する

    scanf(" %c",&a);

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/13 22:10

    ご回答いただきましてありがとうございました。

    キャンセル

  • 2017/11/13 22:11

    案4:scanfのパターンをちゃんと勉強する
    についてもう少し詳しくお聞かせいただけますと幸いです。

    キャンセル

  • キャンセル

  • 2017/11/16 10:20

    ありがとうございます。

    キャンセル

checkベストアンサー

+1

見逃されているようなので。

小文字アルファベットの場合は、その文字の次の大文字

各文字にはキャラクタコードという数字がわりあてられています。
なので、基本的には次の文字は、+1してあげればよいです。(zの場合は、マイナス25すればaに戻ります)

あとは、小文字のアルファベットの判定は関数がありますので、それを使うのがいいかと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/13 22:10

    ご回答いただきましてありがとうございました。
    教えていただいた方法で解決できました。

    キャンセル

+1

scanfで文字を取得する際、改行コードがバッファに残ってしまっています。
whileブロックの終わる前にgetcharを置くのが最も単純な対処法かと思います。

また、ご提示のコードの場合、ループはfor文で記述した方が良いかと思われます。

コメントを受けて

バッファについて
入力はまずバッファ領域に置かれ、それから書式指定子に従って値を変数に保管します。
たとえば、こんなコードを書いたとします。

#include <stdio.h>

int main(void) {
    char hoge;
    char fuga[100];

    scanf("%c", &hoge);
    scanf("%s", fuga);

    printf("%c, %s\n", hoge, fuga);
    return 0;
}

『一文字入力してから、文を入力』という意図かと思いますが、これを無視すると...

hogehoge
h, ogehoge

こんな風になるんですね。(一行目が入力、二行目が出力)
一回目のscanfで読み切れなかった文字列が二回目に回ってしまうのです。


for文について
for文とwhile文は、だいたいのケースで置き換え可能です。
しかしそれでは分かりにくいため、なんとなく使い分けが存在します。

for(int i = 0; i < 10; i++) {を見ただけで、10回繰り返すことが自明になります。

もう少し使い分けについて書こうと思ったのですが、なんとも不明瞭になってしまうのでやめました。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/13 21:46

    ご回答いただきましてありがとうございます。
    なぜfor文の方が今回のコードでは適切なのでしょうか。
    改行コードがバッファに残ってしまうということに関しても、初学者で恐縮ですが、もう少し詳しくお聞かせいただけますと幸いです。

    キャンセル

  • 2017/11/16 10:21

    ご丁寧に回答を編集いただきましてありがとうございます。大変勉強になりました。

    キャンセル

同じタグがついた質問を見る

  • C

    4514questions

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

  • UNIX

    429questions

    UNIXとは、AT&Tのベル研究所で開発されたコンピューター用のマルチユーザー・マルチタスクのオペレーションシステム(OS)です。政府や教育機関や研究所で広範囲に採用されています。

  • Emacs

    155questions

    GNU Emacsと拡張性の高い、高性能なテキストエディタです。豊富なライブラリの導入により、統合開発環境やWebブラウザとしても機能させる事が可能です。