Discussion :

[Cassius28]

Bonjour,

Je d�bute dans la programmation C++ et pour m'entrainer un peu, j'ai voulu implementer la class string avec quelques fonctions.
J'ai quelques soucis avec la classe +.
Je sais que le probl�me vient surement du pointeur mais je n'arrive pas � le resoudre.
Mon constructeur est "impos�" avec un const char*.
Donc je voulais savoir si j'ai rat� un truc et quelle est la meilleure solution pour avoir en sortie un string concatener. (je sais que c'est pas la meilleure solution, pour l'instant je veux juste avoir en retour un string concatener)

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
    MyString operator +(MyString a){
        int al = a.length();//length() deja implementé 
        int bl = this->length();
        char * charRes = new char [al+bl+1];
        for (int i = 0; i <= bl; i++) {
              charRes[i] = this->m_string[i];     
             }
                for (int i = bl+1; i <= bl+al+1; i++) {
             charRes[i] = a.m_string[i-(bl+1)];
                        }
     MyString res(charRes);
 
    return res ;
    }

[Bousk]

Tu as le droit de dire quel est le probl�me, parce que "j'ai un probl�me" ou "�a marche pas", �a n'a jamais donn� beaucoup d'infos.
Ensuite, t'as rat� un sacr� truc, le chapitre 0 des tableaux qui dit qu'on it�re sur [0, length-1] sur un tableau de taille length.
Parce qu'autant ta premi�re boucle va juste copier le \0, autant la seconde va overflow et le const char* final va �tre coup� parce qu'il s'arr�te probablement sur le premier \0 rencontr�.
Faudrait aussi g�rer la m�moire, parce que �a sent la fuite un peu partout.

[Cassius28]

Merci pour la r�ponse rapide.
Alors le probl�me exacte est que lorsque j'affiche "charRes", j'ai bien mes deux string concaten�s. Mais lorsque j'appelle la fonction dans le main, je n'ai que un string qui apparait. Est ce aussi un probl�me de m�moire ? Car je n'ai pas 'limpression que �a overflow. Je me trompe surement mais bon.

[foetus]

Ton probl�me est trivial

En recopiant ta premi�re cha�ne de 0 � bl inclus, tout simplement tu recopies le caract�re sentinelle '\0'.
� v�rifier, mais tu recopies bl + 1 caract�res puis al + 1 : donc il doit y avoir 1 d�passement

Et ton code, c'est 95% du C ... inutile parce qu'il existe des fonctions comme memset ou strcpy.

[Cassius28]

Bonjour,
Oui vous avez raison, c'est un probl�me de capacit�.
Pour tout ce qui est strcpy, je voulais justement faire sans mais quand je change le +1, �a marche.
Je vous remercie pour votre aide !