プロトタイプ宣言を使った計算の簡略化

プロトタイプ宣言を使ってコードを書いたのですが長くなってしまいました。
もっと簡単にできる方法を教えてください。
実行例↓

#include<stdio.h>
#define SIZE 20
void get_score( int score[ ], int n);
void error_message(int point);
void show_array( int array[ ], int n);
int max_array(int array[ ], int n);
int min_array(int array[ ], int n);
double average_array(int array[ ], int n);
int array[SIZE];
int score[SIZE];

int main(){
  int n,max,min;
  double ave;
  printf("データ数を入力してください（1～20）:");
  scanf("%d",&n);
  get_score(score,n);
  printf("\n");
  printf("データを表示します\n");
  show_array(array,n);
  printf("\n");
  max=max_array(array,n);
  min=min_array(array,n);
  ave=average_array(array,n);

  printf("最大値は%dです\n",max);
  printf("最小値は%dです\n",min);
  printf("平均値は%.1lfです\n",ave);
  return 0;
}

void get_score( int score[ ], int n){
  int i;
  for(i=0;i<n;i++){
    printf("No.%d:",i+1);
    scanf("%d",&score[i]);
    array[i]=score[i];
    while( score[i]<0 || score[i]>100){
      error_message(score[i]);
      printf("No.%d:",i+1);
      scanf("%d",&score[i]);
      array[i]=score[i];
    }
  }
}


  void error_message(int point){
    if( point<0){
      printf("****入力ミス:%dは0未満です****\n",point);
    }
    if( point>100){
      printf("****入力ミス:%dは100をこえています****\n",point);
    }
  }

  void show_array( int array[ ], int n){
    int i;
    for(i=0;i<n;i++){
      printf("No.%d:%d\n", i+1,array[i]);
    }
  }

  int max_array(int array[ ], int n){
    int i,max;
    max=array[0];
    for(i=1;i<n;i++){
      if(array[i]>max){
	max=array[i];
      }
    }
    return max;
  }

  int min_array(int array[ ], int n){
    int i,min;
    min=array[0];
    for(i=1;i<n;i++){
      if(array[i]<min){
        min=array[i];
      }
    }
    return min;
  }
  
  double average_array(int array[ ], int n){
    int i,sum=0;
    double ave;
    for(i=0;i<n;i++){
      sum += array[i];
    }
    ave=(double)sum/n;
    return ave;
  }

jimbe

2021/11/12 16:20

このコードは実行例のように動作しているのでしょうか。

bafubafu_

2021/11/12 16:23

はい。入力ミスをするとエラーメッセージがでます。

jimbe

2021/11/12 16:46 編集

失礼しました、score と array と二つも配列があったので、ご提示のようにはならないかと思ってしまいました。なぜ score に入力してarray に入れ直しているのでしょうか。

bafubafu_

2021/11/12 16:59

大学の授業で習って、友達と協力して書いた結果このようなかたちになりました。正直自分でもあまりよくわかってないです。申し訳ないです。

jimbe

2021/11/12 17:13

なるほど。この場合は配列の中身や並び順は入力後は最後まで変わりませんので、つまり最後まで score と array は同じです。ですので、どちらかに統一したほうが良いでしょう。さらに言えば、配列は各関数にパラメータとして渡していますので、グローバルである必要はありません。 int score[SIZE] (もしくは array, 統一した方 ) は main 関数内に入れた方が良いと思います。

bafubafu_

2021/11/12 17:29

丁寧にご教示いただき有難うございます。

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

回答2件

ベストアンサー

プロトタイプ宣言を無くしたい/減らしたいということでしたら、"呼ばれる関数" を "呼ぶ関数" より先に書いておくことで出来ます。

プロトタイプとは関係ありませんが、 get_score と error_message の箇所は次のようにしますと、入力の scanf 等や判定式が 1 ヶ所になります。
( error_message を is_error と名前を変えています。)

c
1  int is_error(int point){
2    if( point<0){
3      printf("****入力ミス:%dは0未満です****\n",point);
4      return 1;
5    }
6    if( point>100){
7      printf("****入力ミス:%dは100をこえています****\n",point);
8      return 1;
9    }
10    return 0;
11  }
12
13void get_score( int score[ ], int n){
14  int i;
15  for(i=0;i<n;i++){
16    printf("No.%d:",i+1);
17    scanf("%d",&score[i]);
18    array[i]=score[i]; //必要?
19    if(is_error(score[i])){
20      i--;
21    }
22  }
23}