Discussion :

[exhortae]

Bonsoir,

voilà j'essaie d'implementer une classe String, pour débuter je veux juste surdéfinir l'opérateur = afin de pouvoir faire :

Code :

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

1
2
3
4
String str1 ("bb");
String str2;
 
str2 = str1;

J'ai donc pondu le code suivant :

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
39
40
41
42
43
44
45
46
47
48
49
#include <iostream>
 
using namespace std;
 
class String
{
   private :
 
      char* m_string;
 
   public :
 
      String (char *string = "")
      {
         m_string = new char[strlen (string) + 1];
         strcpy (m_string, string);
      }
 
      ~String ()
      {
         delete m_string, m_string = NULL;
      }
 
      String & operator= (const String &right)
      {
         delete m_string, m_string = NULL;
 
         String (right.m_string);
 
         return *this;
      }
 
      void display ()
      {
         cout << m_string;
      }
};
 
int main ()
{
   String str1;
   String str2 ("Salut jonjon");
 
   str1 = str2;
 
   str1.display ();
 
   return EXIT_SUCCESS;
}

dans la fonction operator= j'essaie d'appeler le constructeur de l'objet pour pouvoir changer sa valeur, mais ça ne marche pas.
j'aimerais éviter de faire

Code :

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

1
2
3
4
5
6
7
8
9
      String & operator= (const String &right)
      {
         delete m_string, m_string = NULL;
 
         m_string = new char[strlen (right.m_string) + 1];
         strcpy (m_string, right.m_string);
 
         return *this;
      }

merci

[Luc Hermitte]

L'idiome du sawp s'applique parfaitement à ce cas.

Code :

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

1
2
3
4
5
String & String::operator=(String const& rhs) {
   String tmp(rhs);
   this->swap(tmp);
   return *this;
}

PS: Tu a oublié plein de "const", et c'est delete[] qu'il te faut appeler, pas delete.

[exhortae]

L'idiome du sawp s'applique parfaitement à ce cas.

Code :

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

1
2
3
4
5
String & String::operator=(String const& rhs) {
   String tmp(rhs);
   this->swap(tmp);
   return *this;
}

PS: Tu a oublié plein de "const", et c'est delete[] qu'il te faut appeler, pas delete.

Bonsoir,

désolé mais je ne sais pas ce qu'est l'idiome du swap,

faut-il que je définisse dans ma classe une fonction

void swap (const String &str)

mais franchement je vois pas trop comment faire ?

Sinon pour les const oui j'en ai oublié (faut dire que je débute en c++), je posterais le code dans un autre topic pour correction afin d'avoir de bonnes habitudes.

Merci

[Davidbrcz]

Un article traitant de ca :
http://en.wikibooks.org/wiki/More_C%.../Copy-and-swap

[exhortae]

Un article traitant de ca :
http://en.wikibooks.org/wiki/More_C%.../Copy-and-swap

Bonsoir,

oauis c'est pas facile cette histoire , je vais laisser en suspend (le temps que j'arrive à la partie des exceptions) pour pouvoir comprendre tout ça. En attendant j'applique la 2ème solution qui est plus simple.

Sinon une dernière chose

je sais que

const int *p c'est pas la même chose que int * const p

mais est_ce que const String &right c'est la même chose que String const &right


Merci

[Jean-Marc.Bourguet]

Je sais que

const int *p c'est pas la même chose que int * const p

mais est_ce que const String &right c'est la même chose que String const &right

Oui. De même que const int* p est la même chose que int const* p.

[nicroman]

const int *p c'est pas la même chose que int * const p

mais est_ce que const String &right c'est la même chose que String const &right

En fait, pour savoir, il suffit de parenthéser les éléments deux à deux en partant de la gauche:

String const & right => ((String const) & ) right
right est donc de type ((String const) &) qui est une référence sur (String const) qui est un String constant.

const String & right => ((const String) & ) right
right est donc de type ((const String) &) qui est une référence sur (const String) qui est un String constant (const T et T const sont similaires).

int * const p => ((int *) const) p
p est donc de type ((int *) const) qui est une valeur constante de pointeur sur int (donc un pointeur constant).

const int * p => ((const int) *) p
p est donc de type ((const int) *) soit un pointeur sur un entier constant

[Davidbrcz]

C'est normal que ca ne marche pas !
Dans operator= , String (right.m_string) crée un objet temporaire mais tu ne touche aucunement à *this.

De ce fait, une 1er solution pourrait être :

Code :

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

1
2
3
4
5
6
7
8
9
 
String & operator= (const String &right)
      {
         delete m_string, m_string = NULL;
 
         *this = String (right.m_string); // ca va planter ici
 
         return *this;
      }

Mais ce code va plante car tu as un appel récursif non controlé de operator=

Il faut donc alors recopier manulement la chaine dans operator= .

Edit : Ou passer par une fonction swap comme le dit Luc