Erreur d'appel au constructeur

Bonjour
je rencontre un probleme dans mon code :
Premier fichier le header: Etudiant.h

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 34 35

#ifndef ETUDIANT_H_INCLUDED #define ETUDIANT_H_INCLUDED   #include <iostream> #include <string>   //Class Etudiant   class Etudiant { public : //Constructeur par défaut Etudiant(); //Constructeur avec parametres pour la surcharge Etudiant(std::string e_Nom,std::string e_Prenom,int e_Age); //Detructeur ~Etudiant(); std::string afficherNom(); std::string afficherPrenom(); int afficherAge();       private: //Attributs std::string m_nom; std::string m_prenom; int m_age;   };         #endif // ETUDIANT_H_INCLUDED

---

Fichier Etudiant.cpp

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 34 35 36 37 38

#include <iostream> #include <string> #include "Etudiant.h"   using namespace std;   //Etudiant constructeur par défaut Etudiant::Etudiant() { m_nom = ""; m_prenom= ""; m_age = 0; } //Etudiant avec surcharge du constructeur Etudiant::Etudiant(string e_Nom,string e_Prenom,int e_Age):m_nom(e_Nom),m_prenom(e_Prenom),m_age(e_Age) {   } //Destructeur Etudiant::~Etudiant() {   } //Afficher le nom string Etudiant::afficherNom() { return m_nom; } //Afficher le prenom string Etudiant::afficherPrenom() { return m_prenom; } //Afficher l'age int Etudiant::afficherAge() { return m_age; }

---

Fichier main

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#include <iostream> #include <string> #include "Etudiant.h" using namespace std;   int main() { //On crée un objet personne Etudiant Personne; Personne.Etudiant("toto","lele",40);// ici j'ai un message d'erreur : Invalid use of Etudiant::Etudiant   return 0; }

---

Merci d'avance pour votre aide

Bernard

Hello,

Citation:

---

Envoyé par zoltars43

Code:

---

1 2 3 4 5 6 7 8

int main() { //On crée un objet personne Etudiant Personne; Personne.Etudiant("toto","lele",40);// ici j'ai un message d'erreur : Invalid use of Etudiant::Etudiant   return 0; }

---

---

Un constructeur ne s'appelle qu'à la création de l'objet, pas après.

Code:

---

1 2 3 4 5 6 7

int main() { //On crée un objet personne Etudiant Personne("toto","lele",40);   return 0; }

---

Merci de ta réponse rapide
que tu pense tu de mon code ?
j'apprend les classes en ce moment et pas facile .

Bonjour,

Pas grand chose à dire du code. N'oubliez pas d'utiliser les listes d'initialisation (notamment pour le constructeur par défaut).
De plus, n'oubliez pas de passer par des références constantes dans ce cas :

Code:

---

Etudiant(std::string e_Nom,std::string e_Prenom,int e_Age);

---

->

Code:

---

Etudiant(const std::string& e_Nom,const std::string& e_Prenom,int e_Age);

---

Cela évitera des copies inutiles en interne.

Bonjour,

Créer un constructeur par défaut (étudiant sans nom ni prénom ni age) est d'un intérêt très limité ici.
Attention au nom des méthodes : afficherNom(), afficherPrenom(), afficherAge() n'affichent rien du tout, elles ne font que retourner des valeurs. Des noms comme getNom(), getPrenom() getAge() seraient plus corrects. Sans oublier que ces méthodes ne modifient pas l'instance, il serait donc bon de les qualifier de const.

Les includes #include <iostream> dans les deux fichiers .h et cpp de la classe ne servent à rien, autant les supprimer, de même que les #include <string> dans le deux .cpp : il est passé avec l'#include "Etudiant.h"

Votre constructeur par défaut devrait plus ressembler à votre constructeur surchargé.
Dans le sens où, il vaut mieux directement construire les propriétés de l'instance avec les bonne valeurs plutôt que de les construire sans puis assigner les valeurs.

Code:

---

1 2 3 4

Etudiant::Etudiant():m_nom(""), m_prenom(""), m_age(0); { }

---

Le constructeur par défaut de std::string crée déjà une chaîne vide.
Il est donc inutile de les initialiser à "" ;)