Petit logiciel qui ne fonctionne pas toujours

Petit logiciel qui ne fonctionne pas toujours - C++ - Programmation

Marsh Posté le 25-05-2006 à 11:06:59    

Bonjour,
 
Je debute en C++ et pour m'entrainer j'ai essayer de faire un petit logiciel de statistique.... Mais la j'ai un probleme et je cale ...
En fait mon pogiciel ne fonctionne que des fois lorsque je lui demande la médiane et cela dépend des valeurs que je rentre au debut (mais lequels précisement je sais pas), les autre fois il se ferme directement sans rien affiché comme s'il efféctuer une fonction "quitter" mais de temp en temp il plante aussi, je suis obliger d'aller dans le gestionnaire des taches pour le fermer...
De plus Il y a quelque chose que je ne comprend pas dans le code (je l'ai mi en commentaire rouge et souligné) et je pense que ca peut venir de là mais...
 
Voici le code (il est peut être pas super mais bon je debute):
 

Code :
  1. #include <cstdlib>
  3. #include <iostream>
  5. using namespace std;
  7.     float somme_val();
  9.     int somme_effect();
  11.     float somme_totale();
  13.     float liste();
  15.     int nbvaleurs, choix, med;
  17.     float moy;
  21.     float * pval = new float[nbvaleurs];
  23.     int * peffect = new int[nbvaleurs];
  25.     float * liste_val = new float[somme_effect()];
  31. int main(int argc, char *argv[])
  33. {
  35.     cout<<"rentrez le nombre de valeurs differentes."<<endl;
  37.     cin>>nbvaleurs;
  41.     for(int i=0; i<nbvaleurs; i++){
  43.             cout<<"rentrez une valeur";
  45.             cin>>pval[i];
  47.             cout<<"rentrez son effectif";
  49.             cin>>peffect[i];
  51.             }
  55.     for(int i=0; i<nbvaleurs; i++){
  57.             cout<<pval[i]<<"\t";
  59.             }
  61.     cout<<endl;
  63.     for(int i=0; i<nbvaleurs; i++){
  65.             cout<<peffect[i]<<"\t";
  67.             }
  71.     cout<<endl<<"pour calculer la moyenne tapez1, la mediane tapez2."<<endl;
  73.     cin>>choix;
  75.     switch(choix){
  77.                   case 1: moy = somme_totale()/somme_effect();
  79.                           cout<<"la moyenne est de:"<<moy<<endl;
  81.                           break;
  83.                   case 2: liste();//pour somme_effect impaire
  85.                           int j = somme_effect();
  87.                           med = ((j+1)/2)-1;//le -1 sert a cerner la bonne case du tableau.
  89.                           cout<<"la mediane est:\t"<<liste_val[med];
  95.                   }
  97.     delete pval, peffect, liste_val;
  99.     system("PAUSE" );
  101.     return EXIT_SUCCESS;
  103. }
  107.     float somme_val(){
  109.                 float x;
  111.                 for(int i=0; i<nbvaleurs; i++){
  113.                          x += pval[i];
  115.                          }
  117.                  return x;
  119.                  }
  123.     int somme_effect (){
  125.                  float x; //ici  si je met int pour correspondre au type de la fonction, la moyenne est fausse.
  127.                  for(int i=0; i<nbvaleurs; i++){
  129.                          x += peffect[i];
  131.                          }
  133.                  return x;
  135.                  }
  139.     float somme_totale(){
  141.                    float x;
  143.                    for(int i=0; i<nbvaleurs; i++){
  145.                           x += pval[i] * peffect[i];
  147.                           }
  149.                    return x;
  151.                    }
  155.     float liste(){
  159.           int m= 0;
  161.           for(int i=0; i<nbvaleurs; i++){
  163.                   for(int n=0; n<peffect[i]; n++){
  165.                           liste_val[m]= pval[i];
  167.                           m++;
  169.                           }
  171.                   }
  173.           }


 
 
 
 
 
PS: Je sais pas si le forum est fait pour vérifier des codes entiers mais bon, je ne sais pas a qui m'adresser..
 
Aidez-moi s'il vous plait.


Message édité par elephant13 le 25-05-2006 à 11:18:28
Reply

Marsh Posté le 25-05-2006 à 11:06:59   

Reply

Marsh Posté le 25-05-2006 à 11:33:40    

utilise std::vector<float>
 
et utilise ton compilateur correctement : t'as des dizaines de variables pas initialisées ...

Reply

Marsh Posté le 25-05-2006 à 11:40:19    

Dsl mais deja je ne connais pas std::vector<float> et de plus je ne voi pas de variable non innitialisées....

Reply

Marsh Posté le 25-05-2006 à 11:48:25    

tu lis pas les erreurs que te donne ton compilateur ????

Reply

Marsh Posté le 25-05-2006 à 11:49:33    

Il ne m'en donne pas.. sauf le commentaire mis en rouge et souligné...
Aytrement je me suis renseigné sur vector et vais essayer.

Reply

Marsh Posté le 25-05-2006 à 12:11:05    

Voila ce que j'ai fais.. pas super...

Code :
  1. #include <cstdlib>
  2. #include <iostream>
  3. #include <vector>
  5. using namespace std;
  7.     float somme_val();
  8.     int somme_effect();
  9.     float somme_totale();
  10.     float liste();
  12.     int nbvaleurs, choix, med;
  13.     float moy;
  14.     std::vector <float> pval (nbvaleurs);
  15.     std::vector <int> peffect (nbvaleurs);
  16.     std::vector <float> liste_val;
  17.    
  18.    
  20.    
  21. int main(int argc, char *argv[])
  22. {
  23.     cout<<"rentrez le nombre de valeurs differentes."<<endl;
  24.     cin>>nbvaleurs;
  25.    
  26.     for(int i=0; i<nbvaleurs; i++){
  27.             cout<<"rentrez une valeur";
  28.             cin>>pval[i];
  29.             cout<<"rentrez son effectif";
  30.             cin>>peffect[i];
  31.             }
  32.            
  33.     for(int i=0; i<nbvaleurs; i++){
  34.             cout<<pval[i]<<"\t";
  35.             }
  36.     cout<<endl;
  37.     for(int i=0; i<nbvaleurs; i++){
  38.             cout<<peffect[i]<<"\t";
  39.             }
  40.            
  41.     cout<<endl<<"pour calculer la moyenne tapez1, la mediane tapez2."<<endl;
  42.     cin>>choix;
  43.     switch(choix){
  44.                   case 1: moy = somme_totale()/somme_effect();
  45.                           cout<<"la moyenne est de:"<<moy<<endl;
  46.                           break;
  47.                   case 2: liste();
  48.                           int j = somme_effect();
  49.                           med = ((j+1)/2)-1;//le -1 sert a cerner la bonne ncase du tableua.
  50.                           cout<<"la mediane est:\t"<<liste_val[med];
  51.                                
  52.                        
  53.                   }
  54.     pval.clear();
  55.     peffect.clear();
  56.     liste_val.clear();
  57.                  
  58.     system("PAUSE" );
  59.     return EXIT_SUCCESS;
  60. }
  61.    
  62.     float somme_val(){
  63.                 float x;
  64.                 for(int i=0; i<nbvaleurs; i++){
  65.                          x += pval[i];
  66.                          }
  67.                  return x;
  68.                  }
  69.                
  70.     int somme_effect (){
  71.                  float x;
  72.                  for(int i=0; i<nbvaleurs; i++){
  73.                          x += peffect[i];
  74.                          }
  75.                  return x;
  76.                  }
  77.                
  78.     float somme_totale(){
  79.                    float x;
  80.                    for(int i=0; i<nbvaleurs; i++){
  81.                           x += pval[i] * peffect[i];
  82.                           }
  83.                    return x;
  84.                    }
  85.                  
  86.     float liste(){
  87.           for(int i=0; i<nbvaleurs; i++){
  88.                   for(int n=0; n<peffect[i]; n++){
  89.                           liste_val.push_back(pval[i]);
  90.                           }
  91.                   }
  92.           for(int i=0 ; i <liste_val.size() ; i++){
  93.           cout <<liste_val[i] << endl;}
  95.           }


 
J'essaye, j'essaye mais je n'arrive jamai a quelque chose qui fonctionne....

Reply

Marsh Posté le 25-05-2006 à 12:20:33    

#
   float somme_val(){
#
               float x;
 
 
mais bien sur ...

Reply

Marsh Posté le 25-05-2006 à 12:29:56    

Je sais pas si c'est un coup de chance ou si tu l'as fait exprès, mais déjà au départ tu initialises tes 2 premiers tableaux avec une taille égale à 0 ( merci les variables globales... :ange:), or, pour ton 3ème tableau tu fais appel à somme_effect().
 
code de somme_Effect

Code :
  1. int somme_effect (){
  2.                  float x;
  3.                   for(int i=0; i<nbvaleurs; i++)
  4.                   {
  5.                     x += peffect[i];
  6.                    }
  7.                  return x;
  8.                  }


 
x n'est pas initialisé et a donc un contenu indéterminé.
nbvaleurs est initialisé à 0 donc tu ne passes pas dans ta boucle for et tu renvois directement x avec son contenu indéterminé, alors après le tableau bonne chance pour qu'il ait la taille que tu veux...

Reply

Marsh Posté le 25-05-2006 à 12:47:03    

Reply

Marsh Posté le 25-05-2006 à 18:30:56    

Ouai effectivement vu de ce point.. Je n'y avait pas pensé

Code :
  1. #include <cstdlib>
  2. #include <iostream>
  3. #include <vector>
  5. using namespace std;
  7.     float somme_val();
  8.     float somme_effect();
  9.     float somme_totale();
  10.     float liste();
  12.     int nbvaleurs, choix, med;
  13.     float moy, med1;
  14.     std::vector <float> pval;
  15.     std::vector <int> peffect;
  16.     std::vector <float> liste_val;
  17.    
  18.    
  20.    
  21. int main(int argc, char *argv[])
  22. {
  23.     cout<<"rentrez le nombre de valeurs differentes."<<endl;
  24.     cin>>nbvaleurs;
  25.    
  26.     for(int i=0; i<nbvaleurs; i++){
  27.             int j;
  28.             cout<<"rentrez une valeur";
  29.             cin>>j;
  30.             pval.push_back(j);
  31.             cout<<"rentrez son effectif";
  32.             cin>>j;
  33.             peffect.push_back(j);
  34.             }
  35.            
  36.     for(int i=0; i<nbvaleurs; i++){
  37.             cout<<pval[i]<<"\t";
  38.             }
  39.     cout<<endl;
  40.     for(int i=0; i<nbvaleurs; i++){
  41.             cout<<peffect[i]<<"\t";
  42.             }
  43.            
  44.     cout<<endl<<"pour calculer la moyenne tapez1, la mediane tapez2."<<endl;
  45.     cin>>choix;
  46.     switch(choix){
  47.                   case 1: moy = somme_totale()/somme_effect();
  48.                           cout<<"la moyenne est de:"<<moy<<endl;
  49.                           break;
  50.                   case 2: liste();
  51.                           int j = static_cast<int>(somme_effect());
  52.                           med = ((j+1)/2)-1;//le -1 sert a cerner la bonne ncase du tableua.
  53.                           cout<<"la mediane est:\t"<<liste_val[med];
  54.                                
  55.                        
  56.                   }
  57.     pval.clear();
  58.     peffect.clear();
  59.     liste_val.clear();
  60.                  
  61.     system("PAUSE" );
  62.     return EXIT_SUCCESS;
  63. }
  64.    
  65.     float somme_val(){
  66.                 float x;
  67.                 for(int i=0; i<nbvaleurs; i++){
  68.                          x += pval[i];
  69.                          }
  70.                  return x;
  71.                  }
  72.                
  73.     float somme_effect (){
  74.                  float x;
  75.                  for(int i=0; i<nbvaleurs; i++){
  76.                          x += peffect[i];
  77.                          }
  78.                  return x;
  79.                  }
  80.                
  81.     float somme_totale(){
  82.                    float x;
  83.                    for(int i=0; i<nbvaleurs; i++){
  84.                           x += pval[i] * peffect[i];
  85.                           }
  86.                    return x;
  87.                    }
  88.                  
  89.     float liste(){
  90.           for(int i=0; i<nbvaleurs; i++){
  91.                   for(int n=0; n<peffect[i]; n++){
  92.                           liste_val.push_back(pval[i]);
  93.                           }
  94.                   }
  95.           for(int i=0 ; i <liste_val.size() ; i++){//affiche la liste du tableau
  96.           cout <<liste_val[i]<< endl;}
  98.           }


 
Voila le code avec quelque modif, il a l'air de marcher, qu'en pensez vous?


Message édité par elephant13 le 25-05-2006 à 18:31:39
Reply

Marsh Posté le 25-05-2006 à 18:30:56   

Reply

Marsh Posté le 25-05-2006 à 19:20:40    

non ca marche pas
on t'a deja dit que tes variables 'x' ne sont pas initialisées, pourquoi tu ne le fais pas ?
 
tu peux aussi largement simplifier en utilisant std::accumulate par exemple (et supprimer ces variables globales inutiles)
 

Reply

Marsh Posté le 25-05-2006 à 19:42:10    

Oui je peux simplifier peut-être maiis je prefere faire un code que je comprend plutot ke de recpopier betement des bouts de code un peu partout....
 
Autrement je vien seulement de comprendre ce que vous entendiez par initialisé, pour moi c'est par ex (int x) alors que pour vous c'est appliqué une valeur a x, non?
 
PS: ca fait a peu pres un mois que je me suis mis au c++ alors si vous pouviez parler un peu plus "francais" ca arrangerais les choses...
 

Code :
  1. #include <cstdlib>
  2. #include <iostream>
  3. #include <vector>
  5. using namespace std;
  7.     float somme_val();
  8.     float somme_effect();
  9.     float somme_totale();
  10.     void liste();
  12.     int nbvaleurs, choix;
  13.    
  14.     std::vector <float> pval;
  15.     std::vector <int> peffect;
  16.     std::vector <float> liste_val;
  17.    
  18.    
  20.    
  21. int main(int argc, char *argv[])
  22. {
  23.     cout<<"rentrez le nombre de valeurs differentes."<<endl;
  24.     cin>>nbvaleurs;
  25.    
  26.     for(int i=0; i<nbvaleurs; i++){
  27.             int j;
  28.             cout<<"rentrez une valeur";
  29.             cin>>j;
  30.             pval.push_back(j);
  31.             cout<<"rentrez son effectif";
  32.             cin>>j;
  33.             peffect.push_back(j);
  34.             }
  35.            
  36.     for(int i=0; i<nbvaleurs; i++){
  37.             cout<<pval[i]<<"\t";
  38.             }
  39.     cout<<endl;
  40.     for(int i=0; i<nbvaleurs; i++){
  41.             cout<<peffect[i]<<"\t";
  42.             }
  43.            
  44.     cout<<endl<<"pour calculer la moyenne tapez1, la mediane tapez2."<<endl;
  45.     cin>>choix;
  46.     switch(choix){
  47.                   case 1:float moy;
  48.                            moy = somme_totale()/somme_effect();
  49.                           cout<<"la moyenne est de:"<<moy<<endl;
  50.                           break;
  51.                   case 2: liste();
  52.                           int med;
  53.                           int j = static_cast<int>(somme_effect());
  54.                           med = ((j+1)/2)-1;//le -1 sert a cerner la bonne ncase du tableua.
  55.                           cout<<"la mediane est:\t"<<liste_val[med];
  56.                                
  57.                        
  58.                   }
  59.     pval.clear();
  60.     peffect.clear();
  61.     liste_val.clear();
  62.                  
  63.     system("PAUSE" );
  64.     return EXIT_SUCCESS;
  65. }
  66.    
  67.     float somme_val(){
  68.                 float x=0;
  69.                 for(int i=0; i<nbvaleurs; i++){
  70.                          x += pval[i];
  71.                          }
  72.                  return x;
  73.                  }
  74.                
  75.     float somme_effect (){
  76.                  float x=0;
  77.                  for(int i=0; i<nbvaleurs; i++){
  78.                          x += peffect[i];
  79.                          }
  80.                  return x;
  81.                  }
  82.                
  83.     float somme_totale(){
  84.                    float x=0;
  85.                    for(int i=0; i<nbvaleurs; i++){
  86.                           x += pval[i] * peffect[i];
  87.                           }
  88.                    return x;
  89.                    }
  90.                  
  91.     void liste(){
  92.           for(int i=0; i<nbvaleurs; i++){
  93.                   for(int n=0; n<peffect[i]; n++){
  94.                           liste_val.push_back(pval[i]);
  95.                           }
  96.                   }
  97.           for(int i=0 ; i <liste_val.size() ; i++){//affiche la liste du tableau
  98.           cout <<liste_val[i]<< endl;}
  100.           }


 
C'est mieux?


Message édité par elephant13 le 25-05-2006 à 19:46:12
Reply

Marsh Posté le 25-05-2006 à 20:25:55    

mêmes réflexions que hier ... des tonnes de variables pas initialisées. Sors-toi les mains des poches et mets toi au boulot.
 
et dégage toutes ces variables globales;

Message cité 1 fois
Reply

Marsh Posté le 25-05-2006 à 22:32:08    

JE COMPREND PAS CE QUE TU DIS.
Sors-toi les mains des poches et exprime toi explicitement!!!

Message cité 2 fois
Reply

Marsh Posté le 25-05-2006 à 22:50:01    

ouvre un dictionnaire et compare : initialiser et declarer.
Faudra aussi voir à prendre son vallium [:pingouino]

Reply

Marsh Posté le 26-05-2006 à 02:08:06    

elephant13 a écrit :

JE COMPREND PAS CE QUE TU DIS.
Sors-toi les mains des poches et exprime toi explicitement!!!


 
"Initialiser une variable" signifie grosso-modo "lui donner dès le départ une valeur connue avant de s'en servir"
 
J'ai pas regardé ton code mais apparemment tu as plein de variables dont les valeurs ne sont pas connues au départ. Si tu les remplis avant de les traiter ça peut passer mais si tu commences à les traiter avant de les remplir, ça va merder.


---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
Reply

Marsh Posté le 26-05-2006 à 02:19:55    

elephant13 a écrit :

JE COMPREND PAS CE QUE TU DIS.
Sors-toi les mains des poches et exprime toi explicitement!!!


Figure 1 : des méfaits de la caféine en intraveineuse


---------------
J'ai un string dans l'array (Paris Hilton)
Reply

Marsh Posté le 26-05-2006 à 09:40:54    

Taz a écrit :

et dégage toutes ces variables globales;


 
je suis foutu, je commence à mettre des ; à la fin des phrases;

Message cité 1 fois
Reply

Marsh Posté le 26-05-2006 à 09:42:06    

Taz a écrit :

je suis foutu, je commence à mettre des ; à la fin des phrases;


[:ddr555]


---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
Reply

Marsh Posté le 26-05-2006 à 16:19:30    

Merci Sve@r,
Mai si mon premier traitement de la variable c'est lui donner une valeur grace a "cin" je n'est pas besoin de l'initialiser avant, non? Ou en ai l'interet?

Message cité 1 fois
Reply

Marsh Posté le 26-05-2006 à 20:18:28    

elephant13 a écrit :

Merci Sve@r,
Mai si mon premier traitement de la variable c'est lui donner une valeur grace a "cin" je n'est pas besoin de l'initialiser avant, non? Ou en ai l'interet?


 
Là, il y a généralement 2 attitudes
- je n'initialise que les variables qu'il faut, juste avant de m'en servir (faut être sûr de soi)
- j'initialise toutes mes variables dès le début comme ça je suis certain de ne pas en oublier
 
Personnellement je préfère la première, mais d'autres préfèrent la seconde. La seconde est généralement mieux quand on est en groupe de programmeurs car quand les programmeurs se plongent dans le code de leurs copains, ils n'ont pas envie de se faire chier à vérifier, en plus de leur boulot, si toutes les variables de leurs copains ont bien une valeur avant qu'elles soient utilisées...


---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
Reply

Marsh Posté le    

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed