0

2D配列の要素の最小、最大、平均を取得するのに問題があります。

生徒と成績を含む2D配列があります。

randで成績を生成しています。たとえば、2,2と入力すると、印刷されます

Courses : 01   02   Average   Min   Max
    ID   
    01     8   50     29
    02    74   59     29

、私の平均関数は最初の平均を取り、他の平均は取りません。

これが私のコードです;

int A[30][30];

int findAverage(int noOfStudents ,int noOfGrades ){

    float sum,average;




    for (int i = 0 ; i < noOfGrades ; i++) {
        for (int j = 0; j<noOfStudents; j++) {

        sum += A[i][j];

        }
        average = sum / noOfGrades;
       // cout << "   " << format(average);
        sum = 0;
        return format(average);

    }

そしてここで私はそれをどのように使うか

int main() {

    int noOfCourses , noOfStudents;
    cin >> noOfCourses >> noOfStudents;
    cout << "Courses : " ;

    for (int i = 0; i < noOfCourses; i++) {
        if (i+1 >= 10) {

            cout << i+1 << "   ";
        }else{
            cout <<"0" << i+1 << "   ";

        }
    }

    cout << "Average   Min   Max";

    for(int i=0; i<noOfStudents; i++) {         //This loops on the rows.        

        for(int j=0; j<noOfCourses; j++) {     //This loops on the columns
            A[i][j] = genGrade(); 
        }
    }

    cout << "\n    ID   " << endl;

    for(int i=0; i<noOfStudents; i++) {  //This loops on the rows.

        if (i+1 >= 10) {

            cout <<"    " << i+1 << "   ";
        }else{
            cout <<"    0" << i+1 << "   ";

        }

        //cout <<"    0" << i+1 << "   ";

        for(int j=0; j<noOfCourses; j++) {  //This loops on the columns


            if (A[i][j] >= 10 && A[i][j] <=99) {

                cout <<" " << A[i][j]  << "  ";
            }
            if(A[i][j] < 10) {

                cout <<"  " << A[i][j] << "  ";
            }
            if (A[i][j] == 100) {

                cout << A[i][j] << "  ";
            }
        }
        cout <<"   "<<findAverage(noOfStudents,noOfCourses);
        cout << endl;
    }
}

私は何を間違っているのですか？また、アレイごとの最小値、最大値を取得するにはどうすればよいですか？

4

3 に答える 3

1

手始めに、ループの内側から戻ってきます。

for (int i = 0 ; i < noOfGrades ; i++) {
  for (int j = 0; j<noOfStudents; j++) {
    ...
  }
  ...
  return ...;
}

外側のループが1回だけ実行されることがわかりますか？

于 2012-05-04T16:27:02.067 に答える
1

このタスクにはコンテナを使用することを強くお勧めします。たとえば、次のようにすることができます

typedef std::vector<float> grades;
std::vector<grades> student_grades;

//populate 

for(const grades& gr : student_grades) {
    float min, max, avg;
    std::tie(min, max)=std::minmax(gr.begin(), gr.end());
    avg=std::accumulate(gr.begin(), gr.end(), 0.0) / gr.size());
    std::cout << "Min" << min << " Max: " << max << " Avg: " << avg << std::endl;
}

http://en.cppreference.com/w/cpp/algorithm/minmax

http://en.cppreference.com/w/cpp/algorithm/accumulate

http://en.cppreference.com/w/cpp/utility/tuple/tie

于 2012-05-04T16:29:27.193 に答える
0

問題は、findAverage関数内で、行をループしているループ内にreturnステートメントがあることです。非常に簡単な修正が必要な場合はfindAverage、平均を計算する行を示す別のパラメーターを関数に追加します。

int findAverage(int course ,int noOfGrades ){

    float sum,average;


    for (int j = 0; j<noOfStudents; j++) {

        sum += A[course][j];

    }
    average = sum / noOfGrades;
    return format(average);
 }

このように呼んでください

cout <<"   "<<findAverage(i,noOfCourses);
于 2012-05-04T16:30:40.733 に答える