Discussion :

[Sachiel31]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Médinoc]

Euh... Pourquoi appelles-tu explicitement le destructeur ?

[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

[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 ?

[Sachiel31]

Classe Bordereau :

fichier .h

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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...

[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.

[Médinoc]

Ça peut aussi être un problème de double destruction...

[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.

[Médinoc]

Code C++ douteux :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[Sachiel31]

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


Mais bon, même avec ce changement, je garde toujours mon problème

[Médinoc]

Qu'est-ce qu'il y a dans LectureFichOrigine() ?

[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.

[Médinoc]

Code suspect :

Sélectionner tout - Visualiser dans une fenêtre à part

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)

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Les librairies à utiliser sont bien string et stdio ???

[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.

[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.

[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.

[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.


Je prend toutes les idées

P.S.: mon nouveau main.cpp

[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.

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

Voila voila.

pour votre aide.

A+

Sachiel