Discussion :

[StarCluster]

Bonjour,

J'ai un soucis sur une comparaison entre deux unsigned char qui me donne des résultats inattendu.

Pour commencer, voici le code fautif :

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
bool RectangleMap::contain(const int x, const int y) const
{
	int pos = this->toFlattenIdx(x, y);
 
	if( pos < 0 || pos >= this->length )
		return false;
 
	std::cout << pos << " -> " << this->map[pos] << " and we don't want " << static_cast<unsigned>(this->Blocked) << std::endl;
 
	if( this->map[pos] == this->Blocked )
	{
		std::cout << "Good!" << std::endl;
		return false;
	}
	std::cout << pos << " -> " << this->map[pos] << " Nieh!?!" << std::endl;
 
	return true;
}

Ici, this->map est de type : unsigned char* tandis que this->Blocked est de type unsigned char et vaut 0. La variable map contient uniquement des 0 ou des 1.
La fonction int toFlattenIdx(const int x, const int y) const prend deux coordonnées x et y et me renvoie l'indice correspondant dans le tableau aplati (map en l'occurence).
Les cout sont là pour déboguer.

Mon problème avec ce code, c'est que this->map[pos] == this->Blocked est tout le temps faux, même lorsque map[pos] vaut 0 (et est donc égale à Blocked).
Je ne comprend pas pourquoi. Les deux variables sont du même type, donc il n'y a pas de cast implicite qui pourrais intervenir....

Quelqu'un a t'il une idée ?

PS : y a t'il moyen d'utiliser la balise de code en mode "inline" ?

[Invité]

Salut

Ligne 8, tu affiches this->map[pos] et static_cast<unsigned>(this->Blocked), avec map[pos] et Blocked tous deux de type unsigned char.
Du coup, est-ce que tu n'aurais pas le caractère '0' dans map et non l'entier 0.
En d'autres termes, que se passe-t-il si tu affiches plutôt static_cast<unsigned>(this->map[pos]) ?

Et pour la balise, c'est "c" à la place de "CODE".

[StarCluster]

Salut

Ligne 8, tu affiches this->map[pos] et static_cast<unsigned>(this->Blocked), avec map[pos] et Blocked tous deux de type unsigned char.
Du coup, est-ce que tu n'aurais pas le caractère '0' dans map et non l'entier 0.
En d'autres termes, que se passe-t-il si tu affiches plutôt static_cast<unsigned>(this->map[pos]) ?

Et pour la balise, c'est "c" à la place de "CODE".

J'ai effectivement oublié le cast pour l'affichage.
Je n'avais pas pensé au caractère '0' ! Et c'était ça.... Merci. Je me suis fait prendre au piége sur la lecture de fichier. Le code suivant

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
std::ifstream mapFile(argv[1], std::ios::in);
if( !mapFile )
{
	std::cerr << "Cannot open file '" << argv[1] << "'" << std::endl;
	return -1;
}
 
mapFile >> mapWidth >> mapHeight;
 
map = new unsigned char[mapWidth * mapHeight];
 
for(unsigned int i = 0; i < mapWidth * mapHeight; ++i)
	mapFile >> map[i];
 
mapFile.close();

lit les 0 et 1 comme des caractères, pas comme des nombres. Reste à trouver comment les lui faire lire comme des nombres.

Habitude du C, je ne suis pas encore rompu au subtilité liè à la surcharge de fonction/opérateur du C++.

[Invité]

Reste à trouver comment les lui faire lire comme des nombres.

Peut-être en remplaçant la ligne 13 par :

Code :

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

1
2
3
4
5
{
    int n;
    mapFile >> n;
    map[i] = static_cast<unsigned char>(n);
}

[StarCluster]

C'est aussi la solution à laquelle je suis venu. J'aurais préféré évité de passer par une valeur intermédiaire (question de goût), mais sans passer par les fonctions du C, ça ne me semble pas possible.

Merci pour l'aide !

[Bousk]

Si tu es sur de n'avoir que des chiffres, tu peux toujours tenter un - '0' pour chaque valeur mapFile >> map[i]; map[i] -= '0';