整数の不明な数を読み取り、それらを単語で印刷する

Question

stdinからの入力を処理しようとしていますが、壁にぶち当たっています。 私の目標は数字の数字（0〜99）を読んで、それぞれを単語で印刷することです。 私の最初の試みでした：

int main(void) { 

char *a[20] = {"zero","one","two","three","four","five","six","seven","eight", 
"nine","ten","eleven","twelve","thirteen","fourteen","fifteen","sixteen", 
"seventeen","eighteen","nineteen"}; 

char *b[8] = {"twenty","thirty","fourty","fifty","sixty","seventy","eighty","ninety"}; 

int num=0, tens=0, ones=0; 

while (scanf("%d", &num)==1){ 
    tens = num/10; 
    ones = num%10; 

    if (tens>1){ 

     printf("%s ", b[tens-2]); 
     printf("%s \n", a[ones]); 
    } 
    else 
     printf("%s \n", a[num]); 
} 
printf("done"); 
return 0; 
} 

出力は正しいですが、scanf関数は、ループを終了することはありません。

第二の試み：ここ

int main(void) { 

char *a[20] = {"zero","one","two","three","four","five","six","seven","eight", 
"nine","ten","eleven","twelve","thirteen","fourteen","fifteen","sixteen", 
"seventeen","eighteen","nineteen"}; 

char *b[8] = {"twenty","thirty","fourty","fifty","sixty","seventy","eighty","ninety"}; 
char line[1024], *ptr = NULL; 
long num; 
int tens=0, ones=0; 

if(fgets(line, sizeof(line), stdin)!=NULL){ 


    do { 
     num = strtol(line, &ptr, 10); 
     tens = num/10; 
     ones = num%10; 

     if (tens>1){ 

      printf("%s ", b[tens-2]); 
      printf("%s \n", a[ones]); 
     } 
     else 
      printf("%s \n", a[num]); 
    }while (*ptr!= '\n'); 
} 

printf("done"); 
return 0; 
} 

私はコンパイルエラーを取得し、それが動作するかどうか私にはわからないので、問題を見つけることができません。

[UPDATE]：2番目のコードは実行されますが、 12 35 51のように1つ以上の数字を入力すると無限に最初の数字（12文字）が印刷されます。

ご協力いただければ幸いです。

Answer 1

あなたはとても近いです。 strtolの戻り値を確認し、strtolの呼び出しに続いてendptrに基づいてポインタアドレスを更新するだけです。また、ディスカッションで述べたように、コンバージョンの範囲外の値もチェックする必要があります。これは、すべてのことが必要とされている。

#include <stdio.h> 
#include <stdlib.h> 
#include <errno.h> 

int main(void) { 

    char *a[] = {"zero","one","two","three","four","five","six", 
       "seven","eight","nine","ten","eleven","twelve", 
       "thirteen","fourteen","fifteen","sixteen", 
       "seventeen","eighteen","nineteen"}; 

    char *b[] = {"twenty","thirty","fourty","fifty","sixty", 
       "seventy","eighty","ninety"}; 
    char line[1024] = ""; 
    long num; 
    int tens=0, ones=0; 

    if (fgets (line, sizeof(line), stdin) != NULL) { 

     char *p = line, *ep = NULL; 
     errno = 0; 

     while (errno == 0) { 
      num = strtol (p, &ep, 10);  /* convert to long */ 
      if (p == ep) break;    /* no digits, break */ 
      p = ep;       /* update p to ep */ 
      if (num < 0 || 99 < num) {  /* validate range */ 
       fprintf (stderr, "error: %ld - out of range.\n", num); 
       continue; 
      } 
      tens = num/10; 
      ones = num%10; 
      if (tens > 1) { 
       printf ("%s ", b[tens-2]); 
       printf ("%s \n", a[ones]); 
      } 
      else 
       printf("%s \n", a[num]); 
     } 
    } 

    printf ("done\n"); 

    return 0; 
} 

使用例/出力

$ ./bin/n2string 
12 55 61 -1 33 102 4 
twelve 
fifty five 
sixty one 
error: -1 - out of range. 
thirty three 
error: 102 - out of range. 
four 
done 

以上、それを見て、ご質問があれば私に知らせてください。

Answer 2

コードの目的は、入力された数字のテキスト版を印刷することです。あなたのコードはほとんど変わりません。ライブラリを追加して実行しました。

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

int main(void) { 

    char *a[20] = {"zero","one","two","three","four","five","six","seven","eight", 
        "nine","ten","eleven","twelve","thirteen","fourteen","fifteen","sixteen", 
        "seventeen","eighteen","nineteen"}; 

    char *b[8] = {"twenty","thirty","fourty","fifty","sixty","seventy","eighty","ninety"}; 
    char line[1024], *ptr = NULL; 
    long num; 
    int tens=0, ones=0; 

    if(fgets(line, sizeof(line), stdin)!=NULL){ 


     do { 
      num = strtol(line, &ptr, 10); 
      tens = num/10; 
      ones = num%10; 

      if (tens>1){ 

       printf("%s ", b[tens-2]); 
       printf("%s \n", a[ones]); 
      } 
      else 
       printf("%s \n", a[num]); 
     }while (*ptr!= '\n'); 
    } 

    printf("done"); 
    return 0; 
} 

テスト

19 
nineteen 
done 

たぶん、問題は、コードの目的を知らなかったということでした。