Discussion :

[Petrolevb]

Bonjour à tous

Je vous présente ici mon problème car j'aimerais beaucoup comprendre ce qu'il se passe dans mon code et dans l'exécution du programme.

J'ouvre un fichier en binaire (fichier de données brutes qui importe peu en soi)
Je déclare ensuite un buffer d'une taille de 32, puis lis depuis mon stream.

Code :

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

1
2
char buffer[SIZE_ENTRY];
ifs.read(buffer, SIZE_ENTRY);

Je déclare ensuite quelques booléens avec des comparaisons entre le contenu du buffer et des constances connues à l'avance

Code :

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

1
2
bool deletedFile = buffer[0] == 0xe5 ;
bool longName = buffer[0] == 0x41 || buffer[0] == 0x42;

Et ici est mon problème
La variable longName est valide, c'est a dire que la comparaison est bonne.
Mais pas la variable deletedFile
C'est extrêmement étrange, mais après tâtonnement divers, pour qu'elle soit valide, il faut que ma comparaison soit

Code :

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

bool deletedFile = buffer[0] == 0xffffffe5 ;

Pourquoi ?
Jusqu'a présent, toutes mes comparaisons fonctionnaient normalement (tel 0x42 un peu plus haut)
De même qu'un dump de la section me confirme qu'il n'y a absolument aucune suite de 0xFF avant cet octet

Code :

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

1
2
3
70 00 74 00 00 00 ff ff    6d 00 00 00 76 00 00 00
e5 4f 4d 4f 50 45 52 53    57 4d 56 20 00 00 36 6b
69 32 69 32 00 00 39 6b    69 32 0c 00 93 a3 7a 00

La section d'octet que je copie est bien la seconde et la troisième ligne et le buffer est de toute façon écrasé par la lecture mais on peut tout de même voir que les octets précédents sont des 0x00 et non des 0xFF

Merci pour toute explication
PetroleVB

[Médinoc]

Tu dois utiliser unsigned char.

Un char signé supérieur à 127 est négatif, donc quand il est promu en int...

[Petrolevb]

Mon Dieu... c'était si simple ...

effectivement, cela a marché
j'ai du remanier le code un peu car les fstream prennent des char en entrée,
mon ifs.read ressemble maintenant à

Code :

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

ifs.read((char*)(&buffer[0]), SIZE_ENTRY);

Merci pour cette réponse rapide

[Médinoc]

Remplace ton cast C-style par un reinterpret_cast<>, inutile de mettre en danger la const-correctness de ton programme.

Il devrait même être possible de rendre ton ifstream non-signé, et ainsi s'affranchir complètement du cast.