opérateur This .

Version imprimable

opérateur This .

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:

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:

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:

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.

03/12/2010, 18h19
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é:ccool:

Le this est utile pour la surcharge de operator=() et d'autre cas plus subtil et tu en auras rarement l'utilité.
03/12/2010, 20h35
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 .
03/12/2010, 20h40
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.
03/12/2010, 21h07
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:

1 2 3 4 5 Ville::Ville(void):NomVille ("Inconnu"), NombreHabitant(0), NomPays("Inconnu"), categorie_ville("Inconnue"){}
03/12/2010, 22h26
10_GOTO_10

Citation:

Envoyé par exe2bin

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
04/12/2010, 20h54
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