CodeS-SourceS est hébergé par Frontier.fr

Ce site au démarrage

begin process at 2012 05 28 14:06:53

Accueil Codes Tutoriels Forum Livres Emploi Services Connexion

Trouver un code source :

dans

[ Dernières recherches ]

Accueil > Forum >

BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

Poser une question dans le forum ou lancer une discussion

BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

lundi 12 juillet 2010 à 16:38:13 | BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

eupeup

41 msg

54 msg CS

Salut à tous,

j'ai un soucis avec des pointeurs, des constructeurs et des destructeurs.... si quelqu'un peut me donner un tit coup de main...

Donc en gros, j'ai un première classe CEmetteur qui contient :
- un string
- un pointeur sur une structure Rating (qui contient trois arguments, deux string et un double)...
L'allocation de mémoire de ce pointeur sur Rating est réalisée dans le constructeur...
Je définis donc un destructeur contenant un simple delete _rating (qui est le nom de mon pointeur)

Ensuite j'ai une seconde classe CExpoEmetteur qui contient entre autres :
- quelques attributs privés de type double, string, ...
- un pointeur sur un objet CEmetteur
1 - le constructeur prend en argument un objet Emetteur, et j'y alloue de la mémoire, donc ca donne un truc du genre :
ExpoEmetteur(CEmetteur emetteur, autres arguments...) : _AutresArguments(autresArgument)
{
_emetteur = new CEmetteur(emetteur);
}
2 - J'ai ensuite créé un destructeur qui fait un delete de _emetteur...

Donc jusque là ca va, enfin je crois... mais j'ai le soucis suivant, dans une aute classe je manipule ces objets de la manière suivante :

CEmetteur emetteur(string argEmetteur, argStructRating1, argStructRating2, argStructRating3);
CExpoEmetteur *expoEmetteur = new CExpoEmetteur(emetteur, autres arguments...);

J'ai mis un pointeur, car je dois en créer un certain nombre que je place ensuite dans un vector.... enfin bref...

Donc mon soucis est le suivant, mon objet emetteur est bien créé, mon pointeur de CExpoEmetteur aussi... cependant au moment de la construction de ce dernier, il y a un appel au destructeur de CEmetteur qui réalise un delete de ma structure Rating.... alors que je ne l'est pas demandé évidemment...

Donc je pense savoir ce qu'il se passe, en amont je crée un objet CEmetteur, et ensuite je le donne en argument dans le constructeur de CExpoEmetteur, et dans ce constructeur j'alloue de la memoire comme ca

_emetteur = new CEmetteur(emetteur);

Dans cette instruction, le constructeur de CEmetteur est appelé, alors que mon objet CEmetteur est déjà créé, donc pour passer de l'un à l'autre il fait appel au destructeur, bien qu'il ne soit pas appelé...
Je sais pas si c'est ca... mais si oui que faut-t-il faire... et sinon... que faire également...

merci beaucoup d'avance

lundi 12 juillet 2010 à 17:07:26 | Re : BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

CptPingu

2061 msg

2475 msg CS

Réponse acceptée !
C'est un problème extrêmement classique.

Tu oublies que tout objet est passé par *copie* !!!
Ex:

Code C/C++ :

#include <iostream>

class Pouet
{
public:
  Pouet(int* i)
  : _i (i)
  {
    std::cout << "kikoo" << std::endl;
  }
  ~Pouet()
  {
    std::cout << "destroy" << std::endl;
    delete _i;
  }
  int* _i;
};

void testBug(Pouet p)
{
  std::cout << *p._i << std::endl; // Ok la 1ere fois
  // bug, car copie, donc p, et donc i, va se faire detuire ici
  // appel du destructeur, car p est une variable local temporaire
}

void test(const Pouet& p)
{
  std::cout << *p._i << std::endl;
  // ok, car pas de copie !
}

int main()
{
  int* i = new int[1];
  *i = 5;
  Pouet pouet(i);
  test(pouet);
  test(pouet);
  test(pouet);
  testBug(pouet);
  testBug(pouet);
  testBug(pouet);

  return 0;
}

La solution est simple: si tu as des pointeurs en interne deux solutions:
- Tu ne copie jamais ces objets, tu peux désactiver la copie des objets en héritant de boost::noncopyable, ou en laissant la definition des opérateurs de copies, vide.
- Tu réécris les opérateurs de copies, de manière à faire une véritable copie de ce que pointent tes pointeurs.

Ex qui résoud le problème pour l'exemple donné précédemment:

Code C/C++ :

  Pouet(const Pouet& p)
  {
    // On copie pour de vrai, on ne fais pas seulement "pointer le pointeur".
    _i = new int[1];
    *_i = *p._i;
  }

________________________________________________________________________
Historique de mes créations, et quelques articles:
http://0217021.free.fr/portfolio
Merci d'utiliser Réponse acceptée quand un post répond à votre question

lundi 12 juillet 2010 à 17:20:21 | Re : BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

eupeup

41 msg

54 msg CS

Super merci...

j'avais pas pensé à mettre en argument une référence constante...
J'ai opté pour ça du coup... et évidemment ca marche, hehe!!!!

