Obtenir la Moyenne,Min et Max de nombres dans le tableau 2D

J'ai un problème avec l'obtention de la min,max, moyenne de l'un des éléments de tableau 2D.

J'ai un tableau 2D qui contient des Étudiants et des Grades.

Je suis de la génération de la sortes avec rand. Par exemple Quand je rentre de 2,2 elle imprime m' 

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

Mon moyen de la fonction prend la première de celles de la moyenne et de ne pas prendre les autres en moyenne.

Voici mon code ;

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);

    }

et voici comment je l'utilise

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;
}
}

Ce que je fais mal ? Aussi, Comment pourrais-je obtenir le min,le max de ce tableau?

  • Qu'est-ce que la déclaration de A?
  • int A[30][30]; , désolé 🙂
  • Vous n'avez pas initialisé sum avant de l'utiliser, et vous retournez au cours de la première itération de la boucle externe. Est-il une raison pour laquelle vous n'utilisez pas std::vector pour stocker les données et std::accumulate pour additionner les valeurs? Vous pouvez obtenir des min et max avec min_element et max_element.
InformationsquelleAutor Yeliz Il | 2012-05-04
  1. 0

    Le problème est que dans votre findAverage de la fonction vous avez une instruction return dans la boucle qui se boucle sur les lignes. Si vous voulez une solution simple d'ajouter un autre paramètre à la findAverage fonction qui indique le rang de calculer la moyenne.

    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);
}

    et de l'appeler comme ça

    cout <<"   "<<findAverage(i,noOfCourses);
    • Ah, Merci beaucoup. Cela a fonctionné assez bien.
    • Vous avez encore pour initialiser somme à 0.0.
    InformationsquelleAutor seanwatson
  2. 1

    Pour commencer, vous êtes de retour à partir de l'intérieur de votre boucle:

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

    Pouvez-vous voir comment la boucle externe ne jamais exécuter une fois?

    • Mais quand les deux boucles s'exécute , puis Il imprime-moi la dernière place de la première
    • C'est parce que vous réinitialisez sum dans chaque boucle.
    • Entendu, Merci beaucoup! 🙂
    InformationsquelleAutor Robᵩ
  3. 1

    Je voudrais fort vous recommandons d'utiliser des conteneurs pour cette tâche. Par exemple, vous pouvez effectuer les opérations suivantes

    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

    • Hmm, je n'ai pas beaucoup d'idée sur les conteneurs. Je sais que sur les vecteurs mais qu'est-cravate et s'accumulent faire? et ai-je besoin d'une bibliothèque pour eux?
    • std::cravate(min, max)=std::minmax(gr.begin(), gr.end()); Ce qui fait que l'opération de réellement faire?
    • Non, vous n'avez pas besoin d'une bibliothèque pour eux, ils sont dans la bibliothèque standard, d'accumuler ajoute de la valeur dans le conteneur, et std::minmax retourne un tuple (un groupement de valeurs) des valeurs minimale et maximale. std::cravate déballe que tuple dans des variables existantes. C'est très propre, moderne et efficace, le retour de plusieurs valeurs à partir d'une fonction. Je vais ajouter quelques références à ma réponse.
    • Ah, j'ai essayé le code mais il ne marche pas reconnaître la cravate :/ un
    • #include <n-uplet>, et vous assurer que vous êtes compilateur avec -std=c++0x c'est à partir de la nouvelle norme.
    • si vous n'avez pas bonne c++11 de soutien que vous pouvez déterminée ne leur indiviuallly comme suit: float min=std::min(gr.begin(), gr.end()); float max=std::max(gr.begin(), gr.end()); float avg=std::accumulate(gr.begin(), gr.end(), 0.0f); espère que ça aide
    • J'utilise Xcode 4 , Il ne marche pas reconnaître tuple malheureusement. Mal-être essayer la deuxième option, merci!
    • cool, n'oubliez pas que les conteneurs sont à votre ami, vous devez toujours utiliser contient des tableaux où c'est possible.
    • Quelle est l'erreur que j'obtiens maintenant est " Appelé objet de type 'float' est pas une fonction ou un pointeur de fonction. avg fonctionne très bien mais min et max qui les fais
    • assurez-vous d'avoir #include <algorithm> oh désolé, je pense que cela devrait me std::max_element et std::min_element mes excuses.
    • Je l'utilise déjà, mais il ne marche pas encore permettez-moi de compiler malheureusement
    • qu'est-ce que l'erreur et la ligne de code exacte

    InformationsquelleAutor 111111