Discussion :

[[Hugo]]

Bonjour,

J'ai un pitit problème : j'aimerais créer localement un objet avec un constructeur ou l'autre selon l'état d'un booléen (ici steady_state_used) mais lorsque j'utilise la structure ci-dessous :

Code :

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

1
2
if(!steady_state_used) freevol_etasta freevol_dlg(this, &d_transf);
else freevol_etasta freevol_dlg(this, &d_transf, &deps1, &deps2, &dsig1, &dsig2);

mon compilo me jette en disant : "freevol_dlg undeclared, first use of this function" : est-il possible de créer un objet dans un if ou un else ???

Merci

[bogoss91]

Si je me souviens bien, quand tu crée quelque chose dans un bloc (une variable, une instance, etc.), il est détruit à la fin du bloc, donc ici à la fin du if-else.

[[Hugo]]

argh ! tu as raison, il me semble bien... merci Je vais contourner le problème de façon pas trop élégante, mais tant pis

[koala01]

Salut,

Cela s'appelle la "portée"...

Le problème est encore pire, car, le code "idéalement écrit" est

Code :

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

1
2
3
4
5
6
7
8
if(!steady_state_used) 
{
    freevol_etasta freevol_dlg(this, &d_transf);
}
else 
{
    freevol_etasta freevol_dlg(this, &d_transf, &deps1, &deps2, &dsig1, &dsig2);
}

et là, on se rend compte qu'il y a deux blocs séparés: un pour le "if" et un pour le "else"

Comme on "sort" du bloc (et donc de la porté du if) dés qu'on atteint l'accolade fermante avant le else, et qu'on sort de celle du else dés qu'on atteint l'accolade fermante, ce code n'a, en définitive que l'effet de créer un objet et de le détruire aussi sec

[NiamorH]

Si tu dois construire l'objet pour le renvoyer et t'en servir dans ton application, utilise new.

Si l'objet n'est destiné qu'à être utiliser dans cette fonction, utilise new avec std::auto_ptr.

[koala01]

Une autre solution est, peut etre, de passer par l'opérateur ternaire:

Code :

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

1
2
3
4
freevol_etasta freevol_dlg(this,
    ( steady_state_used ?
        &d_transf, &deps1, &deps2, &dsig1, &dsig2 :
        &d_transf));

(solution non testée)

*Normalement* la déclaration de freevol_dlg étant dans la portée "générale" de la fonction, cela devrait pouvoir passer.

Evidemment, cela nécessitera peut etre un petit commentaire (car, du point de vue de la lisibilité )