begin process at 2012 05 28 17:49:44
  Trouver un code source :
 
dans
 
Accueil > Forum > 

C++ & C++ .NET

 > 

Divers

 > 

Débutant(e)

 > 

Destructeur intempestif


Derniers messages déposésPoser une question dans le forum ou lancer une discussion

Destructeur intempestif

mercredi 30 décembre 2009 à 21:24:57 | Destructeur intempestif

mliuej

Bonsoir,

J'ai un nouveau problème dans mon application : lorsque je crée un objet, plusieurs destructeurs s'enclenchent sans que j'en aie donné l'instruction... Voici le résultat :

Code :
Achat d'un nouveau billet :
----------------------------

Veuillez entrer le nom du visiteur : Clinton
Veuillez entrer le prenom du visiteur: Bill
Veuillez entrer la date de naissance du visiteur: 01/01/2000
Veuillez entrer le numero du visiteur : 007
Veuillez entrer le numero du billet : 1
Veuillez entrer la date de validite du billet : 20/12/2005
Merci !

Destructeur : Visiteur
Destructeur : Personne
BILLET
------
Numero du billet : 1
Date d'echeance : 20/12/2005

Nom du visiteur : Nom
Prenom du visiteur : Prenom
Numero du visiteur : 0000000000
Destructeur : Visiteur
Destructeur : Personne
Destructeur : Visiteur
Destructeur : Personne
Destructeur : Visiteur
Destructeur : Personne
Appuyez sur une touche pour continuer...


On voit ici que le visiteur prend les données par défaut malgré les valeurs que je lui donne. Pourtant, dans les constructeurs, je n'invoque jamais les destructeurs !

Code C/C++ :
void AchatBillet()
{
    system("cls");

    fl<<"Achat d'un billet";

    string n, p, dn, nv, nb, db;
    
    cout<<endl<<"Achat d'un nouveau billet : ";
    cout<<endl<<"----------------------------"<<endl<<endl;
    
    cin.clear();
    cin.ignore(numeric_limits<streamsize>::max(), '\n');
    
    cout<<"Veuillez entrer le nom du visiteur : ";
    cin>>n;
    cout<<"Veuillez entrer le prenom du visiteur: ";
    cin>>p;
    cout<<"Veuillez entrer la date de naissance du visiteur: ";
    cin>>dn;
    cout<<"Veuillez entrer le numero du visiteur : ";
    cin>>nv;
    
    Visiteur v1(n, p, dn, nv);
    
    cout<<"Veuillez entrer le numero du billet : ";
    cin>>nb;    
    cout<<"Veuillez entrer la date de validite du billet : ";
    cin>>db;    
    cout<<"Merci !"<<endl<<endl;

    BilletEntree _be(v1, nb, db);
    
    _be.affichage();    
    fbe.save(_be);  
    pause();
}


Code C/C++ :
BilletEntree::BilletEntree(Visiteur v, string numB, string dateB)
{
    numeroBillet = numB;
    dateBillet = dateB;
    setnomVisiteur(v.getNom());
    setprenomVisiteur(v.getPrenom());                                 
}


Code C/C++ :
Visiteur::Visiteur(string n, string p, string dn, string nv) : Personne(n, p, dn)
{
    setNumeroVisiteur(nv);
}  


Code C/C++ :
Personne::Personne (string n, string p, string d)
{
    Nom = n;
    Prenom = p;
    DateNaissance = d;
}


Est-ce une caractéristique des "string" ou bien ai-je loupé un épisode ?

Pour info, voici le bout de code de la sauvegarde du BilletEntree :

