Discussion :

[Invité]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Iradrille]

Hello,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
int main()
{
//On crée un objet personne
    Etudiant Personne("toto","lele",40);
 
    return 0;
}

[Invité]

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

[LittleWhite]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

->

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Cela évitera des copies inutiles en interne.

[Invité]

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"

[Sergejack]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
Etudiant::Etudiant():m_nom(""), m_prenom(""), m_age(0);
{
}

[Bousk]

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