Discussion :

[alfred5]

Bonsoir tt le monde;
je suis entrain de réaliser un petit exercice on créant un objet "ville".
pour cette effet j'ai créé 3 fichier : main.cpp ; ville.cpp et ville.h
au cours d'implémentation de mes méthodes, j'ai utilisé l'opérateur this pour référencer l'objet courant.
mais lorsque je compile mon projet, il me retourne des erreurs qui ont une relation avec "this".
voile le contenu des 3 fichier:
main.cpp :

Code :

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

1
2
3
4
5
6
7
#include <iostream>
#include <string>
using namespace std;
int main(int argc, char *argv[])
{
    system("PAUSE");
}

ville.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
#include <iostream>
#include <string>
#include "ville.h"
 
using namespace std;
/* 
   Implémentation ;
*/
// Constructeur par défaur : Initialiser les paramêtres .
   Ville::Ville(void){
         NomVille ="Inconnu";
         NombreHabitant = 0;
         NomPays="Inconnu";
         categorie_ville="Inconnue";
   }
// Constructeur surchargé :
   Ville::Ville(string Nom,int Nombre ,string Pays){
         NomVille= Nom;
         NombreHabitant = Nombre ;
         NomPays = Pays;
   }
// Catégorie de la ville 
   void Ville::categorie(int nombre_cat){
         int tab_cat[3]={0,10000,100000};
         string categorie[3]={"?","A","B"};      
         for(int i=0;i<=2;i++){
               if(this.NombreHabitant >= tab_cat[i]){
                     this.categorie_ville=categorie[i];
                       }
         }
   }
// Description de la ville .
   string Ville::description(void){
         cout<<this.NomVille<<" est une ville de "<<this.NomPays<<" ;elle comporte "<<this.NombreHabitant<<" habitant "<<endl;
         cout<<" Donc; Elle est de categorie "<<this.categorie<<endl;
   }

ville.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
using namespace std;
class Ville{
      private :
          string NomVille;
          int NombreHabitant;
          string NomPays;
          string categorie_ville;
      public :
             Ville (void); // Constructeur par défaut.
             Ville (string,int,string);// Constructeur paramétré .
             void categorie(int);
             string description(void);
             //comparer(int,int);
             ~Ville(void);        
};

SVP de m'aider de corriger mes erreurs.

[Astraya]

Bonsoir,
Le this ne te sert à rien dans ce cas de figure. Supprime le carrément.

Note: en c++, pas de paramètre veux dire rien, en C veux dire un void ( je crois ). Tu ne dois pas mettre de void en C++ à ton constructeur, le compilateur le fera pour toi et ça sera dans l'ensemble un code moins chargé de choses inutiles à sa relecture. Tout comme le this que tu as rajouté

Le this est utile pour la surcharge de operator=() et d'autre cas plus subtil et tu en auras rarement l'utilité.

[exe2bin]

Salut à toi ;
pourrais tu préciser quelles erreurs !?
Ton code à l'air assez sain et bien que "this" ne serve à rien dans ton programme
je vois pas pourquoi il générerait des erreurs .
Corrige aussi la déclaration de ton Destructeur qui ne doit JAMAIS comporter
d'arguments , même void .

[10_GOTO_10]

Corrige aussi la déclaration de ton Destructeur qui ne doit JAMAIS comporter d'arguments , même void .

Tu dois confondre avec le type de retour. Pour ce qui est des arguments, le void est inutile, mais pas faux.

Cours de C/C++ : http://cpp.developpez.com/cours/cpp/...page_10#LX-H-1

[alfred5]

Bonsoir tout le monde;
merci bcp pour votre réponse;
Pour gl :si je supprime la ligne using namespace std; dans le fichier .h
pour exe2bin : je pense que y a pas de souci de mettre void comme paramètre du destructeur.jel'ai mis et pas de problème.
le compilateur me retourne un erreur.
de toute façon j'ai supprimé l'opérateur this....et tout ce que vous avez remarqué
je copile maintenant, et il y aura aucun erreur.je vais maintenat

[gl]

En complément de la réponse de Astraya, this est un pointeur, ce n'est donc pas this.XXX mais this->XXXXX.

Sinon au passage :

• Il y a un loupé dans l'utilisation du nom categorie. En particulier this.categorie dans la fonction description() est incorrect (n'existe pas).
• On ne met pas de using namespace dans un header.
• La fonction description() est censé retourner une string mais ne retourne rien.

[guillaume07]

je rajouterais qu'il est préférable d'initialiser les membres de la classe comme ceci
( http://cpp.developpez.com/faq/cpp/?p...EUR_liste_init )

Code :

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

1
2
3
4
5
 
   Ville::Ville(void):NomVille ("Inconnu"),
         NombreHabitant(0),
         NomPays("Inconnu"),
         categorie_ville("Inconnue"){}