Très mauvais d'implémenter tes fonctions dans ton .h : par convention, on fait ça dans un fichier .cpp à part (StrIZI.cpp par exemple) qui inclue le header. Ca permet de mieux contrôler la compilation de ton code.
Autre chose, tu a oublié dans ton header les directives :
#ifndef __STRIZI_H__
#define __STRIZI_H__
//Le code
#endif
Ces directives sont TRES importantes : elles protègent des erreurs (ou warning selon les compilateurs) de "multiple symbol redefinition", et des fatal error d'inclusions infinies, comme par exemple :
//StrIZI.h
#include "fff.h"
//fff.h
#include "StrIZI.h"
On se retrouve dans une boucle infinie pour le compilateur --> Erreur de compilation.
Dans ta fonction Val, pourquoi n'utilise tu pas la fonction atoi si le système est égal à 10 ?
A la fin :
if (Result==StrNull)Result="0";
return Result;
Pourquoi créer une chaine rien que pour une comparaison ? Je te conseillerai de le remplacer par ça :
if(!strcmp(Result.Chaine, "")) Result = "0"
return Result;
Bon sinon, la classe m'a l'air simple d'utilisation, mais il aurait fallu rajouter une fonction du genre sprintf pour mettre des nombres et tout ça. Bon, ca vaut bien un 7 on va dire.
- Messages postés
-
746
- Date d'inscription
- vendredi 17 juin 2005
- Statut
- Membre
- Dernière intervention
- 23 mai 2007
9- Messages postés
-
2023
- Date d'inscription
- mardi 24 septembre 2002
- Statut
- Membre
- Dernière intervention
- 28 juillet 2008
5