C++

Petit logiciel qui ne fonctionne pas toujours

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Petit logiciel qui ne fonctionne pas toujours

elephant13

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 :

#include <cstdlib>
#include <iostream>
using namespace std;
float somme_val();
int somme_effect();
float somme_totale();
float liste();
int nbvaleurs, choix, med;
float moy;
float * pval = new float[nbvaleurs];
int * peffect = new int[nbvaleurs];
float * liste_val = new float[somme_effect()];
int main(int argc, char *argv[])
{
cout<<"rentrez le nombre de valeurs differentes."<<endl;
cin>>nbvaleurs;
for(int i=0; i<nbvaleurs; i++){
cout<<"rentrez une valeur";
cin>>pval[i];
cout<<"rentrez son effectif";
cin>>peffect[i];
}
for(int i=0; i<nbvaleurs; i++){
cout<<pval[i]<<"\t";
}
cout<<endl;
for(int i=0; i<nbvaleurs; i++){
cout<<peffect[i]<<"\t";
}
cout<<endl<<"pour calculer la moyenne tapez1, la mediane tapez2."<<endl;
cin>>choix;
switch(choix){
case 1: moy = somme_totale()/somme_effect();
cout<<"la moyenne est de:"<<moy<<endl;
break;
case 2: liste();//pour somme_effect impaire
int j = somme_effect();
med = ((j+1)/2)-1;//le -1 sert a cerner la bonne case du tableau.
cout<<"la mediane est:\t"<<liste_val[med];
}
delete pval, peffect, liste_val;
system("PAUSE" );
return EXIT_SUCCESS;
}
float somme_val(){
float x;
for(int i=0; i<nbvaleurs; i++){
x += pval[i];
}
return x;
}
int somme_effect (){
float x; //ici si je met int pour correspondre au type de la fonction, la moyenne est fausse.
for(int i=0; i<nbvaleurs; i++){
x += peffect[i];
}
return x;
}
float somme_totale(){
float x;
for(int i=0; i<nbvaleurs; i++){
x += pval[i] * peffect[i];
}
return x;
}
float liste(){
int m= 0;
for(int i=0; i<nbvaleurs; i++){
for(int n=0; n<peffect[i]; n++){
liste_val[m]= pval[i];
m++;
}
}
}

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

Publicité

Taz

bisounours-codeur

utilise std::vector<float>

et utilise ton compilateur correctement : t'as des dizaines de variables pas initialisées ...

elephant13

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

kirua_sama

Learn sciences with senses

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

elephant13

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

elephant13

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

Code :

#include <cstdlib>
#include <iostream>
#include <vector>
using namespace std;
float somme_val();
int somme_effect();
float somme_totale();
float liste();
int nbvaleurs, choix, med;
float moy;
std::vector <float> pval (nbvaleurs);
std::vector <int> peffect (nbvaleurs);
std::vector <float> liste_val;
int main(int argc, char *argv[])
{
cout<<"rentrez le nombre de valeurs differentes."<<endl;
cin>>nbvaleurs;
for(int i=0; i<nbvaleurs; i++){
cout<<"rentrez une valeur";
cin>>pval[i];
cout<<"rentrez son effectif";
cin>>peffect[i];
}
for(int i=0; i<nbvaleurs; i++){
cout<<pval[i]<<"\t";
}
cout<<endl;
for(int i=0; i<nbvaleurs; i++){
cout<<peffect[i]<<"\t";
}
cout<<endl<<"pour calculer la moyenne tapez1, la mediane tapez2."<<endl;
cin>>choix;
switch(choix){
case 1: moy = somme_totale()/somme_effect();
cout<<"la moyenne est de:"<<moy<<endl;
break;
case 2: liste();
int j = somme_effect();
med = ((j+1)/2)-1;//le -1 sert a cerner la bonne ncase du tableua.
cout<<"la mediane est:\t"<<liste_val[med];
}
pval.clear();
peffect.clear();
liste_val.clear();
system("PAUSE" );
return EXIT_SUCCESS;
}
float somme_val(){
float x;
for(int i=0; i<nbvaleurs; i++){
x += pval[i];
}
return x;
}
int somme_effect (){
float x;
for(int i=0; i<nbvaleurs; i++){
x += peffect[i];
}
return x;
}
float somme_totale(){
float x;
for(int i=0; i<nbvaleurs; i++){
x += pval[i] * peffect[i];
}
return x;
}
float liste(){
for(int i=0; i<nbvaleurs; i++){
for(int n=0; n<peffect[i]; n++){
liste_val.push_back(pval[i]);
}
}
for(int i=0 ; i <liste_val.size() ; i++){
cout <<liste_val[i] << endl;}
}

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

Taz

bisounours-codeur

