Discussion :

[phenix1988]

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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
#include <iostream>
using namespace std; 
 
 
 
class zstring
{
    private:
    char *m_txt;
    int m_taille;
 
 
    public:
    zstring();
    zstring(const char*);
    zstring(const zstring &);
    ~zstring();
 
    zstring operator=(const zstring &);
    zstring operator=(char*);
    zstring operator+(const zstring &);
    zstring operator+(char*);
    bool operator==(const zstring &);
    bool operator<(const zstring &);
    bool operator<<(const zstring &);
    int taille(const char*);
    char* copier(const char *);
    void afficher();
 
};
 
//Constructeur par défaut
zstring::zstring()
{
    m_txt=NULL;
    m_taille=0;
}
// le constructeur
 
zstring::zstring(const char* ch)
{
    m_txt=copier(ch);
    m_taille=taille(ch);
}
 
int zstring::taille(const char *ch)
{
    int i=0;
    while(ch[i]!='\0')
    i++;
return i;
}
 
char* zstring::copier(const char *ch)
{
    int i=0;
    char *copie;
    while(ch[i]!='\0')
    {
    copie[i]=ch[i];
    i++;
    }
    copie[i]='\0';
    return copie;
}
 
zstring::zstring(const zstring & chaine)
{
    m_txt=copier(chaine.m_txt);
    m_taille=chaine.m_taille;
}
 
zstring::~zstring()
{
    delete[] m_txt;
}
 
void zstring::afficher()
{
    cout << m_txt << endl;
}
 
 
int main()
{
    zstring chaine("Bonjour");
        chaine.afficher();
 
 
    return 0;
}

[loufoque]

Parce que ton code invoque un comportement indéfini ?

[Arzar]

Salut!
Alors déjà, un delete[] non accompagné de son new[], c'est pas bon signe.
De plus :

Code :

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

1
2
3
4
5
6
7
8
9
 
char* zstring::copier(const char *ch)
{
int i=0;
char *copie;
while(ch[i]!='\0')
{
copie[i]=ch[i];
...

Ouch.
Char* copie est déclaré mais pas initialisé. Tu écris dedans alors qu'il pointe quelque part au hasard dans la mémoire.

[camboui]

Sympathique code de débutant. Soyez indulgents
Je modifierais la 2ème ligne de la méthode copier() comme ceci:

Code :

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

1
2
3
...
char *copie=new char[taille(ch)+1];
...

Ça devrait être suffisant pour afficher "bonjour".

[white_tentacle]

Je suppose que c'est un exercice (sinon, utilise std::string).

Donc, pose-toi aussi la question suivante : que se passe-t-il si on fait le code suivante :

Code :

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

1
2
3
4
5
zstring chaine("Bonjour");
chaine.afficher();
chaine.copier(" tout le monde");
chaine.afficher();
return 0;

(après avoir pris en compte les remarques d'Arzar et de Camboui)

[phenix1988]

merci ça marche maintenant, j'ai oublié d'initialiser copie

[white_tentacle]

Relis les commentaires qui ont été faits, et réfléchis-y.

Tu as oublié d'autres choses