0

私が抱えている問題は、関数displayAllExam()を呼び出すと、メインでexam[]と呼ばれる配列をゼロに設定しても、ゼロに設定されているものもあれば、おそらくRAMのスペースを表す乱数に設定されているものもあります。コードを実行すると配列に値を追加できますが、50個の値を追加しないとそこに乱数が含まれるため、これは私のdisplayExamAverage()関数を台無しにしています。また、配列の最高値と最低値を見つけるためのdisplayHighLow関数に本当にこだわっています

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#define pause system("pause")
#define cls system("cls")
#define flush fflush(stdin)
#define SIZE 50

//Lets Prototype
int getChoice();
void displayMenu();
void addScores(int a[],int *c);
void displayAllExam(int a[]);
void displayExamAverage(int a[], int *c);
void displayHighLow(int a[], int *c);


main(){
    int exams[SIZE] = {0}, counter = 0;
    char choice;
    do {
        choice = getChoice();
        switch(choice){
        case 1:
            addScores(&exams[SIZE],&counter);
            break;
        case 2:
            displayExamAverage(&exams[SIZE],&counter);
            break;
        case 3:
            displayHighLow(&exams[SIZE], &counter);
            break;
        case 4:
            displayAllExam(&exams[SIZE]);
            break;
        }

    }while (choice != 5);
}//end main

 int getChoice(){
    int result = 0;
    do{
        displayMenu();
        scanf_s("%i", &result);
        if(result < 1 || result >5 ){
            printf("Invalid Selection, please enter a number between 1 and 5\n");
        pause;}
    } while(result < 1 || result > 5);
    return result;
}//end getChoice

void displayMenu() {
    cls;
    printf("MAIN MENU\n\n");
    printf("1: add an exam score\n");
    printf("2.Display exam average\n");
    printf("3.Display High and Low score\n");
    printf("4.Display all scores\n");
    printf("5.Quit\n\n");
    printf("Enter your choice: \n\n");
    return;
}//end display menu

void addScores(int a[SIZE],int *c){
    int i=0;
    printf("please enter your score..");
    scanf_s("%i", &a[i]);
}//end add scores

void displayExamAverage(int a[],int *c){
    float average, total = 0.0;
    int i;
    for(i=0; i < *c; i++)
        total = total + a[i];
     average = total/ *c;
     printf("%f",average);
     pause;
}//end displayExamAverage

void displayHighLow(int a[],int *c){
    int high= a[0], low= a[0];
    int i,x,y;
    for(i=0; i < *c; i++){
        if(a[*c]> a[0])
            high = a[i];
    }
    for(i=0; i < *c; i++){
            if(a[*c]<a[0])
            low = a[i];
    }
    printf("The low is %i", low);
    printf("The high is %i", high);
    pause;

}//end DisplayHighLow

void displayAllExam(int a[SIZE]){
    int i;
    for(i=0;i < SIZE; i++)
        printf("the %i score is %i\n",i+1,a[i]);
    pause;
}//end displayAllExam
4

1 に答える 1

2

配列の末尾の直後の位置へのポインターを渡しています。読み取りまたは書き込みを計画している場合、これは大したことではありません。

displayAllExam(&exams[SIZE]);

代わりに、次のように渡します。

displayAllExam(exams);

他のすべての通話についても同様です。

于 2013-10-30T02:41:38.780 に答える