Erreur à la sortie du programme

Version imprimable

Bonjour,

J'ai un petit problème lors de l'exécution de mon programme : à la fin de celui ci j'ai un message d'erreur (cf. image "erreur2.jpg").

Mon programme est structuré comme cela :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
int main(void)
{
	// nom du fichier d'origine
	char nomFichO[20] = "";
	
	
	/***** Partie recuperation des donnes du fichier d'origine *****/
	
	cout << "Veuillez saisir le nom du fichier d'origine : ";
	cin >> nomFichO;
	
	
	// ouverture du fichier en lecture
	std::ifstream monFichier(nomFichO);
	
	
	// test l'existance du fichier
	if(is_readable(nomFichO) != 0){
		// test l'ouverture fu fichier
		// si tous les tests sont OK
		if(monFichier){
			// creation de l'objet
			Bordereau monBordereau;

			// nom du nouveau fichier
			char nomFichN[20] = "";

			int nbDestinataire = 0;
			
			
			// lecture du fichier ligne par ligne et recuperation des infos
			lectureFichOrigine(monFichier, &monBordereau);
			
			// recupere nombre de destinataires enregistres
			nbDestinataire = monBordereau.conteneurDest.size();
			
			
			/***** Partie creation du nouveau fichier avec les donnees recuperees *****/
			
			cout << "\nVeuillez saisir le nom du nouveau fichier : ";
			cin >> nomFichN;
			
			// création du nouveau fichier
			std::ofstream nouvFich(nomFichN, ios::trunc);
			
			
			// ecriture des informations recuperees dans le nouveau fichier
			ecriture(nouvFich, &monBordereau, nbDestinataire);
			
			// fermeture du fichier
			nouvFich.close();
			
			
			/***** FICHIER PRET A ETRE UTILISE *****/
			
			// suppression des variables
			monBordereau.~Bordereau();
		}ERREUR (cf. image "erreur.jpg")
		else{
			// si le nom de fichier est incorrect
			// ou si le fichier n'est pas accessible en lecture
			system("cls");
			cout << "Erreur a l'ouverture du fichier :" << endl;
			cout << "nom de fichier incorrect ou interdit en lecture !!!" << endl;
			main();
		}
	}
	// si le fichier n'existe pas
	else{
		system("cls");
		cout << "Erreur a l'ouverture du fichier : fichier introuvable !!!" << endl;
		main();
	}
	
	// fermeture du fichier d'origine
	monFichier.close();
	
	
	return 0;
}

En mode debug, j'ai vu que l'erreur se situe au moment de sortir de 2ème if.
Mais je vois pas ce qui bloque :(

Si vous avez une idée, elle est la bienvenue ;)

Merci

25/07/2006, 09h31
Médinoc

Euh... Pourquoi appelles-tu explicitement le destructeur ?
25/07/2006, 09h40
Sachiel31

C'était un test pour voir si ca pouvait résoudre mon problème.
Mais ca ne change rien qu'il y soit ou pas :(
25/07/2006, 09h55
Médinoc

Je ne pense pas que quiquonque puisse t'aider sans voir cette fameuse classe Bordereau...

Tu as essayé d'utiliser le debugger DANS le destructeur, pour voir quelle partie de celui-ci plante ?

Classe Bordereau :

fichier .h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
#ifndef _BORDEREAU_H_
#define _BORDEREAU_H_
 
#include <iostream.h>
#include <string>
#include <vector>
 
#include "emetteur.h"
#include "destinataire.h"
 
/***********************************\
* un bordereau est composé :
*       - d'un emmetteur
*       - d'une liste de destinataires
*       - de la date d'execution
*       - de la devise
*       - du montant total
\***********************************/
 
 
class Emetteur;
class Destinataire;
 
 
class Bordereau
{
private:
	std::string date;
	std::string devise;
	std::string montantTotal;
 
public:
	std::vector<Destinataire> conteneurDest;
	std::vector<Destinataire>::iterator itDest;
 
	std::vector<Emetteur> conteneurEm;
	std::vector<Emetteur>::iterator itEm;
 
	Emetteur theEmetteur;
	Destinataire theDestinataire;
 
	Bordereau();
	~Bordereau();
 
	// assesseurs et mutateurs des attributs de la classe
	const std::string get_date() const;
	void set_date(std::string value);
 
	const std::string get_devise() const;
	void set_devise(std::string value);
 
	const std::string get_montantTotal() const;
	void set_montantTotal(std::string value);
 
};
 
 
#endif // _BORDEREAU_H_

fichier .cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
#include "bordereau.h"
#include "operateur.h"
 
 
 
// constructeur par défaut
Bordereau::Bordereau(){
	set_date("");
	set_devise("");
	set_montantTotal("");
}
 
// destructeur
Bordereau::~Bordereau(){
}
 
 
//assesseur et mutateur de l'attribut date
const std::string Bordereau::get_date() const{
	return date;
}
 
void Bordereau::set_date(std::string value){
	date = value;
	return;
}
 
 
//assesseur et mutateur de l'attribut devise
const std::string Bordereau::get_devise() const{
	return devise;
}
 