Code C/C++ :
void FichierBilletEntree::save(BilletEntree &be)
{    
    numeroBillet = be.getnumeroBillet();
    Nom = be.getnomVisiteur();
    Prenom = be.getprenomVisiteur();
    Date = be.getdateBillet();
    
    string num = fichier + ".txt";

    ofstream liste(num.c_str(), ios::app);
    if(liste)
    {
        liste<<"Nom du visiteur : "<<Nom<<endl;
        liste<<"Prenom du visiteur : "<<Prenom<<endl;
        liste<<"Billet numero : "<<numeroBillet<<endl;
        liste<<"Date d'echeance : "<<Date<<endl<<endl;
        liste.flush();
    }
    else
    {
        cout<<"Impossible d'ouvrir le fichier : "<<what<<endl;
        system("PAUSE");
        exit(1);
    }
    
    ofstream file(filename().c_str(),ios::trunc);
    if(file)
    {
        file << be;
        file.flush();
    }
    else
    {
        cout<<"Impossible d'ouvrir le fichier : "<<filename()<<endl;
        system("PAUSE");
        exit(1);
    }
    
}
jeudi 31 décembre 2009 à 00:28:42 | Re : Destructeur intempestif

aardman

Membre Club
Salut,
Quand tu ne passes pas ton objet par référence, c'est une nouvelle copie qui est crée à chaque fois.
Le constructeur de BilletEntree devrait plutôt prendre un const Visiteur& v en param plutôt qu'un Visiteur v.
Le remarque tient aussi pour les strings. Si tu pouvais mettre un printf dans le destructeur de string, tu serais surpris par le nombre de copie inutiles que ton programme fait.
jeudi 31 décembre 2009 à 00:52:16 | Re : Destructeur intempestif

mliuej

Merci bien pour ta réponse. J'ai testé mais je me retrouve toujours devant le même problème, à part le premier destructeur de Visiteur et le premier destructeur de Personne qui ont disparu... Pourtant je pense bien avoir fait ce qu'il fallait

Code :
Achat d'un nouveau billet :
----------------------------

Veuillez entrer le nom du visiteur : deney
Veuillez entrer le prenom du visiteur: geof
Veuillez entrer la date de naissance du visiteur: 08/07/1988
Veuillez entrer le numero du visiteur : 007
Veuillez entrer le numero du billet : 1
Veuillez entrer la date de validite du billet : 20/12/2049
Merci !

BILLET
------
Numero du billet : 1
Date d'echeance : 20/12/2049

Nom du visiteur : Nom
Prenom du visiteur : Prenom
Numero du visiteur : 0000000000
Destructeur : Visiteur
Destructeur : Personne
Destructeur : Visiteur
Destructeur : Personne
Destructeur : Visiteur
Destructeur : Personne
Appuyez sur une touche pour continuer...


Code C/C++ :
void AchatBillet()
{
    system("cls");

    fl<<"Achat d'un billet";

    string n, p, dn, nv, nb, db;
    
    cout<<endl<<"Achat d'un nouveau billet : ";
    cout<<endl<<"----------------------------"<<endl<<endl;
    
    cin.clear();
    cin.ignore(numeric_limits<streamsize>::max(), '\n');
    
    cout<<"Veuillez entrer le nom du visiteur : ";
    cin>>n;
    cout<<"Veuillez entrer le prenom du visiteur: ";
    cin>>p;
    cout<<"Veuillez entrer la date de naissance du visiteur: ";
    cin>>dn;
    cout<<"Veuillez entrer le numero du visiteur : ";
    cin>>nv;
    
    Visiteur v1(n, p, dn, nv);
    
    cout<<"Veuillez entrer le numero du billet : ";
    cin>>nb;    
    cout<<"Veuillez entrer la date de validite du billet : ";
    cin>>db;    
    cout<<"Merci !"<<endl<<endl;

    BilletEntree _be(v1, nb, db);
    
    _be.affichage();    
    fbe.save(_be);  
    pause();
}


Code C/C++ :
BilletEntree::BilletEntree(const Visiteur& v, string numB, string dateB)
{
    numeroBillet = numB;
    dateBillet = dateB;
    setnomVisiteur(v.getNom());
    setprenomVisiteur(v.getPrenom());                                 
}


