C++

[Solved] Erreur d'acces memoire a l'execution

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [Solved] Erreur d'acces memoire a l'execution

agyspace

J'ai une petite question concernant les mysteres de la gestion memoire en C++.
Voici les classes A, B et BImpl:

Code :

class B
{
public:
virtual void printHello() = 0;
};
class BImpl : public B
{
public:
void printHello()
{
printf("Hello",%s);
}
};
class A
{
public:
~A()
{
delete(_myBImpl);
_myBImpl = NULL;
}
void setAttribute(B* myB)
{
_myBImpl = static_cast<BImpl*>(myB);
}
B &getAttribute()
{
return *_myBImpl;
}
private:
BImpl* _myBImpl;
};

Et ma fonction de test:

Code :

int main()
{
BImpl b;
A a;
b.printHello();
a.setAttribute(&b);
}

Ce test plante a l'execution. Apparemment un pb d'acces memoire.

Si par contre j'utilise la factory suivante:

Code :

class FactoryB
{
public:
FactoryB() {}
~FactoryB() {}
static B* createBInstance()
{
BImpl *bPtr = 0;
bPtr = new(BImpl);
return bPtr;
}
};

Et les test suivant:

Code :

int main()
{
BImpl *b = static_cast<BImpl*>(FactoryB::createBInstance());
A a;
b->printHello();
a.setAttribute(b);
}

Ca marche nickel! Quelqu'un pourrait-il me dire en detail ce qui se passe dans le premier cas? Pourquoi ca plante?

Merci d'avance

Message édité par agyspace le 13-04-2006 à 19:10:08

Publicité

agyspace

Petit indice que j'ai oublie! Le crash se fait quand on sort de la fonction, en particulier lors de la destruction de l'objet A, c'est l'instruction "delete(_myBImpl)" qui ne passe pas.

L'erreur se manifeste sous la forme d'une fenetre popup Visual Studio.

Et enfin pour ceux qui se demandent pourquoi je fais un cast dans la methode setSecurity() de A, c'est parce en vrai, je n'utilise pas des classes A, B et BImpl mais d'autres classes plus etoffees et que j'ai besoin de faire ce cast a ce moment la.

agyspace

Et je me suis trompee aussi sur l'instruction de delete. Il faut lire bien evidemment:
delete _myBImpl;
et non pas
delete(myBImpl);

Taz

bisounours-codeur

dans ton premier exemple, t'as un delete, mais pas de new, y a pas à chercher ...

sinon vire ces cast, ils ne servent à rien.

bPtr = new(BImpl); -> new BImpl;

nargy

ligne 12: %s?
ligne 27: fille=base? d'habitude c'est le contraire

dans le main:
ligne 3: ``b`` est mis sur la pile
ligne 6: ``a`` va supprimer ``b`` avec un delete (qui supprime sur le tas)
Si tu utilise delete avec un objet créé sur la pile tu obtient une violation mémoire.

agyspace

Bon, quelques mises au point s'imposent je crois :P:
Comme je l'ai dit les petits bouts de code que j'ai donne sont une simplification de mon vrai code.
Je me suis trompee sur la syntaxe du printf (que j'ai mis juste pour l'exemple mais en realite je ne fais pas de printf je fais un autre buisiness )

Code :

printfHello()
{
printf("%s \n","Hello" );
}

Ensuite, le cast dans l'exemple que j'ai donne est inutile c'est vrai. Dans mon vrai code j'utilise dans la classe A des methodes de BImpl non definis dans B, ce qui me force a faire un cast de B vers BImpl.

Pour repartir sur le fon du probleme:
Si je comprends bien un objet qui n'est pas cree avec new est cree sur la pile, et faire un delete dessus cree une violation memoire. C'est bien ca?

nargy

> C'est bien ca?
- tout à fait

> Dans mon vrai code j'utilise dans la classe A des methodes de BImpl non definis dans B, ce qui me force a faire un cast de B vers BImpl.
- ton vrai code est faux setAttribute() devrait prendre dans ce cas un BImpl* en paramètre.

agyspace

Et bien en fait, c'est un peu complique, mais je suis obligee d'utiliser le type B* en interface. Mon client ne doit pas connaitre (pour diverses raisons internes a ma societe) la classe BImpl. D'ou le cast...

nargy

Utilise le rtti() et un throw().

agyspace

le rtti()? je ne connais pas

Publicité

nargy

J'ai pas de doc particulière à te proposer, mais il y en a plein sur le net.
Chaque classe possède un numéro, qui identifie son type. Pour vérifier que le type d'une classe polymorphe est bien celui que tu attends, tu vérifie son rtti. Sinon, tu lance une exception. Comme ça le programmeur qui utilise ta librairie et qui mets en paramètre une classe incompatible est prévenu par un message d'erreur.
C'est néanmoins une méthode à n'utiliser qu'en dernier recours. Vérifie d'abord que tu ne peut pas réarranger plus judicieusement ton diagramme objet.

agyspace

Ok merci pour ton aide

FORUM HardWare.fr

Programmation

C++

[Solved] Erreur d'acces memoire a l'execution

Sujets relatifs
erreur (de débutant) dans un batch windows [Résolu]	erreur JXTA
Erreur de routine, quelqu'un peut me dire ou je boulette ?	[Mambo] - Site temporairement indisponible - Erreur 2
erreur sur strcmp	Erreur : Trop de champs définis
plantage lors de l'exécution (un seul cas de figure)	script méteo problème à l'execution
migrer base de donné acces	[c] Allocateur memoire
Plus de sujets relatifs à : [Solved] Erreur d'acces memoire a l'execution

Page générée en 0.033 secondes