Discussion :

[Krustig]

Après s'être inspiré d'un ancien post sur le forum developpez.com j'ai tenté d'utiliser find_if.

Mais j'ai quelques soucis...

Voici ce que je veux faire :

Detecter un noeud dans une liste d'ojbet "Emprunteur" grâce à un ID stocké dans cette objet.

Voici mon "code" :

Code :

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

1
2
3
4
5
6
7
void HashT::remove(string itsPrenom, string itsNom) // Suppression d'une adhérent par nom et prenom
{
	list<Emprunteur>::iterator empIterator;
	int id = HashFX(itsNom + itsPrenom);
	empIterator = find_if(EmprunteurList.begin(), EmprunteurList.end(), MaClassEgal(id));
 
}

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
#pragma once
#include "Emprunteur.h"
 
struct MaClassEgal
{
    MaClassEgal(int n)
	{
		nb = n;
	}
 
    bool operator ()(const Emprunteur& m) const
    {
		return m.GetId() == nb;
    }
 
private :
 
    int nb;
};

Merci pour votre aide

[Sylvain Togni]

Ça a l'air correct, quel est le problème ?

[Krustig]

La voici : error C2275: 'EmprunteurList'*: utilisation non conforme de ce type comme expression

Voici la définition dans le .h

Code :

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

1
2
3
4
5
6
7
8
9
10
 
typedef list<Emprunteur> EmprunteurList;
 
....
 
 
private:
	EmprunteurList **mTab;
	unsigned int mTabSize;
	unsigned int mnElement

Je ne pense pas que ca vient de là étant donné que j'arrive à ajouter des éléments dans la liste... donc dans le tableau de pointeur sur liste Emprunteur (pfiou:p)

[Laurent Gomila]

Code :

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

EmprunteurList.begin(), EmprunteurList.end()

EmprunteurList est un type, pas une variable.

A part ça, un list<>** ... c'est vraiment nécessaire ?

[Krustig]

Si ca ne tenait qu'a moi : non !
Je dois réaliser une table de hachage externe donc je vais devoir stocker les informations dans une liste chainée à l'indice du tableau modulo ma taille utile.

Sinon tout fonctionne très bien maintenant !
Merci pour votre aide !

PS : je suis ouvert à toutes critiques de mon code, donc n'hésitez pas s'il existe une meilleur solution que <> ** ! Comprendre ses erreurs c'est apprendre

[Krustig]

Cela dit, j'ai encore une question au sujet du find_if !
Comment savoir s'il a vraiment trouvé la valeur recherchée ?
Car find_if return last s'il n'a rien trouvé mais si je n'ai qu'un element dans ma liste : first = last, donc je risque d'avoir des erreurs....

[poukill]

EmprunteurList est un type, pas une variable.

Laurent a trouvé ton problème !

[Krustig]

to the first element in the range for which the application of pred to it does not return false (zero).
If pred is false for all elements, the function returns last. www.cplusplus.com

Tu en es sur pour le end ?