ExpoEmetteur(const CEmetteur& emetteur, autres arguments...) : _autresArguments(autresArgument)
{
_emetteur = new CEmetteur(emetteur);
}

Encore merci!!!

lundi 12 juillet 2010 à 17:27:07 | Re : BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

CptPingu

2061 msg

2475 msg CS

Quelque soit ta solution, il faut gérer ce cas. Soit en désactivant la copie, soit en la remplaçant.
C'est le genre de bug vicieux qui va vite revenir sinon :p

Soit dit en passant, pense à tout passer en "const&" si tu ne modifies pas l'objet passé en argument. Le différence de performance peut être assez monstrueuse.

________________________________________________________________________
Historique de mes créations, et quelques articles:
http://0217021.free.fr/portfolio
Merci d'utiliser Réponse acceptée quand un post répond à votre question

lundi 12 juillet 2010 à 20:06:20 | Re : BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

eupeup

41 msg

54 msg CS

Ok, merci pour le conseil...

Sinon j'ai fait un tit test et ca marche, en revanche, j'ai un soucis que je comprends pas...

Je reprends mon exemple :
ExpoEmetteur(const CEmetteur& emetteur, autres arguments...) : _autresArguments(autresArgument)
{
_emetteur = new CEmetteur(emetteur);
}

Admettons que je veuille créer un vector d'objet ExpoEmetteur...
Je fais d'abord :

ExpoEmetteur expo = ExpoEmetteur(emetteur, autres arguments...);
vector<ExpoEmetteur> listExpo;
listExpo.push_back(expo);

et ca marche très bien...

En revanche, lorsque je fais

vector<ExpoEmetteur> listExpo;
listExpo.push_back(ExpoEmetteur(emetteur, autres arguments...));

Ca ne marche plus, il repasse par le destructeur... et fait donc un delete de _emetteur...
Est ce normal?? Il me semble que ces deux bouts de code sont semblables...
Ou est ce propre à la classe vector, et il repasse un nouvelle fois en copie et tutti quanti...

Merci d'avance!!!

mardi 13 juillet 2010 à 09:59:15 | Re : BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

CptPingu

2061 msg

2475 msg CS

ExpoEmetteur expo = ExpoEmetteur(emetteur, autres arguments...);

La syntaxe correcte serait (On evite une copie inutile):

ExpoEmetteur expo(emetteur, autres arguments...);

Si tu ne veux pas appeler le destructeur et le constructeur sans cesse, tu dois faire un std::vector de ExpoEmetteur*.

Code C/C++ :

ExpoEmetteur* expo = new ExpoEmetteur(emetteur, autres arguments...);
std::vector<ExpoEmetteur*> listExpo;
listExpo.push_back(expo);
// Ne pas oublier de détruire tous ce qui est dans le vecteur, une fois que l'on a fini de travailler avec.

PS: Évite absolument le using namespace std, voir:

[ Lien ]

________________________________________________________________________
Historique de mes créations, et quelques articles:
http://0217021.free.fr/portfolio
Merci d'utiliser Réponse acceptée quand un post répond à votre question

mardi 13 juillet 2010 à 10:04:44 | Re : BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

CptPingu

2061 msg

2475 msg CS

Ca ne marche plus, il repasse par le destructeur... et fait donc un delete de _emetteur...

Petite précision: cette erreur ne devrait pas arriver. Quelque soit la manière dont tu manies ta classe (donc même avec un std::vector de celle-ci), il ne devrait pas y avoir de problème.
Pour que ta classe soit parfaitement conçu, il est impératif, indispensable, obligatoire et très important que tu redéfinisses l'opérateur de copie ! Ce n'est pas une option ou une méthode alternative.
D'une manière générale, dès que tu as un pointeur dans une classe, l'opérateur doit être recodé.

________________________________________________________________________
Historique de mes créations, et quelques articles:
http://0217021.free.fr/portfolio
Merci d'utiliser Réponse acceptée quand un post répond à votre question

vendredi 23 juillet 2010 à 18:50:51 | Re : BESOIN D'AIDE : PB AVEC POINTEURS, CONSTRUCTEUR ET DESTRUCTEUR

eupeup

41 msg

54 msg CS

Salut à tous!!!

bon j'ai encore un tit soucis...

D'une part, j'ai une classe définissant un objet CExposition.
D'autre part, j'ai un fichier qui contient une fonction calculExposition() ne prenant rien en argument et retournant un objet CExposition.
Et finalement dans mon main, afin de tester, j'ai codé un truc du genre :

CExposition test = calculExposition();

Et là, j'ai un problème, dans ma fonction tout se passe normalement jusqu'au return, et là patatra, il y a un appel du destructeur de CExposition (sans que je le demande bien sur, hehe)...
J'imagine que c'est à cause de la copie... est ce bien cela? et que faire?

De plus, dans un message précédent, on m'a précisé que pour avoir une classe bien conçue, je dois recoder l'operateur de copie...
Alors je vais poser une question conne, mais l'opérateur de copie est bien l'opérateur '=', c'est bien cela?

