[Debutant] Initialisation d'une classe

Bonjour,

je m'exerce en C++. Je développe une classe Client, qui représente un Client.

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

class CClient { public: CClient(int id); virtual ~CClient();   private: int m_id;   public: void setId(int id) {m_id=id;} int getId() {return m_id;} [..] } [..] CClient::CClient(int id){ setId(id); } [..] int main(int argc, char *argv[]) { CClient client(4); cout<<endl<<client.getId()<<endl; } [..]

---

En sortie, j'obtiens une valeur à la con. -858993460 au lieu d'obtenir la valeur 4.
Est-ce que d'après vous je fait une erreur qui vous semble flagrante ?

Merci
Ciao

J'ai trouvé mon erreur.
En fait dans le code laissé, il manque:

Code:

---

1 2 3 4 5 6 7 8 9 10

class CClient { public: CClient(); CClient(int id); virtual ~CClient(); [..] CClient::Client(){ CClient(0); }

---

Apparemment, on ne peut pas appeler un constructeur depuis un autre constructeur contrairement à Java. Je n'avais pas eu d'erreur. Je m'étais dit que ça allait. Je vais retourner à mon livre de C++.

++

Tu peux poster quelque chose qui reproduit le problème?

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

$ cat etienne.cpp #include <iostream>   class CClient { public: CClient(int id); virtual ~CClient() {}   private: int m_id;   public: void setId(int id) {m_id=id;} int getId() {return m_id;} };   CClient::CClient(int id) { setId(id); }   int main(int argc, char *argv[]) { CClient client(4); std::cout << std::endl << client.getId() << std::endl; } $ como -o etienne etienne.cpp Comeau C/C++ 4.3.3 (Oct 24 2003 16:00:23) for RedHat_LINUX_INTEL_ELF Copyright 1988-2003 Comeau Computing. All rights reserved. MODE:strict errors C++   [~/src]$ ./etienne   4

---

Commentaires gratuits et non solicités:

• j'aime pas using namespace;
• l'idée d'un setter sur un id de client me semble bizarre; a priori CClient devrait être une classe avec sémantique de référence et l'id initialisé dans le constructeur et jamais modifié après
• l'initialisation de membres se fait plutôt:
  Code:

  ---

  1 2 3 4

  CClient::CClient(int id) : m_id(id) { }

  ---

Donc pour initialiser mes variables je dois utiliser une liste d'initialisation comme ce que tu as spécifié ?
Pour l'id, vous avez raison, ce n'est pas très logique.
Et supposons que j'ai les constructeurs suivants:

Code:

---

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

class CClient { public: CClient(LPCWSTR name,LPCWSTR ville,LPCWSTR adresse);   private: LPCWSTR m_name; LPCWSTR m_adresse; LPCWSTR m_ville; [..] }   CClient::Client(LPCWSTR name,LPCWSTR ville,LPCWSTR adresse=L""):m_name(name),m_ville(ville),m_adresse(adresse){ }

---

Et si je souhaite instancier un client sans spécifier le paramètre adresse:

Code:

---

1 2	CClient client(L"Microsoft France",L"Paris");

---

C bien ça ?

Citation:

---

Envoyé par Jean-Marc.Bourguet

Tu peux poster quelque chose qui reproduit le problème?

---

En fait, je dois mettre mon code entier ?

Citation:

---

Envoyé par Jean-Marc.Bourguet

a priori CClient devrait être une classe avec sémantique de référence

---

Que signifie sémantique de référence ?

En fait, oui j'aimerais développer une classe client, j'ai fait ceci, qu'en pensez-vous ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

class CClient { public: CClient(long id,LPCWSTR name,LPCWSTR adresse,LPCWSTR ville); virtual ~CClient();   long getId(){return m_id;}   LPCWSTR getName(){return m_name}; void setName(LPCWSTR name){m_name=name};   LPCWSTR getAdresse(){return m_adresse}; void setAdresse(LPCWSTR adresse){m_adresse=adresse};   LPCWSTR getVille(){return m_ville}; void setVille(LPCWSTR ville){m_ville=ville};   private: long m_id; LPCWSTR m_name; LPCWSTR m_adresse; LPCWSTR m_ville; }

---

Et j'aimerais aussi faire une classe fournisseur. Pensez-vous que je devrais développer une classe plus générale dont les classes Client et Fournisseur hériteraient ?

Qu'en pensez-vous ?
Merci.
++

Citation:

---

Envoyé par etiennegaloup

Citation:

---

Envoyé par Jean-Marc.Bourguet

Tu peux poster quelque chose qui reproduit le problème?

---

En fait, je dois mettre mon code entier ?

---

Non. Couper le plus possible mais garder ce qui fait que le problème puisse être reproduit. C'est énervant d'avoir à faire des petites modifications pour que le code compile... surtout pour ne pas arriver au final à reproduire le problème.

Citation:

---

Citation:

---

Envoyé par Jean-Marc.Bourguet

a priori CClient devrait être une classe avec sémantique de référence

---

Que signifie sémantique de référence ?

---

En gros, une classe où l'opérateur == est fondamentalement un test d'égalité des adresses, généralement sans assignation ni constructeur de copies.

Citation:

---

En fait, oui j'aimerais développer une classe client, j'ai fait ceci, qu'en pensez-vous ?

---

Pouvoir changer le nom semble bizarre, mais ce peut être valide.

Citation:

---

Et j'aimerais aussi faire une classe fournisseur. Pensez-vous que je devrais développer une classe plus générale dont les classes Client et Fournisseur hériteraient ?

---

La question est trop générale (de même que la précédante), on ne conçoit pas des classes dans l'absolu sans un contexte d'utilisation.

Citation:

---

Envoyé par Jean-Marc.Bourguet

Non. Couper le plus possible mais garder ce qui fait que le problème puisse être reproduit. C'est énervant d'avoir à faire des petites modifications pour que le code compile... surtout pour ne pas arriver au final à reproduire le problème.

---

Ok, je ferais gaffe la prochaine fois.

Citation:

---

Envoyé par Jean-Marc.Bourguet

En gros, une classe où l'opérateur == est fondamentalement un test d'égalité des adresses, généralement sans assignation ni constructeur de copies.

---

Bon, je n'ai pas tout compris, j'ai encore tellement de trucs à apprendre.

• Sans assignation : sans méthode setter ?
  ni constructeur de copies. kezako ?

Merci d'avance.
++