【C言語】冗長だと思う数字入力プログラムを改善したい

Question

以下のプログラムは3つの数字をスペース区切りで入力して、入力した数字を改行区切りで出力するというコードです。
C言語はあまり慣れていないので、以下のコードに冗長さを感じますが何か改善点はありますか？

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

int main(void) {
    char inputNum[1000];
    int stock[3];
    char token[] = " ";
    char *tok;
    int count = 0;
    
    fgets(inputNum, sizeof(inputNum), stdin);
    tok = strtok(inputNum, token);
    while (tok != NULL) {
        stock[count] = atoi(tok);
        tok = strtok(NULL, token);
        count++;
    }
    
    for (int i = 0; i < sizeof(stock)/sizeof(stock[0]); i++) {
        printf("%d\n", stock[i]);
    }
}

Answer 1

こんにちは。

3つの数字をスペース区切りで入力して、入力した数字を改行区切りで出力するというコードです。

要件がこれだけなら、私だったらお手軽に以下の感じです。

#include <stdio.h>

int main(void)
{
    int num0, num1, num2;
    if (scanf("%d %d %d", &num0, &num1, &num2) != 3)
    {
        fprintf(stderr, "input error\n");
        return 1;
    }
    printf("%d\n%d\n%d\n", num0, num1, num2);
}

処理する数値の数が増える可能性の有無や、改行が可笑しかった時どうするとか、周辺情報が見えないので、バランス点が見えません。
配列を使っても完全に無駄になる可能性を考慮すると、簡単確実なプログラムにしたいと思います。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
そうですよね。やっぱりscanf使うんですね。
fgetsだと他にもっと良いコードは書けないのでしょうか？

[Chironian]

何を持ってして「良い」と言うのかが問題です。
そのプログラムがどのように拡張されるのか、ある程度想定して、それに対応できるように設計されたプログラムは「良い」ものです。

どんな作業をアシストするプログラムなのか見えないと、どのような拡張がありうるのか見当もつかないので、何か拡張できる仕組み（例えば配列とか）を仕込んでもまるっと無駄になるかも知れません。それは確実に「悪い」ものです。

Answer 2

むしろ短い。エラー処理まできちんとやりたいなら、相応の長さになる。

配列の境界チェック
atoi関数 => strtol関数
非常に長い入力への対処
…

どこまで簡潔に書けるか、というのは、どこまで仕様を詰めるか、で簡単に変わります。

ちなみに私なら(ズルして)、

標準入力を1文字ずつ読み込み、数字ならそのまま、空白なら改行、それ以外なら即リターンってコードします。

Comments

[退会済みユーザー]

ご回答ありがとうございます。

Answer 3

#include <stdio.h>

int main(void){
  int i;
  char b[1000];

  fgets(b, sizeof(b), stdin);

  for(i = 0; i < 1000; i++){
    if(b[i] == '\0') break;
    printf("%c", b[i] == ' ' ? '\n' : b[i]);
  }

  return 0;
}

C初心者ですがやってみました。
数字かどうかの判定は行っていません。

#include <stdio.h>
#include <ctype.h>

int main(void){
  int i = 0;
  char b[1000];

  fgets(b, sizeof(b), stdin);

  while(b[i] == ' ' || isdigit(b[i])){
    printf("%c", b[i] == ' ' ? '\n' : b[i]);
    i++;
  }

  return 0;
}

数字の判定を入れたバージョン。

Comments

[退会済みユーザー]

ご回答ありがとうございます。

Answer 4

3つの数字をスペース区切りで入力して、入力した数字を改行区切りで出力

で作りました。改善点は？

#include <stdio.h>
#include <string.h>
#define NUM 3

int main(void) {
    char inputNum[1000];
    int stock[NUM]={'\0'};
    int i,j;

    fgets(inputNum, sizeof(inputNum), stdin);
    for (i=j=0;inputNum[i]!='\n';i++){
        if (inputNum[i]==' '){
           j++;
        }else{
           stock[j]=stock[j]*10+(inputNum[i]-'0');
        }
    }
    for (i=0; i<NUM; i++) {
        printf("%d\n", stock[i]);
    }
}

表示するだけなら

#include <stdio.h>
#include <string.h>

int main(void) {
    char inputNum[1000];
    char *p;
    p=inputNum;

    fgets(inputNum, sizeof(inputNum), stdin);
    while(p=strchr(p,' ')){
       *p='\n';
    }
    printf("%s",inputNum);
}

Answer 5

whileループをちょいと書き換えてみました。

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

int main(void) {
    char inputNum[1000];
    int stock[3];
    char token[] = " ";
    char *tok;
    int count = 0;

    fgets(inputNum, sizeof(inputNum), stdin);
    
    tok = strtok(inputNum, token);
    do { 
        stock[count] = atoi(tok);
        printf("%d\n", stock[count++]);
    } while(tok = strtok(NULL, token), tok != NULL);

    return 0;
}

---

こういうのもありかもしれませんね。

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

int main(void) {
    char inputNum[1000];
    int stock[3];
    char token[] = " ";
    char *tok;
    int count = 0;

    fgets(inputNum, sizeof(inputNum), stdin);
    
    for(tok = strtok(inputNum, token); tok != NULL; tok = strtok(NULL, token), count++) {
        printf("%d\n", stock[count] = atoi(tok));
    }

    return 0;
}

Comments

[退会済みユーザー]

ご回答ありがとうございます。

[退会済みユーザー]

ロジック自体に大きな変更はない感じですかね。

[LouiS0616]

空白を改行に置き換えるのもありですが、そうするとstockが更新できないですからね...
私はここまでです、他の回答の黒魔術っぷりに期待したいですね。

[退会済みユーザー]

ありがとうございます。
助かります！