CLASSE CSTRING

magic_Nono

Messages postés: 1878
Date d'inscription: jeudi 16 octobre 2003
Statut: Membre
Dernière intervention: 16 mars 2011

- 2 sept. 2004 à 15:01
Soilwork9

Messages postés: 16
Date d'inscription: lundi 9 août 2004
Statut: Membre
Dernière intervention: 13 septembre 2004

- 13 sept. 2004 à 08:28

Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/25876-classe-cstring

Soilwork9

Messages postés: 16
Date d'inscription: lundi 9 août 2004
Statut: Membre
Dernière intervention: 13 septembre 2004

13 sept. 2004 à 08:28

Voila, j'ai fait la modification sur ma CString pour utiliser les headers C++ ;)

cs_djl

Messages postés: 3011
Date d'inscription: jeudi 26 septembre 2002
Statut: Membre
Dernière intervention: 27 novembre 2004

8
7 sept. 2004 à 23:16

"alors pourquoi avoir créé iostream.h, pour ensuite dire que c'est deprecated?"

parce que ca n'est pas standard ! iostream.h ca date de l'epoque du c++ pre standard (avant 97), c'est deprecié et ca va disparaitre au prochain standard

maintenant tout est dans le namespace standard, meme lib standard c

Soilwork9

Messages postés: 16
Date d'inscription: lundi 9 août 2004
Statut: Membre
Dernière intervention: 13 septembre 2004

7 sept. 2004 à 22:44

Wow, y a du message ici !
Juste un petit mot au sujet des librairies "C" qui sont deprecated.. J'ai juste a remplacer iostream.h par iostream, et les librairies C classiques par leurs equivalents C++ comme je l'avais dit.. Ca change franchement pas grand chose, d'ailleurs c'est deprecated parce qu'ils ont envie que ca soit deprecated... D'ailleurs, est-ce que iostream existe en C? Je dirais que non, alors pourquoi avoir créé iostream.h, pour ensuite dire que c'est deprecated? :)

Bon en tout cas quand j'aurai le temps, je mettrai mes libs à jour pour prendre en compte tes remarques, djl, ca ne me coute absolument rien de toute facon! Sauf cinq minutes de mon temps pour changer quelques include ;)

++
Soilwork
PS : J'avais pas regardé qui était l'auteur du document. Si c'est le createur du C++, alors ok, il a le droit de faire le boss lol! Je retire ce que j'ai dit sur l'aspect pédant du texte :)

magic_Nono

Messages postés: 1878
Date d'inscription: jeudi 16 octobre 2003
Statut: Membre
Dernière intervention: 16 mars 2011

1
7 sept. 2004 à 15:39

en fait les CString étaient décrétés prioritaires sur mes BString sans causé de conflit

un bug amené par les modfis récentes : const:

les CString prenaient en charge la comparaison avec un caractere =>

résolu en rajoutant ceci ds le header
//deb: rajouté pr régler PB avec les MFC
du coup, pr résoudre ce pb, G inclu ds Visual uniquement les lignes suivantes ds le header.

bool operator==(const char s)const{return operator==((BString) s);}

par ex pr cet opérateur: le suivant était choisi
(syntaxe ptet approximative)

friend bool operator(const char& ,const CString&)const

vu que rien ne laissait présagé de cette erreur... ben détecté il y a une semaine seulement...

y avait une différence ! la comparaison de la casse !
voilu
++
Nono.

cs_djl

Messages postés: 3011
Date d'inscription: jeudi 26 septembre 2002
Statut: Membre
Dernière intervention: 27 novembre 2004

8
7 sept. 2004 à 15:30

ouai je viens de voir ton exemple

> tout ce qui est inliner devrais pas poser de probleme

je repete, ce n'est pas un conseil que je donne, je dis juste que tes fonction inline doivent etre definie dans un header (si tu veut qu'elles soit sytematiquement inliner)

c'etait quoi le bug avec les bstring ?

magic_Nono

Messages postés: 1878
Date d'inscription: jeudi 16 octobre 2003
Statut: Membre
Dernière intervention: 16 mars 2011

1
7 sept. 2004 à 14:25

mmm

G viré les .obj

recompilé:
et tjs des erreurs
211 error(s)

dll.obj : error LNK2005: "public: static class BString __cdecl BIBLI_BC::BLang::ELanguesToString(enum BIBLI_BC::ELangues *)" (?ELanguesToString@BLang@BIBLI_BC@@SA?AVBString@@PAW4ELangues@2@@Z) already defined in dgestionliste.obj
dll.obj : error LNK2005: "public: static class BString __cdecl BIBLI_BC::BLang::getTxtLangue(char const *,bool)" (?getTxtLangue@BLang@BIBLI_BC@@SA?AVBString@@PBD_N@Z) already defined in dgestionliste.obj
dll.obj : error LNK2005: "public: static void __cdecl BIBLI_BC::BLang::initTitre(class CDialog *,char const *)" (?initTitre@BLang@BIBLI_BC@@SAXPAVCDialog@@PBD@Z) already defined in dgestionliste.obj
dll.obj : error LNK2005: "enum BIBLI_BC::ELangues BIBLI_BC::iLangageTrtt" (?iLangageTrtt@BIBLI_BC@@3W4ELangues@1@A) already defined in dgestionliste.obj
dll.obj : error LNK2005: "enum BIBLI_BC::ELangues BIBLI_BC::iLangageLogiciel" (?iLangageLogiciel@BIBLI_BC@@3W4ELangues@1@A) already defined in dgestionliste.obj
dll.obj : error LNK2005: "class BHashTable BIBLI_BC::donneesLinguistiquesSoft" (?donneesLinguistiquesSoft@BIBLI_BC@@3VBHashTable@@A) already defined in dgestionliste.obj
dll.obj : error LNK2005: "class BHashTable BIBLI_BC::donneesLinguistiquesTrtt" (?donneesLinguistiquesTrtt@BIBLI_BC@@3VBHashTable@@A) already defined in dgestionliste.obj

