Discussion :

[oodini]

Hello,

J'ai une fonction de gestion d'exception qui se présente comme suit :

Code :

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

1
2
3
4
5
6
const char *monException::what(void) const
{
    std::string res;
    (...)
    return res.c_str();
}

Elle est appelée par :

Code :

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

1
2
3
4
catch (const monException &monException)
{
    const char *zob = monException.what();
}

Le problème est que ce que je mets comme chaîne de caractères dans la première fonction est perdu lorsqu'elle retourne son résultat.
L'adresse est bien la même, mais les octets qui se trouvent derrière semblent réinitialisés...

Merci pour votre aide.

[Laurent Gomila]

Tu ne devrais pas utiliser std::string dans ta fonction membre what() ; en effet celle-ci ne doit pas pouvoir lever d'exception (ce que peut faire ton code utilisant std::string). D'ailleurs un compilo bien paramétré t'indiquerait que ton prototype de fonction ne correspond pas à celui de la classe mère, il devrait être le suivant :

Code :

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

const char *monException::what(void) const throw()

De même pour le destructeur, d'ailleurs.

Bon ça c'est pour la théorie, en pratique si ta fonction what ne lève pas d'exception (ce qui devrait être le cas la plupart du temps quand même) ça devrait fonctionner.

Ton problème pourrait venir du fait que tu stockes le résultat de what(), qui est lui-même le résultat d'un c_str(), pour le réutiliser plus tard. Or c_str() renvoie une chaîne potentiellement temporaire, càd qui n'existe plus après l'appel.

[oodini]

Hello,

1. Pourquoi ma fonction what() ne devrait-elle pas pouvoir lever d'exception ?

2. Pourquoi le fait d'utiliser std::string est-il susceptible de lever une exception ? (j'en ai besoin, car je dis concaténer des chaînes de caractères)

3; Ce que me renvoie mon what() n'est normalement pas stocké. C'est utilisé comme paramètre du constructeur d'une classe. Et le résultat semble être identique.

[Laurent Gomila]

1. Parce qu'elle est définie comme ça dans la classe std::exception (cf. "throw()"). Il paraît logique qu'une fonction appelée après réception d'une exception n'ait elle-même pas le droit d'en lever une nouvelle.

2. Un exemple tout simple : lors de tes concaténations tu dois allouer un nouvel espace mémoire, ce qui peut lever un std::bad_alloc en cas d'échec.

3. Ah... Dans ce cas, je sèche.

[oodini]

Merci pour les infos, en effet pleines de sens. :-)

Dommage que mon problème prinicpal ne soit toutefois pas résolu. :-|

[Mat007]

Salut,

Tu retournes un pointeur sur quelque chose alloué sur la pile et donc plus valide à la sortie de what()...

edit : et donc il peut se passer potentiellement n'importe quoi, essaye de mettre 'res' en static pour voir ?


MAT.

[Sylvain Togni]

Code :

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

1
2
3
4
5
6
const char *monException::what(void) const
{
    std::string res;
    (...)
    return res.c_str();
}

Ce n'est pas correct car l'objet res est détruit avant de sortir de la fonction, donc la valeur de res.c_str() ne peut pas être utilisée en dehors de la fonction.

Une solution est par exemple de définir res comme membre de la classe.

[oodini]

Effectivement !

Merci beaucoup pour ce coup de pouce !

[oodini]

Ca semblait une bonne idée, mais en fait, je ne peux pas modifier une donnée membre, puisue ma fonction est const. Et je suis obligé de la définir comme telle puisque je l'hérite de la classe exception.... :-/

[Laurent Gomila]

Pourquoi construis-tu ton message dans what() et non dans le constructeur ?

[oodini]

Hmmm...

C'est une piste intéressante. Je m'en vais essayer.

[oodini]

OK, j'ai réussi !

Merci !