#
float somme_val(){
#
float x;

mais bien sur ...

LePhasme

Les Belges domineront le monde

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 :

int somme_effect (){
                 float x;
                  for(int i=0; i<nbvaleurs; i++)
                  {
                    x += peffect[i];
                   }
                 return x;
                 }

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...

skelter

http://en.wikipedia.org/wiki/Accumulate_and_fire

elephant13

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

Code :

#include <cstdlib>
#include <iostream>
#include <vector>
using namespace std;
float somme_val();
float somme_effect();
float somme_totale();
float liste();
int nbvaleurs, choix, med;
float moy, med1;
std::vector <float> pval;
std::vector <int> peffect;
std::vector <float> liste_val;
int main(int argc, char *argv[])
{
cout<<"rentrez le nombre de valeurs differentes."<<endl;
cin>>nbvaleurs;
for(int i=0; i<nbvaleurs; i++){
int j;
cout<<"rentrez une valeur";
cin>>j;
pval.push_back(j);
cout<<"rentrez son effectif";
cin>>j;
peffect.push_back(j);
}
for(int i=0; i<nbvaleurs; i++){
cout<<pval[i]<<"\t";
}
cout<<endl;
for(int i=0; i<nbvaleurs; i++){
cout<<peffect[i]<<"\t";
}
cout<<endl<<"pour calculer la moyenne tapez1, la mediane tapez2."<<endl;
cin>>choix;
switch(choix){
case 1: moy = somme_totale()/somme_effect();
cout<<"la moyenne est de:"<<moy<<endl;
break;
case 2: liste();
int j = static_cast<int>(somme_effect());
med = ((j+1)/2)-1;//le -1 sert a cerner la bonne ncase du tableua.
cout<<"la mediane est:\t"<<liste_val[med];
}
pval.clear();
peffect.clear();
liste_val.clear();
system("PAUSE" );
return EXIT_SUCCESS;
}
float somme_val(){
float x;
for(int i=0; i<nbvaleurs; i++){
x += pval[i];
}
return x;
}
float somme_effect (){
float x;
for(int i=0; i<nbvaleurs; i++){
x += peffect[i];
}
return x;
}
float somme_totale(){
float x;
for(int i=0; i<nbvaleurs; i++){
x += pval[i] * peffect[i];
}
return x;
}
float liste(){
for(int i=0; i<nbvaleurs; i++){
for(int n=0; n<peffect[i]; n++){
liste_val.push_back(pval[i]);
}
}
for(int i=0 ; i <liste_val.size() ; i++){//affiche la liste du tableau
cout <<liste_val[i]<< endl;}
}

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

Publicité

skelter

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)

elephant13

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 :

#include <cstdlib>
#include <iostream>
#include <vector>
using namespace std;
float somme_val();
float somme_effect();
float somme_totale();
void liste();
int nbvaleurs, choix;
std::vector <float> pval;
std::vector <int> peffect;
std::vector <float> liste_val;
int main(int argc, char *argv[])
{
cout<<"rentrez le nombre de valeurs differentes."<<endl;
cin>>nbvaleurs;
for(int i=0; i<nbvaleurs; i++){
int j;
cout<<"rentrez une valeur";
cin>>j;
pval.push_back(j);
cout<<"rentrez son effectif";
cin>>j;
peffect.push_back(j);
}
for(int i=0; i<nbvaleurs; i++){
cout<<pval[i]<<"\t";
}
cout<<endl;
for(int i=0; i<nbvaleurs; i++){
cout<<peffect[i]<<"\t";
}
cout<<endl<<"pour calculer la moyenne tapez1, la mediane tapez2."<<endl;
cin>>choix;
switch(choix){
case 1:float moy;
moy = somme_totale()/somme_effect();
cout<<"la moyenne est de:"<<moy<<endl;
break;
case 2: liste();
int med;
int j = static_cast<int>(somme_effect());
med = ((j+1)/2)-1;//le -1 sert a cerner la bonne ncase du tableua.
cout<<"la mediane est:\t"<<liste_val[med];
}
pval.clear();
peffect.clear();
liste_val.clear();
system("PAUSE" );
return EXIT_SUCCESS;
}
float somme_val(){
float x=0;
for(int i=0; i<nbvaleurs; i++){
x += pval[i];
}
return x;
}
float somme_effect (){
float x=0;
for(int i=0; i<nbvaleurs; i++){
x += peffect[i];
}
return x;
}
float somme_totale(){
float x=0;
for(int i=0; i<nbvaleurs; i++){
x += pval[i] * peffect[i];
}
return x;
}
void liste(){
for(int i=0; i<nbvaleurs; i++){
for(int n=0; n<peffect[i]; n++){
liste_val.push_back(pval[i]);
}
}
for(int i=0 ; i <liste_val.size() ; i++){//affiche la liste du tableau
cout <<liste_val[i]<< endl;}
}

C'est mieux?

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

Taz

bisounours-codeur

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

elephant13

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

Message cité 2 fois

Joel F

Real men use unique_ptr

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

Sve@r

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.

Harkonnen

Modérateur
Un modo pour les bannir tous

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)

Taz

bisounours-codeur

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

Sve@r

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.

elephant13

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

Sve@r

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.

Publicité

FORUM HardWare.fr

Programmation

C++

Petit logiciel qui ne fonctionne pas toujours

Sujets relatifs
Comment fonctionne le md5 dans les .htpasswd ?	petit php
petit probleme de selection multiple avec date	[SHELL] petit script facil mais gros soucit
Besoin de quelques pro pour un petit log	Logiciel de blog ?
TextBox : Petit carré devant chaque saut de ligne	essai de group by (ne fonctionne pas vraiment)
Logiciel pour laboratoire	Quel CMS pour un petit site de musique ?
Plus de sujets relatifs à : Petit logiciel qui ne fonctionne pas toujours

Page générée en 0.280 secondes