C言語のgetcharを使った\nと0の判断がうまくいかない

Question

実現したいこと

エンターは未入力として、０はエラーに判定されないようにしたい

数字以外、未入力の場合はエラーとしてもう一度入力できるようにするためにはどのようにすれば
良いかがわからなく行き詰ってしまったためなにかアドバイスなどいただけないでしょうか

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

0と入力した際、未入力の判定になってしまう

該当のソースコード

int main(void)
{
 int c;
 int i;
 int j;
 int num[256];
 int Nmax = 0;
 char str;


  for ( i = 0; i < 10; i++)
  {
    while (1)
    {
    num[i] = 0;
    str = 0;
    while ((c = getchar()) != '\n')
    {
      if (c >= '0' && c <= '9') 
      {
        num[i] = num[i]*10 + c -'0';
      }
      else
      {
        str = c;
      } 
    }
    if (str > 0)
    {
      num[i] = 0;
    }
    if (num[i] > 0) 
    {
      break;
    }
    printf("未入力です\n");
    }
    if (num[i] % 2 == 1)
    {
      if (num[i] > Nmax)
      {
        Nmax = num[i];
      }
    }
  }
  if (Nmax > 0)
  {
    printf("最も大きい奇数:", Nmax);
  }
  else
  {
    printf("奇数がありませんでした\n", Nmax);
  }
}

ブレイクの条件として
if (num[i] > 0)
{
break;
}
printf("未入力です\n");
}
上記のようにif(num[i] > 0)で条件付けしてるかだとは思うのですが
どのようにすればエンターのみ押下した場合と0だけ入力した場合の違いを判断させるかが思いつきません

Comments

[jimbe]

混乱してしまっているのでしたら、落ち着いて考えてください。ご自身で書いたコードを一番良く分かっているのはご自身しかいないのです。
0 も有効としたいのでしたら、なぜブレイクの条件として num[i] > 0 と 0 を抜いているのでしょうか。

[kotarou9]

コメントありがとうございます。
最初はnum[i]に数値が代入されていれば抜けさせる（エンターのみだとループさせる）つもりで作成していましたがチェックの段階で0を入力するとエンターの時と同様に未入力のエラーに引っかかることに気づきました。

[jimbe]

>num[i]に数値が代入されていれば

の「数値」に 0 が含まれることを想定されていなかったということでしょう。
今の num[i] の値の変化方法では "0" 入力と Enter のみの場合とが区別できないので、もう少し値の変化をダイナミック?にするとか、 otn さんの回答のようにフラグ変数を用いるかということになります。(「区別が付かないなら区別が付くようにする」という意味では同じことです。)

[kotarou9]

返信遅れてしまいすみません
回答・参考のコードまでありがとうございます！
おっしゃられる通り0が含まれることを想定していなかったです
もう一度値の変化の区別の仕方を変えてみて作り直してみます！

Answer 1

入力→変換→チェックと順に行うのも一つのテです。

#include <stdio.h>
#include <stdlib.h> //strtol

int main(void) {
  int max = 0;
  for(int count=0; count<10; ) {

    char s[32] = { 0 };
    for(int j=0, c; (c = getchar()) != '\n'; ) s[j++] = (char)c; //入力一行を s に貯める(改行は含まない)

    char *endp;
    int num = (int)strtol(s, &endp, 10); //s を 10 進数として数値化. 数字でない文字で止まり,そのアドレスが endp に入る

    if(endp == s || *endp != 0) { //Enter だけだったら endp==s となり、数字以外が有ったら *endp!=0 となる
      printf("未入力です\n");
      continue;
    }

    if(num % 2 == 1 && max < num) max = num; //奇数の最大値を更新
    count++; //入力数カウントアップ
  }
  if(max > 0) {
    printf("最も大きい奇数: %d\n", max);
  } else {
    printf("奇数がありませんでした\n");
  }
}

Answer 2

「有効な数字入力があったかどうかフラグ」を設けるのでしょうか。

str = c;

という変数strの意味が不明ですが、そのあたりを意図した物だったのでしょうか？

あと、「123a4」のような入力の時にどうしたいかですね。これも「未入力です」にしたいのなら、数字以外が入力されたら、「有効な数字入力があったかどうかフラグ」を偽にリセットする。
「1234」という値として受け付けたいのなら、数字以外でもリセットしない。

コード例 追記

上記の、「あと、」以降がちょっとおかしかったので、訂正含めてコードを書いてみます。
numを配列にする意味が無いので、単純変数にしました。ソースはindent -nut -br -ceで整形しています。

#include <stdio.h>

#define VALID 1
#define INVALID 2
#define UNDEF 0

int
main (void)
{
  int c;
  int i;
  int num;
  int Nmax = 0;
  int is_number;

  for(i = 0; i < 10; i++) {
    while(1) {
      num = 0;
      is_number = UNDEF;
      while((c = getchar ()) != '\n') {
        if(c >= '0' && c <= '9') {
          num = num * 10 + c - '0';
          if(is_number == UNDEF) {
            is_number = VALID;
          }
        } else {
          is_number = INVALID;
        }
      }
      if(is_number == VALID) {
        break;
      }
      printf ("未入力または数字以外です\n");
    }
    if(num % 2 == 1) {
      if(num > Nmax) {
        Nmax = num;
      }
    }
  }
  if(Nmax > 0) {
    printf ("最も大きい奇数: %d\n", Nmax);
  } else {
    printf ("奇数がありませんでした\n");
  }
}

未入力とエラーでメッセージを変えたい場合は、UNDEFかINVALIDかを判断します。

Comments

[kotarou9]

回答ありがとうございます。
最初「123a4」のように入力した際、エラーが出るようにするつもりでしたがループせず抜けてしまったので文字が入力されたらstrに入力されてnum[i]をリセットするつもりで作成していました。
フラグとは何か調べ下記のコードが真の場合だけ通った際フラグが立ってwhileから抜けれるようにしてみたのですがエンターだけ押して未入力とはならずループから抜けてしまいました

while ((c = getchar()) != '\n')
{
if (c >= '0' && c <= '9' )
{
num[i] = num[i]*10 + c -'0';
}
else
{
str

[otn]

コードが途中で切れてますよ。

> 「123a4」のように入力した際、エラーが出るようにするつもりで

回答に書いた「「有効な数字入力があったかどうかフラグ」を偽にリセットする。」は間違ってました。
1文字ずつ入力しながら判断する方法だと、
・有効な数字だけが入力されている
・数字以外が1文字でも入力された
・どちらかまだ未確定
の3値要りますね。

間違ったお詫びにコードを追記しておきます。
jimbeさんの回答のように、1行入力と行の中を見るのを分けると、ロジックがシンプルで、フラグも要りません。その代わりに1行文を溜めておく文字配列が要ります。

[kotarou9]

コードの切れと返信が遅れてしまい失礼しました
回答・コードまでありがとうございます！
３つの値がいることを曖昧に捉えていたのでわかりやすく解説していただいて納得することができました
もう一度自身でコードを作ってみます！

つたない質問の中で意を汲み取って回答いただけているだけで大変助かっています