pour un ptt apperçu
++

magic_Nono

Messages postés: 1878
Date d'inscription: jeudi 16 octobre 2003
Statut: Membre
Dernière intervention: 16 mars 2011

1
7 sept. 2004 à 13:12

justement djl, si tout est ds le .h, une question se pose !!!

comment résouds tu les inclusions multiples

mettons que j'ai une classe A, ayant 2 membres b et c de classe B et C

soit la classe truc, declarée entierement ds le header

dans ma classe B & ds ma classe C, j'ai des instances de Truc
donc les headers B et C contiennent un include sur Truc mais pas A

là, le compilo va raller car il va trouver deux sortes de Truc, ceux de B & ceux de C
et ces deux Trucs ont été précompilés dans les b.o et c.o

j'espere que vs suivez tjs

sinon, je vais illustrer cet exemple:

Bref, voici l'erreur générée par VC6 & pkoi j'ai abandonné cette idée de tt mettre ds les headers

error LNK2005

detail d'un essai avec MP:

acceleration : nouvelle politique : toutes les bibli sont des headers => inline automatisés.... [ABANDONNE]
=> ne plus inclure les bibli ds les prj...
ABANDONNE car :
> si certains srce ne st pas ds des cpp > compilés plsrs fois
ex=> HashTable.obj : error LNK2005: "class BString __cdecl BIBLI_BC::ELangageDeProgToString(enum ELangageDeProg const &)" (?ELangageDeProgToString@BIBLI_BC@@YA?AVBString@@ABW4ELangageDeProg@@@Z) already defined in AnaTrace.obj
=> Debug/MetaProg.exe : fatal error LNK1169: one or more multiply defined symbols found

Soilwork9

Messages postés: 16
Date d'inscription: lundi 9 août 2004
Statut: Membre
Dernière intervention: 13 septembre 2004

6 sept. 2004 à 14:21

Salut et bienvenue dans le cercle des createurs de classes gérant des chaines de caractères ;)

Bon, je vois que djl et Nono sont arrivés avant moi pour examiner ton code source, donc je ne vais pas spécialement m'attarder sur celui-ci (mais un peu quand meme ;) ), d'autant plus que la solution que tu as choisie ressemble un peu à celle que j'ai utilisée pour ma propre classe CString, donc je ne vais pas critiquer! lol

Quelques petits détails pour commencer... D'abord, il vaut mieux zipper ton code source et l'uploader sur le site, si tu peux faire tout ca, plutôt que de le mettre directement en texte sur la page, c'est beaucoup plus pratique, pour ceux qui veulent l'utiliser, je trouve.
Ensuite, il vaut mieux séparer ton code en deux parties, un header avec la déclaration de ta classe et de ses méthodes (fichier H ou HPP), et un fichier source CPP avec le corps des méthodes. C'est plus pratique et plus lisible.

Deux petites choses concernant le code, au passage: D'abord, n'oublie pas de mettre les include nécessaires à ta classe (en l'occurence, stdlib et string).

Deuxième chose : dans le constructeur qui prend les N premiers caractères d'une autre chaine, tu marques ceci:

for(unsigned long i=0; i < lengthfromstart ;i++)
this->_pstr[i]=chaine[i];
this->_pstr[i] = 0;

Tu déclares la variable i à l'intérieur du FOR, donc sa portée se limite exclusivement à l'intérieur de cette boucle, qui ne contient qu'une seule instruction. Cela signifie qu'à la sortie de la boucle, pour l'instruction:
this->_pstr[i] = 0;
la variable i n'existe plus! A la place, tu dois écrire:
unsigned long i;
for(i=0; i < lengthfromstart ;i++)
this->_pstr[i]=chaine[i];
this->_pstr[i] = 0;

Et la, ca compile :)

Voila, c'est tout pour l'instant, je crois. Bravo en tout cas, c'est pas mal du tout, ca compile sans problèmes ni Warnings (après les deux petites corrections mineures) et tout a l'air de fonctionner, d'après les quelques petits tests que j'ai faits. Well done!

Si tu veux l'améliorer, tu peux ajouter des méthodes de remplacement, qui sont toujours très pratiques!

Bonne continuation!
++
Soilwork

Afficher les 33 commentaires