C++

pb àl'execution car prog trop gros pour DEV-CPP

Recherche :

Mot : Pseudo : Filtrer
Page : 1 2 Page Suivante Page Précédente Bas de page
Auteur	Sujet : pb àl'execution car prog trop gros pour DEV-CPP

bjone

Insert booze to continue

Reprise du message précédent :
oki, c'est pas ça mais:
1) si tu as des pointeurs dans ta classe
2) et que dans le destructeur tu détruits des objets pointés par les pointeurs membres
3) et que tu n'as pas implémenté l' operator = et le constructeur de recopie

=> tu crasheras probablement lors d'un push_back() ou autre opération sur un container de la STL.

Message édité par bjone le 02-02-2005 à 16:44:39

Publicité

bjone

Insert booze to continue

le problème du C++, c'est que pour que -ça marche- il faut que tout soit implémenté d'un coup, sinon tôt ou tard, ça parts en vrille (enfin pas forcément mais avec la STL si).

Message édité par bjone le 02-02-2005 à 16:44:55

bjone

Insert booze to continue

bon alors pour te faire un topo:

soit:

Code :

#include <string>
using namespace std:
class float_nomme
{
public:
string nom;
float valeur;
....
....
};
class etat
{
public:
float_nomme *pression;
....
....
etat()
{
pression=new float_nomme;
}
~etat()
{
if( pression )
delete pression;
}
};

si tu as uniquement -ça-, au push_back() d'une instance de "etat" ou autre opération, ça crashera.

porqué ? les diverses opérations vont à un moment vouloir copier ou assigner un "etat" à un autre ou sur un autre.

imaginons:

etat A, B;

A et B ont bien un "float_nomme" nommé de crée et maintenu par le pointeur "pression".

Code :

int main()
{
etat A,B;
A.pression->value=10; // OK
B.pression->nom="diamètre"; // OK
}

jusque là ça va, mais à l'assignement si il n'y a pas d'opérateur d'assignement tout est -copié- (assigné) par défaut.

donc mettons que A a "pression" qui pointe sur 0x08045646 et B a son "pression" qui pointe sur 0x0564564.

après un:
A=B;

A et B auront "pression" qui pointent tous deux sur le "float_nomme" à 0x0564564.
conséquence =>
1) une modification a travers A ou à travers B atteint le même "float_nomme".
2) l'ancien "float_nomme" de A est perdu dans la nature.
3) à la destruction de A et B, le même "float_nomme" à 0x0564564 sera détruit deux fois.
=> KABOOOM

pour cela il faut implémenter:
1) le constructeur de recopie
2) l'opérateur d'assignement

1) le premier sert à créer un objet pas encore initialisé à partir d'un autre
2) le deuxième sert à écraser un objét déjà initialisé à partir d'un autre

donc le modèle général est:

Code :

class etat
{
public:
float_nomme *pression;
....
....
etat()
{
pression=new float_nomme;
}
// constructeur de recopie
etat( const etat &source )
{
// à ce moment précis, pression est dans le vent (potentiellement
// alétoire)
pression= new float_nomme( *source.pression );
// maintenant pression est -valide-
}
etat &opertor = (const etat &source )
{
// protection anti-: "A=A";
if( &source != this )
{
// suppression de l'ancien
if( pression )
delete pression;
// copie du float_nomme
pression=new float_nomme( *source.pression );
}
return *this;
}
~etat()
{
if( pression )
delete pression;
}
};

voilà si je délires pas trop.

les "if( pression )" sont pas critiques ici, mais ça peut être une bonnée idée de le pratiquée si tu peux partir avec le pointeur "pression" à NULL (pas de float_nomme lié).

dans l'assignement, "pression=new float_nomme( *source.pression )" est un choix, on peut aussi penser à faire un (plus judicieux après réflexion, on assigne aussi les sous objets):
*pression = *session.pression;
pour écraser le "float_nomme" courant par celui de la source, au lieu de supprimer le courant et de recopier celui de la source.
(le comportement est pas forcément le même, attention)

en espérant ne pas avoir dit de conneries (question de principe)

Message édité par bjone le 02-02-2005 à 18:14:28

xterminhate

Si vis pacem, para bellum.

Ca serait p-e plus judicieux d'exploiter un pointeur intelligent pour regler son problème...

---------------
Cordialement, Xterm-in'Hate...

Mackila

if ( truc ) delete truc ;

ca sert à rien, autant écrire directement

delete truc ;

Mr Mala

Mackila a écrit :

if ( truc ) delete truc ;

ca sert à rien, autant écrire directement

delete truc ;

bjone a écrit :

les "if( pression )" sont pas critiques ici, mais ça peut être une bonnée idée de le pratiquer si tu peux partir avec le pointeur "pression" à NULL (pas de float_nomme lié).

Donc bjone se plante ou tu n'as pas lu son post entièrement ? :??:

Tarabiscote

Moi je suis d'accord avec Mackila ca sert à rien car lorsque l'on fait delete c'est déjà prévu de ne rien faire lorsque le pointeur est NULL.

bjone

Insert booze to continue

je sais plus, sûr ?

bjone

Insert booze to continue

oui merde, j'avais oublié. autant pour moi.

HelloWorld

Salut tout le monde!

