Discussion :

[Proteiforme]

Je teste un peu le C++, et j'obtiens à la compilation :

*** Error in `/home/jessie/workspace/c++-002/Debug/c++-002': double free or corruption (fasttop): 0x00000000023f9260 ***

Je cherche et je n'ai pas encore le recul pour comprendre l'erreur.

Voici les sources si quelqu'un peut m'aider à comprendre ce serait sympa.
Je suis sous GNU Debian Linux et j'utilise Eclipse et le compilateur GCC.

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
#include <iostream>
#include <cmath>
#include <fstream>
 
#include "Adresse.h"
 
using namespace std;
 
 
 
int main()
{
	char file[26] = "Essai";
 
	ofstream monFlux(file, ios::binary);
	if(monFlux)
	{
 
	}
	else
	{
	cout << "Erreur" << endl;
	}
 
 
Adresse Personne1(102, "Charles", 1990, 26);
Adresse Personne2(100, "Inconnu", 1900, 0);
 
monFlux.write((char *) &Personne1, sizeof Personne1);
monFlux.close();
 
	ifstream lire(file, ios::binary);
 
 
	lire.read((char *) &Personne2, sizeof Personne2);
	if(lire)
		{
 
		}
		else
		{
		cout << "Erreur" << endl;
		}
 
 
Personne1.Lecture_Attribut();
Personne2.Lecture_Attribut();
 
lire.close();
 
 
}

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
#ifndef ADRESSE_H_
#define ADRESSE_H_
 
#include <iostream>
#include <string>
 
class Adresse
{
 
public:
 
	// Constructeur par defaut
	Adresse();
 
	// Constructeur
	Adresse(	signed int vi_cle,
				std::string string_nom,
				signed short vs_annee,
				signed short vs_age	);
 
 
	void Lecture_Attribut();
 
	~Adresse();
 
private :
 
	signed int mvi_cle;
	std::string mstring_nom;
	signed short mvs_annee;
	signed short mvs_age;
 
 
};
#endif /* ADRESSE_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
#include "Adresse.h"
 
// Constructeur par defaut
Adresse::Adresse()
{
	mvi_cle = 0;
	mstring_nom = "Inconnu",
	mvs_annee = 0,
	mvs_age = 0;
}
 
// Constructeur
Adresse::Adresse(	signed int vi_cle, std::string string_nom, signed short vs_annee, signed short vs_age ) :
										mvi_cle(vi_cle),
										mstring_nom(string_nom),
										mvs_annee(vs_annee),
										mvs_age(vs_age)
{
 
}
 
// Destructeur par Defaut
Adresse::~Adresse()
{
	// TODO Auto-generated destructor stub
}
 
// Methode Lecture Attribut
void Adresse::Lecture_Attribut()
{
	std::cout << mvi_cle << "\n";
}

[ternel]

Cette erreur survient en général quand on libère la mémoire pointée (par un pointeur non null) mais déjà libérée.

L'une des façons les plus courante d'y parvenir, c'est de copier un pointeur dans un constructeur par copie (par exemple, généré par le compilateur), et qu'on détruit le pointé dans le destructeur.

Dans le code que tu montres, je ne le vois pas.
Utilise un débuggeur, pour trouver dans quel destructeur ce produit l'erreur.

[fenkys]

Code C++ :

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

1
2
3
4
5
6
7
8
9
Adresse Personne2(100, "Inconnu", 1900, 0);
 
monFlux.write((char *) &Personne1, sizeof Personne1);
monFlux.close();
 
	ifstream lire(file, ios::binary);
 
 
	lire.read((char *) &Personne2, sizeof Personne2);

Cette partie du code me semble douteuse. La fonction membre read requiert un char *, il faut lui fournir un vrai char *. Caster un pointeur objet en char * ne converti pas l'objet, juste le pointeur. Ton problème vient de ce que la fonction read, de par cette conversion "douteuse" considère la zone mémoire de l'objet de type adresse comme une suite de caractère. Il écrase donc les données spécifique de Personne2 avec des char. Et quand l'objet est détruit, les membres qui contiennent des données incohérentes vont accéder à des zones mémoires interdites.

La bonne pratique consiste à créer un buffer (ici un tableau de char) que tu passes à la fonction lire puis transférer ce buffer à ton objet par la fonction qui va bien (qui dans ton cas n'existe pas encore, tu dois donc la rajouter à ta classe Adresse). Tu peux aussi utiliser une fonction qui utilise un std::string à la place d'un buffer (cf ::getline) que tu passeras à ton objet. Avec cette deuxième méthode tu n'as pas à te préoccuper de la taille des données reçues, string se charge de tout.


Bonne chance.

[Bousk]

Je doute que tu obtiennes ça à la compilation, c'est une erreur à l'éxécution.
Tes read et write avec cast en char* sont bien cavaliers et devraient être modifiés avant toute chose.

[Médinoc]

Wow. On dirait du C, où l'on pouvait se permettre de lire et écrire des structures complètes directement dans un fichier binaire... à condition de savoir ce que l'on faisait.

En C++, cette approche ne passe pas avec les classes modernes. Il faut faire soi-même la sérialisation des données, et C++ préfère fortement les fichiers texte aux fichiers binaires.

[Bousk]

N'allons pas aussi loin, les fichiers binaires sont tout autant utiles et utilisés en C++.
Ton programme affiche ce que tu attends avant de planter ? Que Personne1 et Personne2 ont les mêmes infos ?
A priori, l'erreur vient à la destruction de Personne2.
Avec ton mauvais write, ce que tu écris dans le fichier ne correspond pas à grand chose, surtout pour le nom(? je suppose, on ne sait absolument rien de ton Adresse). A vue de nez tu chopes au mieux le pointeur vers la string interne. Quand tu read, tu donnes ce pointeur à Personne2, et quand chacun le détruit, ça crash.
C'est dommage d'utiliser des fstream pour en faire un truc aussi moche. Pour utiliser fstream et (dé)sérialiser une classe, on surcharge les opérateur >> et <<.

[Proteiforme]

Effectivement c'est à l'execution et non à la compilation ce message d'erreur.

Oui c'est façon C que j'ai voulu copier des blocs d'un coup.

Je travail le C++ mais je sature. Le livre de de jesse liberty le langage C++ et pas tres fournis sur le travail avec les fichiers.
Son exemple et fourni tel quel je l'ai moi meme ecris.

J'ai besoin de comprendre, je mis prend mal pour l'instant.
Oui j'ai ce que je veux, personne2 contient bien personne1.

Le debugger m'amene bien sur string.

Merçi pour ses explications rapides.