Discussion :

[Raiden1234]

Bonjour,

Alors voila, pour continuer mes qestions sur les surcharges d'opérateurs qui me tourmentent, j'en ai une nouvelle qui m'affecte beaucoup :

Entre ces deux surcharges, en tant que fonctions membres, (qui marchent !) de l'opérateur +=, laquelle est la meilleure ?

Code :

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

1
2
3
4
5
6
7
8
inline char * Chaine::operator += (const Chaine &P) {
	return strcat(ch,P.ch);
}
 
inline Chaine & Chaine::operator += (const Chaine &P) {
	strcat(ch,P.ch);
	return *this;
}

Comme vous l'aurez deviné, j'ai une classe Chaine qui a pour membre privé : char *ch;

Merci à vous.

[Luc Hermitte]

2e

[Raiden1234]

Pourquoi la deuxième ? C'est marrant j'aurais plutôt dit la première elle me semble plus simple

[Mat007]

Salut,

Pourquoi la deuxième ? C'est marrant j'aurais plutôt dit la première elle me semble plus simple

Parce qu'un operator+= doit toujours retourner *this.

MAT.

[HanLee]

Salut,

Parce qu'un operator+= doit toujours retourner *this.

MAT.

Ce n'est pas qu'elle doit, c'est juste une pratique pour permettre de chainer les opérations.

[Mat007]

Ce n'est pas qu'elle doit, c'est juste une pratique pour permettre de chainer les opérations.

Oui enfin effectivement j'aurais dû élaborer un peu...
Il est très fortement conseillé de faire en sorte que le comportement suive celui des types de base, comme dans "c'est considéré comme la bonne pratique".
En général dans les documents de conventions de code c'est un must (et non un should, can, might, ..)

Le compilateur ne va en effet pas râler, si c'est ce que tu voulais dire, par contre le collègue qui est assis à côté va peut-être tomber de sa chaise.

MAT.

[Luc Hermitte]

Et puis c'est pas comme si il y avait une rupture d'encapsulation en renvoyant l'adresse d'un buffer temporaire et complètement interne ...

[Médinoc]

Et au passage, avant de faire le strcat, tu devrais vérifier que le buffer est assez grand pour ça...

[corrector]

Et au passage, avant de faire le strcat, tu devrais vérifier que le buffer est assez grand pour ça...

Et comme tu as vérifié avant et que donc tu connais les longueurs des chaines, tu n'as plus besoin de strcat et tu peux utiliser memcpy.

En général, quand je vois une fonction de copie de <string.h> (strcpy, strcat, strncpy , strncat), je soupçonne un non-contrôle des débordements, ou un codage paresseux, traditionnel, conformiste (conformisme : tu as vu quelqu'un faire comme ça, tu fais pareil, le quelqu'un avait vu quelqu'un d'autre faire comme ça...).

[Raiden1234]

Ben en fait j'utilise strcat car c'était des fonctions que l'on nous conseillaient d'utiliser dans ce travail.

Pour ce qui est du buffer qui doit être assez grand, je gère une allocation de la longueur de la chaine en question dans mon constructeur de recopie profonde, mais une fois de plus je ne maitrise pas bien ces concepts vu que je débute là dedans.

En tout cas merci pour vos réponses elles m'ont bien aidé

[corrector]

Ben en fait j'utilise strcat car c'était des fonctions que l'on nous conseillaient d'utiliser dans ce travail.

On : qui ça?

Pour ce qui est du buffer qui doit être assez grand, je gère une allocation de la longueur de la chaine en question dans mon constructeur de recopie profonde, mais une fois de plus je ne maitrise pas bien ces concepts vu que je débute là dedans.

Si ce n'est pas pour apprendre à faire, mais pour utiliser réellement : std::string!

[Raiden1234]

On = Rédigeurs du Sujet

Si ce n'est pas pour apprendre à faire, mais pour utiliser réellement : std::string!

Alors là j'ai rien compris à ce que tu voulais dire

[Davidbrcz]

Il voulait dire que dans un contexte de productionon n'utilise ni ta classe maison,ni des char[] mais des std::string.

[corrector]

Il voulait dire que dans un contexte de production on n'utilise ni ta classe maison,ni des char[] mais des std::string.

Merci.