Discussion :

[darkVador2]

Bonjour
J`ai un probleme de programmation en C++
le but du programme est d'avoir des char* stockes dans une table de hachage de type hash_map<char*, char*>
mon probleme est pour recuper les valeurs dans cette table de hachage.
j'essaie une premiere fois avec un char* a qui je donne directement une valeur par exemple "pentree" la ca marche
st je calcule cette entree a partir d un string de la meme valeur pour obtenir un char* qui aura aussi la meme valeur c ane marche plus

si quelqu'un connait la solution ca serait super sympa de me donner une indication car je planche dessus depuis un bout de temps

voici le morceau de code:

je definis ma table de hachage:

Code :

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

hash_map<char*, char*> maHashTable;

ensuite je la remplis dans une fonction

Code :

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

1
2
3
4
5
6
7
8
void maClasse::createHashTable()
	{
	char * entree1 = "pentree";
	char * entree2 = "sentree";
	maHashTable[albumT] = "sortie1"
	maHashTable[imageN] = "sortie2";
 
}

jusque la tout va bien
ensuite j'essaie d'acceder a ces valeurs dans une autre fonction qui prend en entree un string.
La rien ne va plus

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
std::string maClasse::valueOfHashTable(std::string sWordToReplace)
{
	std::string hashValue;
	char * charHashValue;
	char * apart;
	std::string testString = "pentree";
	char * testPipo = "pentree";
	//je fais l essai avec un string pour retrouver la valeur associee a "pentree"
 
	apart = testBis;
 
       // maintenant je compare les deux valeurs testPipo et apart
       // les deux valent exactement la meme chose
 
if(strcmp(apart,testPipo)==0)
	{
		cout << "the two values are same"<<endl;
		cout << apart<< endl;
		cout << testPipo << endl;
	}
	charHashValue = maHashTable[testPipo];// la ca marche
        charHashValue = maHashTable[apart]; // la ca marche plus
hashValue = charHashValue;
return hashValue;
 
}

Balises code ajoutées.

[Matthieu Brucher]

2 char* identiques ne signifie pas 2 objets pointés égaux, mais bien 2 fois le même pointeur. Donc si tu fais un char* = "toto" et un string "toto" que tu transformes en char*, les 2 sont différents.

[darkVador2]

salut
merci d'avoir repondu aussi rapidement.
Je suis d'accord avec ce que tu as dit mais les deux objets on beau avoir des adresses differentes, la valeur finale est la meme.
Mais je ne vois quand meme pas pourquoi ca bloque puisque lorsque la table a ete construite l'adresse des objets etait differente de celle que j'utilise pour les appeler. Et les deux adresses correspondent a des adresses de char*, chacune representees par la meme valeur.
Est ce que tu pourrais me dire ce que tu penses que je devrais changer?

[Matthieu Brucher]

Tu ne compares pas ce qu'il y a dans l'espace pointé par les char* mais les char* entre eux. Si tu veux comparer les chaînes de caractères, le plus simple est de prendre des string à la place, c'est fait pour ça...

[Médinoc]

Ta hash_map hache directement le pointeur au lieu de son contenu, je pense.
Utilise des std::string dans ta hash_map.

[darkVador2]

ok
merci beaucoup a tous les deux pour vos reponses tres rapides.
Je vais tenter de recreer la Hash_map avec des strings.

[loufoque]

char * entree1 = "pentree";

Il s'agit du genre de code qu'il ne faut jamais écrire.

[Médinoc]

Ah, je ne l'avais pas vu celui-ci: Les chaînes sont constantes, il faut utiliser des char const * ou des tableaux de caractères initialisés.

Si tu as GCC, ajoute l'option de compilation -Wwrite-strings qui t'évitera de telles erreurs.

[darkVador2]

merci beaucoup
ca y est ca a fini par marcher
en fait j' ai pris des const char * et j'ai redefini la methode de comparaison de deux elements

a plus