Merci d'avance

Peupeu

Cette discussion est classée dans : pointeur, constructeur, destructeur, emetteur, cemetteur

Répondre à ce message

Sujets en rapport avec ce message

Constructeur/destructeur [ par Ben0007 ] Hello,En en-tête d'un fichier .cpp je trouve #include blablabla, puis:/******************************************* Constructeur et Destructeur******** Constructeur de copie et destructeur [ par tom24 ] Bonjour à tous! Voila je debute en programmation (je suis en premiere année de DUT informatique), j'ai commencé la programmaton avec java et je conti Probléme variable static et classe incompréhension [ par Booster ] Bonjour,Voilà, j'ai une incompréhension au niveau d'une variable static. Voici mon code :maclasse.hclass maclasse; //Ca me sert juste à ce que ma vari un pointeur sur un tableau de pointeur [ par brekiano ] g un ptt probleme,je veux declarer un pointeur sur un tableau de pointeur, chaque pointeur du tableau doit referer a un neurone,en faite c mon PFEje d Probleme avec pointeur urilisant string [ par Orlyn ] Bonjour, Je viens de finir mon prog de gestion de liste d'attente avec des structures comme ceci : struct Maillon { int numero; Maillon *suivant; Comment lire le N° de série constructeur d'une unité de stockage ? [ par steph12358 ] Bonjour à tous1/ Lors de leur fabrication les disques durs, clés USB et autres recoivent un N° de constructeur unique (en principe)Quelqu'un sait-il e pointeur et adresse [ par nannouuD ] salut a tous,estce que quelqu'un pourra me renseigner sur la signification de cette ecriturestd::complex (*W)[M] = &Z[k][0];sachant les declaration su Comment traiter l'erreur d'un pointeur foireux ? [ par Booster ] Bonsoir à tous,Alors voilà j'ai une fonction comme suit :bool Fonction (void * ptr, int tailleptr){ char message[2000]; memcpy(message , ptr , les pointeurs en c [ par flofloo ] bonjour tout le monde,en faite je voudrais juste qu'on parle un peu des pointeurs en c car j'ai une petite zone d'ombre dessus je voudrais savoir a qu A propos des pointeurs de fichier [ par dayvid75014 ] Bonjour,Je ne comprends pas pourquoi lorsque j'utilise un pointeur de fichier en dehors du main, cela me renvoie une erreur, comme si j'accédais à des

Livres en rapport

Tous les livres pour Pointeur, Constructeur, Destructeur, Emetteur, Cemetteur

Nos sponsors

Derniers Blogs

IMAGINE CUP 2012, MAKE A SIGN EN FINALE par junarnoalg

KINECT 1.5 IS OUT ! par Vko

LES ACTUALITéS DE LA SEMAINE SUR C2I.FR (14 MAI - 20 MAI) par richardc

REACTIVE EXTENSIONS : CONSOMMER DES SERVICES AVEC RX PARTIE 3, LES PIèGES à éVITER par Groc

SHAREPOINT BLOG SITE, PROBLèME D'ARCHIVES par junarnoalg

Forum

RE : MULTIPLICATION MATRICE [MATRICE C++] par joshua69

RE : MULTIPLICATION MATRICE [MATRICE C++] par yann_lo_san

MULTIPLICATION MATRICE [MATRICE C++] par joshua69

SUPERVISION DE LA BATTERIE par romanticgirl

RE : OPENCV ET INTERFACE GRAPHIQUE VISUAL STUDIO par buno

Côté IT

Offres d'emploi

Formations Video

Appels d'offres

Recheche Web Master pour MAJ de plateformes Musical
Budget : 500€
Realisation d-une version 2 d-un site e-commerce existant sous prestashop-REPUBLICATION
Budget : 5 000€
Recherche Infogerance Serveur Debian Linux
Budget : 1 800€

Logiciels

sDEVIS-FACTURES vlPRO (8.1.0.3)
SDEVIS-FACTURES VLPRO (8.1.0.3)

sDEVIS-FACTURES vlPRO a été mis au point pour les particuliers, créateurs, entrepreneurs, artisa...

Cliquez pour télécharger sDEVIS-FACTURES vlPRO
974 Application Server (12.2.4.6)
974 APPLICATION SERVER (12.2.4.6)

Développez de puissantes applications dans un environnement de 'cloud computing', clusterisé, séc...

Cliquez pour télécharger 974 Application Server
vPicture (1.4.2.1)
VPICTURE (1.4.2.1)

Avec vPicture, hébergez vos images facilement et rapidement. vPicture est un utilitaire simple, ...

Cliquez pour télécharger vPicture
Easy-Planning (2.2.1.6)
EASY-PLANNING (2.2.1.6)

Easy-Planning permet de créer des plannings sous la représentation de diagrammes et est adapté au...

Cliquez pour télécharger Easy-Planning
COM-BACKUP (2.0)
COM-BACKUP (2.0)

COM-BACKUP est un logiciel de sauvegarde qui permet de planifier les sauvegardes de vos dossiers ...

Cliquez pour télécharger COM-BACKUP