Discussion :

[Jeano81]

Bonjour à tous.

Ayant pour but de m’entraîner à manipuler du code C++, je suis tombé sur ceci :

http://www.primenumbers.net/Renaud/fr/crypto/XOR.htm

J'ai donc voulu essaye d'appliquer cette méthode de codage de texte sur un fichier, ce qui m'à mené à pondre ce petit programme dont voici le code :

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
#include <iostream>
#include <fstream>
#include <string>
 
using namespace std;
 
int main()
{
	ifstream file("fichier.txt" , ios::in|ios::binary);
	string key = "nimp";
	char texteOrigin, texteConv[10];
 
	for (int i(0), j(0); i < 10;i++, j++)
	{
 
		file.read((char *)&texteOrigin, sizeof(char));
		if (j > key.size())
		{
			j = 0;
		}
 
		texteConv[i] = texteOrigin ^ key[j];
	}
 
	file.close();
	cout << texteConv << endl;
	ofstream fileenc("fichier.txt", ios::out|ios::trunc);
	fileenc << texteConv;
	system("pause");
	return 0;
}

J'aimerais avoir votre avis sur ce code, car je me retrouve avec un petit problème lors du décodage.

à savoir : pour tester ce petit programme, j'ai créé un fichier nommé "fichier.txt".
Ce fichier contient simplement cette ligne : "azertyuiop".

Le codage ce passe bien (je pense).
Mais le décodage... Aie :

Je me retrouve bien avec mes 10 caractères "azertyuiop" en début de mon fichier "fichier.txt", mais ils sont suivit de caractères assez "bizarres", je ne sais pas si c'est pas les caractères résultant du codage précédent.

En mode pas à pas, ma variable texteConv contient bien RIEN-QUE mes caractères "azertyuiop" lorsque je demande au programme de me l'enregistrer dans le fichier txt (ligne d'instruction N°28).

donc je ne comprends pas trop ce qu'il ce passe.

ps:
Texte dans fichier.txt non codé : "azertyuiop"
Texte dans fichier.txt codé : "tpÌÌÌÌÌÌÌÌÌÌÌÌÌpÌÌÌÌÌÌÌÌ"
Texte dans fichier.txt décodé : "azertyuiopÌÌÌÌÌÌÌÌÌÌÌÌÌpÌÌÌÌÌÌÌÌà"2
nimp"
(sans les " aux extrémités).

Merci !

[white_tentacle]

texteConv n’est pas terminé par un '\0'. Donc tu ne peux pas l’envoyer tel quel à travers un flux. Tu peux utiliser « write ».

Plutôt que « read », tu peux utiliser « get » pour lire un caractère à la fois.

Et sinon, plutôt que d’utiliser des char* ou des char[], utilise des std::vector<char> (ou std::array<char, 10>), cela t’évitera des erreurs. char* est à réserver pour les chaînes terminées par des '\0', quand utiliser std::string n’est pas possible.

[Jeano81]

Merci pour votre réponse.

J'ai suivis vos conseils et ai modifié mon code.
Il est à présent comme ceci :

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
#include <iostream>
#include <fstream>
#include <string>
#include <vector>
 
using namespace std;
 
int main()
{
	ifstream file("fichier.txt" , ios::in|ios::binary);
	string key = "nimp";
	vector <char>  texteConv;
	char texteOrigin;
 
	for (int i(0), j(0); i < 10;i++, j++)
	{
		file.get(texteOrigin);
		if (j > key.size())
		{
			j = 0;
		}
 
		texteConv.push_back(texteOrigin ^ key[j]);	
		cout << texteConv[i];
	}
 
	cout << endl;
 
	file.close();
 
	ofstream fileenc("fichier.txt", ios::out|ios::trunc);
	for (int i(0); i < texteConv.size(); i++)
	{
		fileenc.write(&texteConv[i],texteConv.size());
	}
 
	system("pause");
	return 0;
}

à présent, dans mon fichier décodé je me retrouve avec ceci :

"azertyuiopzertyuiopÍertyuiopÍÍrtyuiopÍÍÍtyuiopÍÍÍýyuiopÍÍÍýýuiopÍÍÍýýýiopÍÍÍýýýýopÍÍÍýýýýŸpÍÍÍýýýýŸó", au lieu de "azertyuiop" à l'origine.

Bien que sur la console, mon résultat qui s'affiche est bien "azertyuiop"...

Je pense que mon erreur viens de la dernière boucle for (ligne 32 à 35), j'ai pensé à le modifier comme ceci :

Code :

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

1
2
3
4
for (int i(0); i < texteConv.size(); i++)
{
	fileenc.write(&texteConv[i],i);
}

pour écrire caractère par caractère et non 10 fois 10 caractères (ce qui était illogique dans le code), mais ça ne fonctionne pas non plus...

J'ai aussi essayé :

Code :

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

1
2
3
4
for (int i(0); i < texteConv.size(); i++)
{
	fileenc.put(texteConv[i]);
}

Mais c'est pas non plus approprié apparemment.

[Médinoc]

C'est pourtant bien la dernière solution qui est la bonne...

[prgasp77]

L'erreur vient de là : fileenc.write(&texteConv[i],texteConv.size());. D'un côté tu demandes d'écrire un caractère à la fois et de l'autre tu lui demandes d'écrire toutes les données d'un coup. Prends quelques minutes pour lire et comprendre la documentation de std::ostream::write() et celle de std::vector::data et tu sauras quoi écrire .

[Jeano81]

D'un côté tu demandes d'écrire un caractère à la fois et de l'autre tu lui demandes d'écrire toutes les données d'un coup.

Voilà pourquoi j'ai fini par écrire : fileenc.write(&texteConv[i],i); (que j'ai écris dans le post précédent). Mais ça fonctionne toujours pas ! où alors quelque chose m'échappe !

[prgasp77]

RTFM !

1. Que prend en second argument la méthode std::basic_ostream::write ?
2. Qu'est i ?
3. Est-ce la même chose ?