c言語のgetcharの構文をfgetsの構文に変換する方法を教えて欲しい

Question

前提

100文字以内で入力した英文をすべて大文字に返す。ただし、EOFでプログラムを終了させる。

実現したいこと

getcharやputcharを用いて以下のようなソースコードを作ったが、fgetsやputsを用いてよりスマートなソースコードにできるのか知りたいです。また、それ以外の方法でもスマートにできるものがあれば教えて欲しいです。

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

100文字を越える英文を入力したときにエラー判定が出ずにそのまま次のループの入力とされてしまう。

該当のソースコード

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

int main(void)
{
int i=1,ch;

printf("Please input a sentence with 100 characters or less.\nPlease press Ctrl-Z to stop this program.\n");
ch = getchar();
while(1)
{
while(i<100)
{
	if(ch == EOF)
	{
		break;
	}
	else if((ch >=97)&&(ch <= 122))
	{
		ch= ch-32;
		i++;
	}
	else if((ch >=65)&&(ch <= 90))
	{
		i++;
	}
	else if(ch == 10 )
	{
		break;
	}
	putchar(ch);
	ch=getchar();
}
if(ch == EOF)
	{
		break;
	}
printf("\n");
printf("Please input a sentence with 100 characters or less.\nPlease press Ctrl-Z to stop this program.\n");
ch = getchar();
i=1;
}

return 0;
}

Comments

[dodox86]

> 100文字を越える英文を入力したときにエラー判定が出ずにそのまま次のループの入力とされてしまう。

コードをスマートにする云々の前に、この問題を解決する必要がありませんか。　コードのロジックの問題です。

[pig_vba]

最後のch = getchar();がおかしいです。これでは1文字しか読み込まないので102文字以上含まれていた場合標準入力に残ったまま次のループに入ります。
バッファクリアがしたいならwhile(getchar()!='\n');とすべきです。
エラー判定が出ないのはそもそもエラー処理を入れてないからですし、EOFも即Returnでいいし、現時点でもかなり修正が必要かなと思いますね…

Answer 1

getchar、putchar は一文字づつ入力し、一文字づつ出力する関数です
当然ながら、１文字読み込んでは変換し、１文字出力するというコードをループさせて実装することになります

fgets、puts は、１行分入力し、１行分出力する関数です
１行入力したあと、その文字列を変換し、結果の１行を出力するというコードとなります

Answer 2

とりあえず書いてみました。動作確認してないので不具合が発生したら言ってください

#include <stdio.h>
#include<string.h>
void upperstring(char *c);

int main(void)
{
    char buffer[256];
    size_t length;
    
while(1)
{
   printf("Please input a sentence with 100 characters or less.\nPlease press Ctrl-Z to stop this program.\n");

    //eofなら終了
    if (fgets(buffer, 256, stdin) == NULL) {
        break;
        }
    //100文字以上ならエラー(該当記述がなかったので今回はエラー終了とする)
    length = strlen(buffer);
    if (length>101) {
        printf("error.Your Input is Over 100 characters.\n");
        return 1;
        //再入力を促したい場合
        //while(getchar()!='\n');
        //continue;
    }
    upperstring(buffer);
    puts(buffer);
    printf("----------------\n");
 
}
return 0;
}

void upperstring(char *c){
   while(*c != '\0'){

        if('a' <=* c && *c <= 'z'){
            *c = *c - 0x20;
        }
        c++;
    }
}