Ecriture/Lecture d'un fichier binaire avec vector

Version imprimable

Bonsoir,
je cherche désespérément à pouvoir écrire dans un fichier binaire (pour empêcher une édition trop simple)
les informations d'une tilemap qui se trouvent dans un struct.

Voici la structure:
Code:

1 2 3 4 5 6 7 8 9 struct dataMap{ unsigned int mapX; unsigned int mapY; vector<vector<int>> dataTexture1; vector<vector<int>> dataTexture2; vector<vector<int>> dataCollision; vector<vector<int>> dataEvent; };
L'idée c'est que puisse définir la largeur et la hauteur de ma carte puis de définir les données de chaque tile
dans un fichier texte. Cette étape est faite et fonctionne. Maintenant je n'arrive pas écrire dans le fichier binaire
toute les informations puisque j'ai des tableaux dynamiques et encore moins à lire le fichier pour retrouver mon
dataMap.

Merci d'avance !

23/12/2015, 00h54
dragonjoker59

Salut!
Des vector de vector? Tu t'amuses bien toi!
As-tu regardé du côté de Boost.Serialize pour ton problème?
23/12/2015, 01h06
herotom99

J'utilise un vector de vector pour les deux dimensions (ça va simplifier mon code par la suite). Oui j'ai regardé un peu la sérialisation
mais à vrai dire je n'es pas vraiment compris le principe et la syntaxe. Est-ce que la sérialisation est la seul solution ?
Même si j'ai bien vu que l'utilisation de plusieurs vecteurs de vecteurs dans un même struct compliquait les choses ^^
23/12/2015, 09h58
stendhal666

Je ne suis pas sûr que d'avoir des vectors de vectors simplifie réellement ton code mais bon, il n'y a rien d'interdit!

Quelles sont exactement les difficultés que tu rencontres pour écrire dans le fichier binaire? Je ne vois pas de rapport immédiat avec la question de l'allocation dynamique de la mémoire...

Pour faire très simple, je verrais le schéma suivant:

- un en-tête qui donnerait le nombre d'enregistrements,
- chaque enregistrement commencerait par les 2*sizeof(size_t) octets contenant les dimensions du vecteur² à allouer,
- puis viendraient les données elles-mêmes.

Pour copier un vecteur d'int, c'est assez facile: fs.write( (char*) vec.data(), vec.size()*sizeof(int) );Copier un vecteur de vecteurs, là, c'est plus pénible parce qu'il faut répéter l'opération vec.size() fois.

Pour la lecture, c'est à peu près la même chose à l'envers, mais il ne faut pas oublier d'allouer d'abord de la mémoire dans le vecteur de destination: vec v(size); fs.read(vec.data(), size*sizeof(int));Attention à bien utiliser RVO / move semantic / emplace_back pour éviter de faire trop de copies de tes données...
23/12/2015, 10h56
Bousk

Déjà, oublie les vector de vector, c'est inutilement compliqué contrairement à ce qu'on peut croire http://cpp.developpez.com/faq/cpp/?p...au-de-tableaux

Citation:

Envoyé par herotom99

Oui j'ai regardé un peu la sérialisation
mais à vrai dire je n'es pas vraiment compris le principe et la syntaxe. Est-ce que la sérialisation est la seul solution ?

Ben écrire dans un fichier ça s'appelle sérialiser à vrai dire. :roll:
23/12/2015, 14h16
herotom99

Hum je veux bien utiliser des tableaux à deux dimensions classique mais le problème c'est que la taille de ma carte (XY)
n'est défini que par l'utilisateur donc je n'es aucun moyen de connaître la taille de mon tableau pour la lecture du fichier :/

