Discussion :

[Knuckles]

Bonsoir tout le monde, j'ai un problème avec la méthode read. Le code parlera mieux que moi :

Code :

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

1
2
3
4
5
6
7
8
int longueurMot;
char* buffer;
nomFichier.read((char *)&longueurMot,sizeof(longueurMot));
 
buffer = new char [longueurMot];
nomFichier.read(buffer,longueurMot);
string valeurMot = buffer;
delete[] buffer;

Malgré le fait que je spécifie la taille du buffer à longueurMot, lorsque j'affiche la valeur de buffer, il me retourne par exemple test°8= alors que mon mot est test, et que la longueur est bien de 4. Je ne vois pas ce qui peut provoquer cette erreur. Merci pour votre aide

[coyotte507]

Peut être un:

Code :

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

buffer[longueurMot] = 0;

[Knuckles]

Cela ne fonctionne pas, j'ai toujours le même affichage

J'ai toujours la possibilité de remplacer les caractères > longueur par des chaînes vides, mais c'est très moche comme méthode! C'est tout de même bizarre qu'il me place ces caractères dans la chaîne.

[JolyLoic]

Le problème n'est pas la lecture du buffer, mais la création de la chaîne à partir de celui-ci. Le constructeur de std::string prenant en paramètre un char* uniquement suppose que ce char* est une chaîne à la mode C, c'est à dire délimitée par un 0 terminal. Donc, deux possibilités :
- Tu alloues ton buffer pour un caractères de plus, que tu mets à 0.
- Tu utilises un autre constructeur de std::string, prenant en paramètre supplémentaire le nombre de caractères à lire du buffer.

La seconde méthode est préférable si ta chaîne peut contenir un 0 intermédiaire.

[JolyLoic]

Peut être un:

Code :

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

buffer[longueurMot] = 0;

Donc, tu affecte une valeur à un élément situé en dehors du tableau...

[coyotte507]

Donc, tu affecte une valeur à un élément situé en dehors du tableau...

oops

[Knuckles]

Le problème n'est pas la lecture du buffer, mais la création de la chaîne à partir de celui-ci. Le constructeur de std::string prenant en paramètre un char* uniquement suppose que ce char* est une chaîne à la mode C, c'est à dire délimitée par un 0 terminal. Donc, deux possibilités :
- Tu alloues ton buffer pour un caractères de plus, que tu mets à 0.
- Tu utilises un autre constructeur de std::string, prenant en paramètre supplémentaire le nombre de caractères à lire du buffer.

La seconde méthode est préférable si ta chaîne peut contenir un 0 intermédiaire.

Le problème c'est que l'erreur est déjà présente avant que je construise le string, un cout << buffer me retourne le même résultat.

Tiens, un buffer[longueurMot]=0 résoud le problème, mais tu viens de dire que ça écrivait à l'extérieur du tableau dc je vois pas comment avoir un résultat juste avec un code propre.

[droggo]

Fio,

Le problème c'est que l'erreur est déjà présente avant que je construise le string, un cout << buffer me retourne le même résultat.

Tiens, un buffer[longueurMot]=0 résoud le problème, mais tu viens de dire que ça écrivait à l'extérieur du tableau dc je vois pas comment avoir un résultat juste avec un code propre.

Il faut créer un buffer de longueur suffisante, c'est tout.

[Knuckles]

Fio,

Il faut créer un buffer de longueur suffisante, c'est tout.

Mon buffer est pile de la bonne taille. longueurMot est exactement de la taille du mot à lire par la suite, donc le problème ne vient pas de la!

J'ai résolu le problème en placant 0 à la fin de la chaîne par un

Code :

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

buffer[longueurMot]=0;

mais je trouve ça assez moche. Tant pis! Si quelqu'un a une idée, n'hésitez pas

[JolyLoic]

Le problème c'est que l'erreur est déjà présente avant que je construise le string, un cout << buffer me retourne le même résultat.

Normal : l'opérateur << avec un cout et un char* s'attend aussi que le char* soit à la mode C, avec un 0 terminal.

Tiens, un buffer[longueurMot]=0 résoud le problème, mais tu viens de dire que ça écrivait à l'extérieur du tableau dc je vois pas comment avoir un résultat juste avec un code propre.

Je me répète :
- Tu alloues ton buffer pour un caractères de plus, que tu mets à 0.

Ou tu utilises ma seconde solution.

[Knuckles]

Normal : l'opérateur << avec un cout et un char* s'attend aussi que le char* soit à la mode C, avec un 0 terminal.



Je me répète :
- Tu alloues ton buffer pour un caractères de plus, que tu mets à 0.

Ou tu utilises ma seconde solution.

Merci beaucoup, je vais prendre la seconde solution et chercher ce fameux constructeur. Je te remercie pour ton aide.