c: 入力した点数の判定について

Question

#include <stdio.h>
#define NUMBER 100
void errorcheck(int *t, int n);

int main(void)
{
    int num;
    int person[NUMBER];
    printf("人数: ");
    scanf("%d", &num);
    printf("点数を入力してください\n");
    
    for (int i=0; i<num; i++) {
        printf("%d人目: ",i + 1);
        scanf("%d", &person[i]);
    }
    puts("エラーチェック終了");
    errorcheck(&num, *(person));
    
    return 0;
}

void errorcheck(int *t, int n){
    
    for (int i = 0; i<*t; i++) {
    if (n <51){
        printf("%d番: %d\n",i + 1,n);
    }else{
        printf("入力エラー\n");
    }
    }
}

やりたいこと

テキストの問題で関数でのエラーチェックの仕方が分からなくて困っています。
一つだけ判定するなら出来るのですが配列の判定となるとどのようにすればいいか分りません。
サンプルコードを交えて教えて頂けると助かります。

改変

#include <stdio.h>
#define NUMBER 100
void errorcheck(int *t, int n);

int main(void)
{
    int num;
    int person[NUMBER];
    printf("人数: ");
    scanf("%d", &num);
    printf("点数を入力してください\n");
    
    for (int i=0; i<num; i++) {
        printf("%d人目: ",i + 1);
        scanf("%d", &person[i]);
    }
    puts("エラーチェック終了");
    errorcheck((person), num);
    
    return 0;
}

void errorcheck(int *t, int n){
    
    for (int i = 0; i<n; i++) {
    if (*(t) <51){
        printf("%d番: %d\n",i + 1,*(t));
    }else{
        printf("入力エラー\n");
    }
    }

完成？

#include <stdio.h>
#define NUMBER 100
void errorcheck(int *t, int n);

int main(void)
{
    int num;
    int person[NUMBER];
    printf("人数: ");
    scanf("%d", &num);
    printf("点数を入力してください\n");
    
    for (int i=0; i<num; i++) {
        printf("%d人目: ",i + 1);
        scanf("%d", &person[i]);
    }
    puts("エラーチェック終了");
    errorcheck((person), num);
    
    return 0;
}

void errorcheck(int *t, int n){
    
    for (int i = 0; i<n; i++) {
        if(t[i]>=1&&t[i]<=50){
        printf("%d番: %d\n",i + 1,t[i]);
    }else{
        printf("%d番入力エラー\n",i+1);
    }
    }
}

Answer 1

課題でシグネチャが指定されているのですね。

void errorcheck(int *t, int n)

質問者さんは人数を第一引数に、データを第二引数にしていますが、逆です。
第一引数として配列の先頭アドレスを、第二引数として人数を渡すようにすると良いでしょう。

追記

私の現状のコードですと最初の要素数だけで判定しまいその結果がその後の判定に繋がってしまう

上手くいかない理由は、常に*tしか参照していないからです。
これでは先頭要素からポインタが移動しません。

次のどちらかの方法で、参照先を変更してください。
・ ループ毎にポインタ変数tをインクリメントする
・ *(t+i)あるいはt[i]を参照する

個人的には後者の方が好みです。

Comments

[退会済みユーザー]

逆にしました。
私の現状のコードですと最初の要素数だけで判定しまいその結果がその後の判定に繋がってしまうのですがこの場合どのようにすると良いのでしょうか？
恐らく条件分岐が原因だと思うのですが

[退会済みユーザー]

一応完成はしたのですが編集したコードで解答としてあっているのでしょうか？
それとシグネチャとはなんでしょうか？

[LouiS0616]

> 解答としてあっているのでしょうか？

課題の『配列変数の中身を-1に置き換える』および『main関数において... 「入力エラー」と表示』の部分を満たしていないように思います。

---
> シグネチャとはなんでしょうか？

関数名と引数の並びのことです。
とりあえず関数のプロトタイプと関係がある概念であることだけ知っておけば良いでしょう。

文意によって若干意味に揺らぎがある場合もあるようですが。

[退会済みユーザー]

条件分岐街の点数が入力された時に-1に置き換える場合どのようにするべきなのでしょうか？既に解決済みになっていますが教えて頂けると嬉しいです。

[LouiS0616]

関数errorcheck内ではprintしません。
エラーメッセージをprintする代わりに、不正なデータを-1に置き換えれば良いです。

問題文を見る限り出力はmain関数に任せるようですから。