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 !