Parce qu'au final, j'ai bien plusieurs tableaux dans un struct mais je dois pouvoir les déclarer sans connaître leur taille final... (d'où les vector).
Donc déjà ça pose un problème.

Ensuite, imaginons que j'ai réussi à définir la taille des tableaux que je voulais. Lorsque je vais vouloir lire mon fichier binaire pour retrouver mes tableaux
je ne vais pas pouvoir puisque la taille du tableau dépend de l'utilisateur... C'est pour ça que j'avais mis appart la taille X et Y de ma map dans le struct.
Sauf que lorsque je lis le fichier je lis toute la structure en même temps ! Donc je ne vois pas comment faire.

Peut être faire deux fichiers avec l'un qui contient les tailles et un autres les valeurs ? mais c'est pas pratique ...

mais sinon je suis bien d'accord qu'utiliser un vector de vector ça devient lourd :p
23/12/2015, 14h35
stendhal666

Utiliser des vectors c'est très bien. Des vectors imbriqués, c'est moins bien. Il suffit d'une fonction pour transformer l'adressage 2D en adressage 1D.
Boost::serialize permet de sauvegarder des vectors dans des archives sans grande difficulté.
Sinon, et c'est ce que je te proposais, place en début d'enregistrement sur 2*sizeof(size_t) octets les dimensions du tabeau, avant les données du vecteur elles-mêmes. Tu lis les dimensions, tu crées un vecteur de lignes*colonnes éléments pour réserver la mémoire et tu copies dedans les lignes*colonnes*sizeof(int) octets suivants du fichier.
23/12/2015, 14h52
dragonjoker59

stendhal666 t'a donné la marche à suivre pour l'écriture et la récupération.
Ensuite, personne n'a parlé d'utiliser des tableaux à 2 dimensions classiques, mais bien de remplacer tes vector< vector< truc > > par des vector< truc >
23/12/2015, 15h00
herotom99

Oui c'est ce que je vais faire (tout mettre en un seul tableau et grâce a une fonction et aux dimensions XY ,obtenir un tableau deux dimensions).
Mais là je regarde surtout la sérialization de vector. sauf que je n'arrive pas vraiment à comprendre comment cela fonctionne (avec boost)
23/12/2015, 15h10
stendhal666

Tu as un tuto sur le site: http://khayyam.developpez.com/articl...serialization/

Bon, après avoir installé/build boost j'ai réécris une partie du code. J'arrive à écrire dans le fichier
(le fichier ne fait pas 0 KB mais 1 KB :) ) par contre lorsque j'essais de lire le fichier je ne retrouve
pas les bonnes valeurs :/
Je précise que les valeurs rentrées sont X = 50, Y = 50 et les tableaux sont remplis de 50*50 fois 1.

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
struct dataMap{
	unsigned int mapX;
	unsigned int mapY;
 
	vector<int> dataTexture1;
	vector<int> dataTexture2;
	vector<int> dataCollision;
	vector<int> dataEvent;
 
	template <typename Archive>
	void serialize(Archive& ar, const unsigned int version)
	{
		ar & mapX;
		ar & mapY;
		ar & dataTexture1;
		ar & dataTexture2;
		ar & dataCollision;
		ar & dataEvent;
	}
};
 
----------------------------------------------------------------------
 
bool EcrireMap(dataMap map)
//Ecrire les informations dans le fichier binaire///
{
	//Trouve le fichier dans lequel écrire
	string NomMap;
	cout << "Nom de la map: ";
	cin >> NomMap;
	NomMap = "Map/" + NomMap + ".bin";
 
	//Ecriture
	ofstream fichier(NomMap.c_str());
	boost::archive::binary_oarchive ofichier(fichier);
	ofichier << map;
 
	return true;
}
 
----------------------------------------------------------------------------
 
void readMap()
//Lire le fichier binaire ciblé//
{
	//Cherche le fichier//
	string NomMap;
	cout << "Nom de la map: ";
	cin >> NomMap;
	NomMap = "Map/" + NomMap + ".bin";
 
	//Lis le fichier
	dataMap MAP;
	ifstream fichier(NomMap.c_str());
	boost::archive::binary_iarchive ifichier(fichier);
	ifichier >> MAP;
 
	//Simple test du fichier lu//
	cout << MAP.mapX << endl;
	cout << MAP.mapY << endl;
}

Ce que j'obtiens après la lecture:
3435973836
3435973836
et pas mes deux 50.

23/12/2015, 18h40
stendhal666

On peut voir le code d'appel?

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
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
bool LoadFile(string filename,int X,int Y,vector<int> &dataTable)
{
	string pathfile = "MapFile/" + filename;
	ifstream fichier;
	fichier.open(pathfile,ios::beg);
	if (fichier.is_open()) 
	{
		int nbVal = X*Y;
		for (int i = 0; i < nbVal; i++)
		{
			vector<int> tableau;
			int valeur;
			fichier >> valeur;
			dataTable.push_back(valeur);
		}
	}
	else
	{
		return false;
	}
	fichier.close();
 
	return true;
}
 
