Discussion :

[hakurou]

bonjour,

alors j'ai un trou et je n'arrive pas a comprendre ce qui se passe, je m'explique

je fais une classe toute bête

Dans le .h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
class Toto
{
 
    public:
 
    Toto(); 
    void choixTitre(const char *titre);
    ~Toto();
 
 
    private:
 
    char V_titre[30];
};

Le but est de recuperer le cons char *titre et de le mettre dans l'attribut char V_titre.
Etant donnée que l'on ne peut pas faire une simple assignation je fait donc de la facon suivante :

dans le .cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
//je zap l'initialisation
 
void Toto::choixTitre(const char *titre)
{
    int i = 0;
    while(*titre){
        V_titre[i] = *titre;
        titre++; i++;
    }
}

Ce code plante et je ne comprend pas pourquoi, mais quand je fais ceci, ca marche :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
//je zap l'initialisation
 
void Toto::choixTitre(const char *titre)
{
    char toto[30] = {0};
 
    int i = 0;
    while(*titre){
        toto[i] = *titre;
        titre++; i++;
    }
}

je trouve ce code identique, la seule difference est que ici, le char est declaré directement dans la methode et non dans la classe.


J'ai l'impression qu'un truc simple m'echappe! mais pourquoi donc ne puis je pas recuperer const char *titre dans le char V_titre ???

[Mat007]

Salut,

Soit tu as oublié de copier-coller la ligne int i = 0; dans ta première implémentation, soit il y a quelque part une autre variable i parasite...
Et dans tous les cas tu oublies de copier le '\0' terminal dans ta boucle.

Sinon en vrac :
. il ne faut pas utiliser de chaînes C en C++ mais plutôt des std::string
. tu mets les données en visibilité protected ce qui semble indiquer que ta classe est prévue pour être dérivée mais ton destructeur n'est pas virtuel
. en général on utilise plutôt ++i que i++ par défaut

MAT.

[hakurou]

oui effectivement dans l'exemple j'ai oublié de copier le int i = 0;

pour le '\0' n'est il pas copié en même temps a la fin de la boucle ? (il me semblais)

pour le ++i, je ne savais pas trop a quel moment il etait utile de l'utiliser ne connaissant pas bien les differences...

heu oui je me suis planté avec le "protected", cette classe n'est pas faite pour faire de l'heritage, donc c'est "private" qu'il faut mettre.

Pour le char c'est parce que j'utilise la library sdl qui a besoin d'un char pour certaines données

[Mat007]

pour le '\0' n'est il pas copié en même temps a la fin de la boucle ?

Et bien non puisque la boucle s'arrête dès que *titre vaut justement 0.

MAT.

[Pacorabanix]

Pour le char c'est parce que j'utilise la library sdl qui a besoin d'un char pour certaines données

avec un std::string ça va aussi très bien, il y a la fonction membre de string c_str() pour ça.

[hakurou]

J'ai changé pour du choixTitre(const string titre); et appliqué la methode c_str() et ca marche ! l'attribut en char recupere bien les données du string.

merci beaucoup !

[Mat007]

J'ai changé pour du choixTitre(const string titre);

Il ne reste plus qu'à passer par référence à la place :

Code :

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

choixTitre(const string& titre);

Et c'est parfait.

MAT.