void Bordereau::set_devise(std::string value){
	devise = value;
	return;
}
 
 
//assesseur et mutateur de l'attribut montantTotal
const std::string Bordereau::get_montantTotal() const{
	return montantTotal;
}
 
void Bordereau::set_montantTotal(std::string value){
	montantTotal = value;
	return;
}

Le destructeur passe bien, j'ai une erreur au moment de passer l'accolade de sortie du if.

25/07/2006, 10h47
Médinoc

Ah, dans ce cas, tu peux retirer la classe Borderaux. C'est un autre élément qui est mal détruit. (Enfin, teste en retirant l'appel explicite au destructeur d'abord, des fois que le compilo nous tente une double destruction...
25/07/2006, 10h57
Sachiel31

J'ai supprimé l'appel explicite du destructeur, ça ne change rien.:(

Je pense que c'est un problème d'allocation mémoire ou de pointeur non détruit mais je vois pas lequel puisque j'ai bien fait attention à tous les détruire.:?
25/07/2006, 11h03
Médinoc

Ça peut aussi être un problème de double destruction...
25/07/2006, 11h04
Sachiel31

Un problème de double destruction???

Les seuls pointeur créés dynamiquement, le sont dans le corps d'une fonction et je les détruis à la fin de cette fonction.
25/07/2006, 11h09
Médinoc
Code:

1 2 3 char nomFichO[20] = ""; //... cin >> nomFichO;
Au fait, tu es sûr qu'il vérifie la taille, l'opérateur >> ?
Ne penses-tu pas qu'une std::string serait plus appropriée ?
25/07/2006, 11h20
Sachiel31

En effet un string est plus approprié et c'est plus propre :mrgreen:

Mais bon, même avec ce changement, je garde toujours mon problème
:arf:
25/07/2006, 11h23
Médinoc

Qu'est-ce qu'il y a dans LectureFichOrigine() ?
25/07/2006, 11h35
Sachiel31

Voila mon fichier main.cpp

Si ca peut éclairer quelques points ;)

La fonction LectureFichOrigine() va lire le fichier texte et stocker différentes infos dans les vector correspondant.
25/07/2006, 13h11
Médinoc
Code:

1 2 3 // suppression des pointeurs pour liberer la memoire delete [] token; delete [] id;
token n'a pas l'air d'être une chaîne allouée dynamiquement, tu ne dois donc pas la deleter.
id doit être deleté dans la boucle (fuite de mémoire)

PS: Si possible, évite d'utiliser strtok(). Essaie plutôt d'utiliser strtok_r() (POSIX) ou strtok_s() (équivalent de strtok_r() pour Windows à partir de Visual 2005)
25/07/2006, 13h55
Sachiel31

En effet mon "delete [] token" ne sert à rien xD

Par contre, visual ne me reconnait ni strtok_s() ni strtok_r().
J'ai droit au joli message d'erreur :

Code:

Bordereau v3\main.cpp(262) : error C2065: 'strtok_s' : undeclared identifier

Code:

Bordereau v3\main.cpp(262) : error C2065: 'strtok_r' : undeclared identifier

Les librairies à utiliser sont bien string et stdio ???
25/07/2006, 14h51
nikko34

Si je me souviens bien, le strtok de visual est déjà modifié pour éviter les problèmes que l'on peut rencontrer sur les autres systèmes.
25/07/2006, 15h46
Médinoc

Le strtok() de visual est thread-safe, mais pas réentrant.

Sachiel31: strtok_r() n'est pas reconnu par visual, et strtok_s() nécessite Visual 2005 ou supérieur.

Si tu ne te retrouve pas à analyser deux chaînes en même temps, sous Windows, tu peux conserver strtok() tant que tu n'auras pas un Visual assez récent.
25/07/2006, 15h50
Sachiel31

Je garde donc strtok() puisque je travail sur un Visual "trop vieux" xD

Ca ne pose pas de problème puisque je ne lis qu'une ligne à la fois.
25/07/2006, 15h52
Sachiel31

J'ai contourné mon problème en faisantun exit(0) à la fin de ma boucle if imbriquée, puisque de toute façon si on arrive à cet endroit c'est que le programme s'achève.

Mais bon ce n'est qu'on moyen de contournement et ça me plait moyen :?

Je continue donc à chercher une solution plus propre pour résoudre ce problème car malgrès tous les changements que l'on m'a conseillés, j'ai toujours le même problème. :mur:

Je prend toutes les idées :mrgreen:

P.S.: mon nouveau main.cpp
26/07/2006, 09h47
Sachiel31

Re bonjour,

J'ai décidé en accord avec ma conscience (...et mon boss quand même) que finalement je ne vais pas passer une journée de plus à essayer de comprendre pourquoi ça plantait à la fin de l'exécution, alors que ça ne devrait pas. :bug:

J'ai trouvé une solution qui marche même si c'est vrai que ce n'est pas forcément la meilleur.

Voila voila.

:merci: pour votre aide.

A+ :coucou:

Sachiel