int GetInformation(dataMap &mapData)
{
	string dataTexture1File;
	string dataTexture2File;
	string dataCollisionFile;
	string dataEventFile;
 
	int m_mapX;
	int m_mapY;
	vector<int> m_dataTexture1;
	vector<int> m_dataTexture2;
	vector<int> m_dataCollision;
	vector<int> m_dataEvent;
 
 
	cout << "Largeur de la map: ";
	cin >> m_mapX;
	cout << endl;
	if (m_mapX <= 0) return 1;
 
	cout << "Hauteur de la map: ";
	cin >> m_mapY;
	cout << endl;
	if (m_mapY <= 0) return 1;
 
	//CHARGEMENT TEXTURE INFERIEUR//
	cout << "Fichier de la texture inferieur: ";
	cin >> dataTexture1File;
	cout << endl << "Chargement du fichier ..." << endl;
	if (LoadFile(dataTexture1File, m_mapX, m_mapY, m_dataTexture1))
	{
		cout << "Chargement du fichier reussi !" << endl << endl;
	}
	else 
	{
		return 2;
	}
 
	//CHARGEMENT TEXTURE SUPERIEUR//
	cout << "Fichier de la texture superieur: ";
	cin >> dataTexture2File;
	cout << endl << "Chargement du fichier ..." << endl;
	if (LoadFile(dataTexture2File, m_mapX, m_mapY, m_dataTexture2))
	{
		cout << "Chargement du fichier reussi !" << endl << endl;
	}
	else
	{
		return 2;
	}
 
	//CHARGEMENT COLLISION//
	cout << "Fichier de collision: ";
	cin >> dataCollisionFile;
	cout << endl << "Chargement du fichier ..." << endl;
	if (LoadFile(dataCollisionFile, m_mapX, m_mapY, m_dataCollision))
	{
		cout << "Chargement du fichier reussi !" << endl << endl;
	}
	else
	{
		return 2;
	}
 
	//CHARGEMENT EVENT//
	cout << "Fichier des evenements: ";
	cin >> dataEventFile;
	cout << endl << "Chargement du fichier ..." << endl;
	if (LoadFile(dataEventFile, m_mapX, m_mapY, m_dataEvent))
	{
		cout << "Chargement du fichier reussi !" << endl << endl;
	}
	else
	{
		return 2;
	}
 
	cout << "----------------------------------------------------" << endl;
	cout << "TEXTURE INFERIEUR: " << dataTexture1File << endl;
	cout << "TEXTURE SUPERIEUR: " << dataTexture2File << endl;
	cout << "COLLISION: " << dataCollisionFile << endl;
	cout << "EVENEMENT: " << dataEventFile << endl;
	cout << "----------------------------------------------------" << endl << endl;
 
	return 0;
}

Ne me tapez pas pour les copier/coller svp ;(
Et sinon dans mon main() je crée juste un dataMap MAP auquel j'applique les informations GetInformation(MAP)

23/12/2015, 21h26
stendhal666

Euh... Il n'y a pas de sérialisation, ni de fichier binaire dans ce code, on est d'accord?
Je pensais que c'était l'objet de la discussion et que tu voulais savoir pourquoi tu n'obtenais pas les bonnes valeurs avec une archive Boost?
23/12/2015, 21h36
herotom99

Hum, si non ? mon commentaire un peu plus haut:
C'est comment j'écris en binaire avec la sérialisation de boost.
Mon dernier commentaire c'est le reste du code (comment je
lis les valeurs d'un fichier texte simple).

J'ai peur de ne pas bien comprendre
23/12/2015, 23h23
herotom99

Je viens de voir que lorsque je sérialise mon fichier en text_oarchive j'obtiens la ligne suivante dans mon fichier:
22 serialization::archive 14 0 0 3435973836 3435973836 0 0 0 0 0 0 0 0
On retrouve bien mes valeurs étrange donc c'est bien lorsque j'écris dans le fichier qu'il y a un problème.
24/12/2015, 11h23
stendhal666

D'accord, mais comme il y a plus de chances que le problème vienne de ton code que de boost, je voulais voir comment, à quel moment, avec quelles variables tu appelais la fonction de sérialisation.
Sans ça, je n'ai pas trop d'idée de ce qui peut se passer...
A part ça, joyeux Noël!
24/12/2015, 15h26
herotom99

Bon et bien en réalité tout fonctionnait bien. Le problème c'est que j'avais oublié bêtement d'appliquer ma lecture (fonction loadMap) à mon struct dataMap. De ce fait
les vectors et mes deux variables étaient vides. Lorsque je lisais la map cela m'affichait pas le contenu de la variable (puisqu'elle était vide) mais l'emplacement de ma
mémoire allouée ... Donc j'ai corrigé l'erreur et tout fonctionne parfaitement maintenant :)

En revanche, chose à noter lorsque je sérialiser en binary_oarchive, le fichier est deux fois plus lourd qu'en text_oarchive. Après peut être que la lecture sera plus rapide
je sais pas ^^ en tout cas il faut juste que le fichier ne soit pas éditable facilement !

Merci beaucoup pour votre aide, ça fait plaisir de voir autant de personne qui rendent service ! Joyeux noël :D