Discussion :

[Paltoquet]

Bonjour,

Je cherche a modifier un void pointer au sein d'une Hastable.
Si le mot est présent dans la table j'incrémente son champs data de type (void *) de 1.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void hastable_add(char *mot){
  Entry e;
  Entry* r;
  e.key=mot;
  r=hsearch(e,FIND);
  //voila ou ca se gate, normalement r contient l'addresse du résultat de ma recherche
  if(r!=null){
    int* nombre;
    nombre=(int *)(r->data);
    (*nombre)++;
    }
}

Je n'ai pas les résultats attendus.
Je ne sais pas si pour changer le champs data je dois faire r->data ou (*r).data
Si je modifie nombre, r->data doit bien être changé par référence non?

Je vous remercie d'avance.

[diogene]

Donne un code compréhensible : il faut aussi donner la définition des types.

Je ne sais pas si pour changer le champs data je dois faire r->data ou (*r).data

C'est la même chose : -> est un sucre syntaxique pour alléger l'écriture.

Si je modifie nombre, r->data doit bien être changé par référence non?

Surement pas. D'ailleurs, tu ne modifies pas nombre mais *nombre

D'après ce que tu écris, data serait un champ d'une structure Entry. Ce champ serait une adresse (sans doute un void*) de quelque chose. Ce quelque chose serait un int que tu veux incrémenter. Dans l'affaire, il n'est pas question de modifier r->data mais l'entier dont l'adresse est dans r->data

[Paltoquet]

Merci pour la réponse.

Je m'excuse pour la propreté du code.

D'après ce que tu écris, data serait un champ d'une structure Entry.

Oui, il s'agit bien d'une structure ENTRY.

Son champs data est bien de type (void*)

Il me semble qu'un cast en (int *) est nécessaire pour accéder à sa valeur afin de la modifier,mais je en sais pas comment changer cette valeur.
Est que je dois passer par un nouveau pointeur pour modifier son champs ?

[Sve@r]

Son champs data est bien de type (void*)

Il me semble qu'un cast en (int *) est nécessaire pour accéder à sa valeur afin de la modifier

Bonjour
Un cast en (int *) n'est nécessaire qu'à la condition que tu veuilles y stocker un int. Si tu veux y stocker un double, tu devras alors le caster en (double *). Et dans tous les cas, t'assurer qu'à l'adresse où tu stockes cet int (ou ce double) tu as la place de l'y mettre...

mais je en sais pas comment changer cette valeur.

Sous réserve que data pointe vers une zone mémoire suffisante pour stocker cette valeur: *(int*)r->data=valeur.

Dans ton cas, il semble que tu veuilles juste incrémenter la valeur qui s'y trouve: (*(int*)r->data)++ (ce qui correspond en plus compact à ce que t'avais écrit avec ton pointeur "nombre"...

Je n'ai pas les résultats attendus.

Alors vérifie les valeurs initiales...

Est que je dois passer par un nouveau pointeur pour modifier son champs ?

Quand tu as des pièces dans la poche droite, est-ce que tu dois d'abord les passer dans la poche gauche pour payer ton pain ?

[diogene]

il y a une ambiguité dans la manière dont tu présentes les choses:
Que veux-tu incrémenter , le champ data (donc l'adresse sans doute d'un int) ou ce qui est pointé par le champ data (apparemment un int) ?
Tu parles toujours d'incrémenter le champ data et tu codes l'incrément de ce qui est pointé par data.
Tout cela n'est pas clair : Quelle est ton intention ?

[Paltoquet]

Je suis désolé si je m'exprime mal.

Le but de cette fonction est de lire un fichier text et d'ajouter chacun des mots à une table dont le champs pointé par data contient le nombre d’occurrence du mot.

Avec vos remarques j'en suis arrivé à ce code là:
la fonction add

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
 
void hash_table_add(char *mot){
	ENTRY* r;
	ENTRY e;
	e.key=mot;
	r=hsearch(e,FIND);
	if(r==NULL){
		int a=1;
		e.data=&a;
		e.key=mot;
		r=hsearch(e,ENTER);
		return;
	}
	else{
	(*((int*)(r->data)))++;
}
}

la fonction hash_table_search

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
int hash_table_search(char *mot){
	ENTRY* r;
	ENTRY e;
	e.key=mot;
	r=hsearch(e,FIND);
	if(r==NULL){
		return 0;
		}
	else {
	return *(int*)(r->data);
}
}

le problème lorsque je test 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
16
17
int main(void)
{
 
  hash_table_create();
  /* Remplissage de la table */
  hash_table_add("foo");
  hash_table_add("bar");
  hash_table_add("bar");
 
  /* Afficher la valeur associée à un mot */
  printf("Nombre d'occurences de bar: %d\n", hash_table_search("bar"));
  printf("Nombre d'occurences de foo: %d\n", hash_table_search("foo"));
  printf("Nombre d'occurences de zoo: %d\n", hash_table_search("zoo"));
 
  hash_table_destroy();
  return EXIT_SUCCESS;
}

j'obtient le même nombre d’occurrences (champs pointé par data) pour les 2 mots.
Si vous avez des conseils n'hésitez pas, merci.