Problème d'affichage (c++)

Version imprimable

1 pièce(s) jointe(s)

Bonjour,

Mon code est le suivant:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
 
#include <iostream>
using namespace std;
 
class Souris
{
 
   public: double poids;
   public: string couleur;
   public: unsigned int age;
   public: unsigned int esperancevie;
   public: bool clonee;
   Souris(double p,string c,unsigned int a, unsigned int esperancevie=36){
	   poids=p;
	   couleur=c;
	   age=a;
	   clonee=false;
	   esperancevie=36;
	   cout<<"Une nouvelle souris avec esperancevie "<<esperancevie<<endl;
   }
 
   public: void afficher(){
	   cout<< "esperance dans afficher "<<esperancevie<<endl;
   }
 
   };
int main()
{
  Souris s1(50.0, "blanche", 2);
  s1.afficher();
 
  return 0;
}

J'ai eu:
http://www.developpez.net/forums/att...1&d=1394823925
Ma question est la suivante: pourquoi j'ai eu deux valeurs différentes pour esperancedevie ?

Avez-vous une idée ?
Merci d'avance.

14/03/2014, 15h12
chercheur111

J'ai changé:

Code:

Souris(double p,string c,unsigned int a, unsigned int esperancevie=36)

par:

Code:

Souris(double p,string c,unsigned int a, unsigned int esperancevie1=36){

Et ça marche :)

Merci.
14/03/2014, 15h17
skeud
Alors déjà plusieurs chose:
Le mot clef public n'est pas obligatoire pour chaque déclaration, une seule fois suffit:
Code:

1 2 3 4 5 6 7 public: void mafunc(); voi mafunc2(); private: int toto; int titi; int tutu;
Dans ce cas, les deux fonctions sont public et les trois int sont private. En voyant ton code, je devine que tu viens du monde java ^^. En c++ on fait les choses un peu différemment (certes ça reviens au meme, mais c'est plus joulis).

Ensuite es-tu sur que ton code compile?

Dans ton constructeur, ta variable d'entrée esperancedevie possède le meme nom que ton attribut. Quand tu modifie esperancedevie dans ton constructeur, en fait tu modifies ta variable d'entré et non pas ton attribut.

Une bonne pratique en c++ est de préfixer tes attribut:
Code:

1 2 3 4 5 6 7 8 9 10 class Maclasse { public: Maclasse(int val1, int val2) { m_val1 = val1; m_val2 = val2; } private: int m_val1; int m_val2; };
Ainsi tu évites ce genre de soucis. En conclusion, essaye de regarder les bonne pratiques c++:
_ visibilité des attributs
_ nommage des variables
_ déclaration dans le header, code dans un cpp.
_ compilation avec -Wall (affichage de tout les warning) -Werror (arret de la compilation lorsqu'un warning est affiché).

Tu peux utiliser les listes d'initialisation, qui évitent ce problème quand les noms sont identiques:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 #include <iostream>
using namespace std;
 
class Souris
{
 
	public: double poids;
	public: string couleur;
	public: unsigned int age;
	public: unsigned int esperancevie;
	public: bool clonee;
 
	Souris(double p,string c,unsigned int a, unsigned int esperancevie=36)
	: poids(p), couleur(c), age(a), esperancevie(esperancevie), clonee(false) {
		cout << "Une nouvelle souris avec esperancevie " << esperancevie << endl;
	}
 
	public: void afficher(){
		cout<< "esperance dans afficher " << esperancevie << endl;
	}
};
 
int main()
{
	Souris s1(50.0, "blanche", 2);
	s1.afficher();
 
	return 0;
}

PS: Ce n'est pas du C++/CLI, c'est du C++ normal. Tu postes dans la mauvaise section du forum.

14/03/2014, 17h20
chercheur111

Merci beaucoup pour vos réponses :)