C++

fonction qui retourne un pointeur

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : fonction qui retourne un pointeur

weblook$$

Bonjour,

j'ai une fonction

COb * f();

dans mon code je fais ça :

COb * o = f();

Pourquoi ne dois-je pas faire de delete ?

Merci

Publicité

IrmatDen

Huh? Je te retourne la question?
Cet objet a peut-être un parent/propriétaire qui se chargera de le détruire lors de la destruction du parent? Sans plus de contexte... Il faut faire un delete.
Mais avec plus de contexte, peut-être pas (bien que ça ne devrait pas mettre à mal le système, mais être un raccourci pour le développeur).

Message cité 1 fois

++fab

victime du syndrome IH

Il faut lire la documentation de la fonction. En tout cas, je trouve que c'est une mauvaise chose que de retourner un pointeur s'il est interdit de le désallouer. En renvoyant alors une référence, on aurait surement éviter quelques erreurs.

weblook$$

Et en renvoyant une référence, que ce passe t-il exactement ?

un objet est déclaré à l'intérieur de la fonction, mis sur la pile, puis une référence à cet objet est renvoyé? pourquoi cette référence resterait-elle valable une fois que la fonction est terminé ? j'aurais justement tendance à penser que cette référence n'est plus valable.

PS: ça fait deux ans que je n'ai pas codé en C++, j'ai oublié certain principe de base.

Message cité 1 fois
Message édité par weblook$$ le 06-04-2007 à 09:19:23

Joel F

Real men use unique_ptr

boost::shared_ptr et ses confrères me semblent de bon [:aloy2]
http://boost.org/libs/smart_ptr/smart_ptr.htm

ca evitera de se posser cette sempiternelle question de delete ou pas

++fab

victime du syndrome IH

weblook$$ a écrit :

un objet est déclaré à l'intérieur de la fonction, mis sur la pile, puis une référence à cet objet est renvoyé? pourquoi cette référence resterait-elle valable une fois que la fonction est terminé ? j'aurais justement tendance à penser que cette référence n'est plus valable.

Effectivement, dans ce cas, elle n'est plus valable.
Si la variable était static, la référence serait valide.
Mais ceci n'est pas spécifique aux références, les pointeurs se comportent de manière équivalente.

weblook$$

un new à l'intérieur d'une fonction alloue sur de la mémoire sur le tas pourtant, donc reste valide une fois la fonction terminée.

weblook$$

IrmatDen a écrit :

hum oui probable, c'est un toolkit que j'utilise donc je sais pas trop comment c'est gauler ,( je sais y a la doc mais bon...)

weblook$$

Code :

class A
{
protected:
A* child;
virtual A* creerEnfantUnique() = 0;
};
class B : public A
{
public:
~B();
inline A* creerEnfantUnique()
{
child = new B();
return child;
}
};
B::~B()
{
if(child)
{
delete child;
}
}
void main()
{
B * parent = new B();
A* enfant = parent->creerEnfantUnique();
delete parent;
}

effectivement , avec ce genre de code , pas de delete dans le main pour le pointeur retourné

Message édité par weblook$$ le 07-04-2007 à 22:43:24

KangOl

Profil : pointeur

faut pas retourner un pointeur mais une référence dans ce cas

Publicité

weblook$$

Non car je me retrouverais avec
A enfant
et le compilo broncherait à cause du virtual pure

++fab

victime du syndrome IH

normal, tu mens au compilateur, il se venge.
edit: (je répondais à un post effacé, pas grave)
edit2: arrête d'éditer test posts en permanence, on a pas que ça à faire de relire chaque fois tous les posts depuis le début et voir ce qui change.

Message édité par ++fab le 07-04-2007 à 22:47:02

KangOl

Profil : pointeur

et d'un autre coté, ton code est foireux :
*/ child n'est pas initialisé
*/ deux appels a creerEnfantUnique et, hop, un memoryleak

++fab

victime du syndrome IH

Oui, et la class A est mon avis inutilisable - ou alors c'est un design plus complexe dont on en voit qu'un bout ici.
Et le test avant le delete est inutile.

weblook$$

Code :

class A
{
protected:
A* child;
virtual A* creerEnfantUnique() = 0;
};
class B : public A
{
private:
bool pourTeFairePlaisir;
public:
B();
~B();
inline A* creerEnfantUnique()
{
if(pourTeFairePlaisir)
{
child = new B();
pourTeFairePlaisir=false;
}
return child;
}
};
B::B()
{
pourTeFairePlaisir = true;
child = NULL;
}
B::~B()
{
if(child)
{
std::cout<<"I will be back";
delete child;
}
}
void main()
{
B * parent = new B();
A * enfant = parent->creerEnfantUnique();
delete parent;
}

Message édité par weblook$$ le 07-04-2007 à 22:55:25

weblook$$

désolé pour les "édit" mais ça évite de laisser des erreurs inutiles

FORUM HardWare.fr

Programmation

C++

fonction qui retourne un pointeur

Sujets relatifs
pointeur et JAVA	[VBS] [résolu]Pb avec fonction STRCOMP
appliquer une fonction js a toutes balises input	Fonction mysql_fetch_assoc
fonction avec table hash en parametre	Erreur d'accès ambigu à une fonction membre après dérivation multiple
comment utiliser cette fonction	Fonction pour écrire un DWORD dans la base de registre avec VB6
Problemme sur fonction [debutant]	fonction setInterval
Plus de sujets relatifs à : fonction qui retourne un pointeur

Page générée en 0.082 secondes