Discussion :

[etiennegaloup]

Bonjour,

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

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

[etiennegaloup]

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

Code :

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

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++.

++

[Jean-Marc.Bourguet]

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

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
$ 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 :

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

  1
  2
  3
  4
  CClient::CClient(int id)
     : m_id(id)
  {
  }

[etiennegaloup]

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 :

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

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 :

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

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

C bien ça ?

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

En fait, je dois mettre mon code entier ?

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 :

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
 
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.
++

[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.

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.

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.

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.

[etiennegaloup]

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.

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.
++