Code C/C++ :
Visiteur::Visiteur(const string& n, const string& p, const string& dn, const string& nv) : Personne(n, p, dn)
{
    setNumeroVisiteur(nv);
}


Code C/C++ :
Personne::Personne (const string& n, const string& p, const string& d)
{
    Nom = n;
    Prenom = p;
    DateNaissance = d;
}


Je sais je suis lourd
mercredi 6 janvier 2010 à 12:10:10 | Re : Destructeur intempestif

djbneben

je pense qu'il faudrait que tu crée un constructeur de copie toi meme, car celui qui est crée par défaut par le compilateur est bete... Donc les pointeur qui sont dans ta classe vont pointer vers la mm adresse, quelque soit l'objet... Faut me corriger si je me trompe
mercredi 6 janvier 2010 à 21:58:23 | Re : Destructeur intempestif

mliuej

Il y a déjà un constructeur de copie mais merci pour la bonne idée


Cette discussion est classée dans : code, string, cout, visiteur, destructeur


Répondre à ce message

Sujets en rapport avec ce message

Premier Poste, premier code :p [ par Locarre ] Bonsoir à tous, je me suis au C++ depuis très peu de temps, voila un petit code source, et j'aimerais bien que vous me disiez si celui ci par sur de b STL: Problème stockage dans un container multimap [ par elnino67 ] Bonsoir,Nous utilisons un programme ayant recours à une multimap utilisant: [code] multimap m;[\CODE]en clé: le champ nom d'une classe Client, et en o Plantage lors de l'exécution ! [ par mliuej ] Bonjour à tous, Voici une fonction simple qui est sensée enregistrer un objet "Visiteur" ainsi qu'un objet "BilletEntree", et les afficher tous les d Erreur dans mon code [ par nourhene4 ] Bonjour, J'ai fais un code d'un programme qui permet de saisir les informations du personnel dans une entreprise ( employé,cadre), mais mon code compo Visual C++ Free() [ par PRZYBYLSKI ] #include #include #include "stdafx.h"Question au Pro?Qu'es qui est faux docteurArrivé à free ça bombe#include <io Problème de chaine(string) et fichier [ par alncool ] Voilà les source je j'ai fait :-Fichier principal-#include "stdafx.h"#using using namespace System;int _tmain(){ // crée un container contenant plu utilisé string dans une dll c++ [ par avg ] Bonjour à tous j'ai un problème avec la création d'une dll avec c++ pour créer les fonctions à exporté de dll si son type de retour est int ou void le Operateur + (string) [ par victorcoasne ] Bonjour, Je voulais faire un code du genre : [code=cpp]#include #include using namespace std; int main() { string test = "a" + "b"; cout Borland recevoir des données via un port serie [ par limdizrad ] Bonjour, Je n'arrive pas à recevoir des données via le module tvacomm de Borland c++ builder, voici le code que j'ai tapé, pouvez me dire ce qui cloc type string dans Visual C++ [ par fadoch ] Bonjour à tous, je travaille avec Visual C++. je souhaite savoir quelle est la différence entre le type "string" et le type "CString". et pourquoi [c


Nos sponsors


Sondage...

Comparez les prix

CalendriCode

Mai 2012
LMMJVSD
 123456
78910111213
14151617181920
21222324252627
28293031   

Consulter la suite du CalendriCode

A découvrir



 
Développement réalisé par Nicolas SOREL (Nix) avec l'aide de : Cyril DURAND et Emmanuel (EBArtSoft), Merci à Vincent pour ses précieux conseils.
CodeS-SourceS.com© Toute reproduction même partielle est interdite sauf accord écrit du Webmaster
CodeS-SourceS.com© est une marque déposée tous droits réservés

Google Coop CodeS-SourceS Google Coop CodeS-SourceS
Temps d'éxécution de la page : 1,420 sec (3)

Nous contacter | Annoncer sur CodeS-SourceS | Mentions légales