Les pointeurs intelligents deviennent utiles dès que y'a un deuxième new dans le constructeur, car si y'a eu un new ok et que les autres echouent hop fuite de mémoire (exception dans le ctor => destructeur pas appelé).

Un bouquin c'est bien, mais je te conseille aussi de lire 2 ou 3 FAQ. C'est hallucinant le temps que tu gagnes à commencer par étudier une FAQ quand tu te mets à un langage.

---------------
FAQ fclc++ - FAQ C++ - C++ FAQ Lite

Publicité

++fab

victime du syndrome IH

HelloWorld a écrit :

On peut aussi utiliser new(nothrow), et tester si le pointeur est nul en cas d'échec.
Comme pointeur intelligent, le auto_ptr<> doit pouvoir suffire.

HelloWorld

Salut tout le monde!

Et tu fais quoi si le new(nothrow) échoue ? Tu lances ti même bad_alloc plus tard
http://www.gotw.ca/publications/mill16.htm
auto_ptr il est bien pour ça oui, sauf qi tu as alloué un tableau (new []).

---------------
FAQ fclc++ - FAQ C++ - C++ FAQ Lite

++fab

victime du syndrome IH

HelloWorld a écrit :

ba tu libère tout à la mimine
ça revient au meme que si tu faisais un bloc try{...} catch(bad_alloc) pour chaque new.
Et non pas un pour tous, genre

Code :

try
{
a = new Foo;
b = new Bar;
}
catch(bad_alloc)
{
//bien emmerdé pour deleter ... ->pointeur à 0, et compter sur delete
}

Message édité par ++fab le 02-02-2005 à 23:05:25

smag

++fab a écrit :

ba tu libère tout à la mimine
ça revient au meme que si tu faisais un bloc try{...} catch(bad_alloc) pour chaque new.
Et non pas un pour tous, genre

Code :

try
{
a = new Foo;
b = new Bar;
}
catch(bad_alloc)
{
//bien emmerdé pour deleter ... ->pointeur à 0, et compter sur delete
}

ça c le dernier chapitre de mon bouquin!!

Que me conseillez vous pour continuer de découvrir ce langage?
des titres de bouquins? des exos? des sites? des tutoriels?

++fab

victime du syndrome IH

smag a écrit :

ça c le dernier chapitre de mon bouquin!!

Que me conseillez vous pour continuer de découvrir ce langage?
des titres de bouquins? des exos? des sites? des tutoriels?

je te conseille "le petit stroustrup illustré"

HelloWorld

Salut tout le monde!

++fab a écrit :

ba tu libère tout à la mimine
ça revient au meme que si tu faisais un bloc try{...} catch(bad_alloc) pour chaque new.
Et non pas un pour tous, genre

Code :

try
{
a = new Foo;
b = new Bar;
}
catch(bad_alloc)
{
//bien emmerdé pour deleter ... ->pointeur à 0, et compter sur delete
}

C'est pas de ça que je voulais parler, mais du fait que après avoir rendu le code exception safe, il fallait bien relancer l'exception pour indiquer l'echec du ctor. Dans le cas d'un try...catch tu relances avec throw;, avec new(nothrow) c'est toi qui fait le throw... Je trouve ça moyen de dire
"Toi fait cette opération sans lever d'exception, si tu échoues, je lève l'exception que tu aurais du lever"
Surtout que y'a pas que bad_alloc de possible, le ctor de l'objet alloué peut très bien lever autre chose, et malgré ton nothrow tu te prend un exception dans la tête.

Code :

class Test
{
public:
Test()
{
throw "dtc";
}
};
int main()
{
try
{
Test * t = new(nothrow) Test;
}
catch ( const char * e )
{
cerr << e << '\n';
}
}

Typiquement avec try...catch c'est lourd oui

Code :

Test::Test():
a(0), b(0)
{
try
{
a = new Foo;
b = new Bar;
}
catch ( ... )
{
delete b;
delete a;
throw; // relancer
}
}
Test::~Test()
{
delete b;
delete a;
}

et donc vive les smart ptr, ça devient tout propre et simple

Code :

Test::Test()
{
a = std::auto_ptr<Foo>( new Foo );
b = std::auto_ptr<Bar>( new Bar );
}
// plus besoin du destructeur

Ce principe c'est le RAII, l'essayer c'est l'adopter :love:

---------------
FAQ fclc++ - FAQ C++ - C++ FAQ Lite

Taz

bisounours-codeur

c'est le principe du bloc try_catch fonctionnel

Harkonnen

Modérateur
Un modo pour les bannir tous

pour la définition du constructeur de copie, y'a un topic
http://forum.hardware.fr/forum2.ph [...] 0&subcat=0

Publicité

Page : 1 2

Page Suivante

Page Précédente

Haut de page

FORUM HardWare.fr

Programmation

C++

pb àl'execution car prog trop gros pour DEV-CPP

Sujets relatifs
Comparatif sur l'interet des languages de prog	Dev-Cpp
changer de user durant l'execution d'un script	[Débutant] pb execution java script
Temps d'éxécution comparable?	temps d'exécution en millisecondes?
cryptage en CPP	probleme dev-C++
verification prog C++
Plus de sujets relatifs à : pb àl'execution car prog trop gros pour DEV-CPP

Page générée en 0.293 secondes