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

###前提・実現したいこと
ここに質問したいことを詳細に書いてください
文字が小文字アルファベットの場合は、入力文字を入力してその大文字を返し、
それ以外の場合は、新しい入力を促すコードを書いています。
100回の反復後にプログラムが終了するように設計しています。

実行したい結果

input a small character = 7
input a small character = 0
input a small character = k
capital is = K

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

いただいたアドバイスを元にコードを修正しましたが、実行すると100回ではなく一度しか実行されないという問題が発生しています。

###該当のソースコード

c
1#include<stdio.h>
2#include <ctype.h>
3int main(){
4  char a;
5  int i = 0;
6    while(i<100){
7        printf("input a small character =");
8        scanf("%c", &a);
9   if(a>='a' && a<='z'){
10        char b = toupper(a);
11        printf("capital is =%c\n", b);
12        }  
13    i = i +1;
14    return 0;
15  }
16}

行動規範の内容に同意します

回答4件

ベストアンサー

日本語がおかしいだけかと思ったら、そのおかしい日本語の通りにプログラムを書いていますね。結果的におかしいプログラムになっています。

文字が小文字アルファベットの場合は、入力文字を入力してその大文字を返し、

それ以外の場合は、新しい入力を促すコードを書いています。

最初に何を判断しているつもりでしょうか？まず入力しないと、判断する文字がありません。

以下を100回繰り返す。文字を入力して、その文字が小文字アルファベットの場合は、その大文字を返し、

それ以外の場合は何もしない。

というのが正しい処理です。

あと、直接質問とは関係無いですが、scanfは、初心者が使うには難しい関数です。まあscanf("%c",&a);くらいであれば、ほぼa=getchar();と同じ機能なので、難しくないのですが、一般には書式指定文字列を熟知していないと使いこなせない上級者向けの関数です。
また、もしscanfを使うなら、返値を必ずチェックしましょう。

投稿2017/11/13 03:08

編集2017/11/13 03:12

otn

総合スコア84529

退会済みユーザー

2017/11/13 03:38 編集

ご丁寧にアドバイスいただきましてありがとうございます。 scanfは難しい関数であるということを存じ上げませんでした。課題で必須なので、初心者ですが書式指定文字列とともに勉強します。

otn

2017/11/13 03:28

回答に書いた通り、`scanf("%c",&a);`くらいであれば大丈夫です。書式が複雑な場合、初心者には、fgets + sscanf をお薦めします。いずれにせよ、返値を必ず確認して下さい。文章通りにプログラムを書くスキルはお持ちのようなので、ロジックが整理できれば大丈夫だと思います。

退会済みユーザー

2017/11/13 03:38

貴重なアドバイスをいただきましてありがとうございました。

otn

2017/11/13 03:46

> 一度しか実行されない returnしているからですね。

行動規範の内容に同意します

100回は入力が大変なので１０回のサンプルプログラム

c
1#include<stdio.h>
2#include <ctype.h>
3int main(){
4  char a;
5  int i = 0;
6    while(i<10){
7       printf("input a small character =");
8       scanf("\n%c", &a);
9       if(a>='a' && a<='z'){
10            char b = toupper(a);
11            printf("capital is =%c\n", b);
12       }
13       i = i +1;
14  }
15  return 0;
16}