Discussion :

[kranagard]

bonjours, je cherche a créer un iterateur sur un container de mon cru mais j'ai un petit soucis sur une méthode.

j'ai declaré:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
 
template<class T>
class IterListe{
    private:
        ListeTrie<T> liste;
        Noeud<T> *noeud;
 
    public:
        IterListe<T>(ListeTrie<T>& l);
        bool end();
        void reset();
        bool operator++();
        bool operator++(int);
        operator T();
        T& operator& ();
};
 
template <class T>
IterListe<T>::IterListe(ListeTrie<T>& l)
{
    liste = l;
    noeud = &(l.first);
}
template <class T>
bool IterListe<T>::end()
{
    return noeud == NULL;
}
 
template <class T>
void IterListe<T>::reset()
{
    noeud = &(liste.first);
}
 
template <class T>
bool IterListe<T>::operator++()
{
    if((noeud = noeud->hasNext())==NULL)
        return false;
    else
        return true;
}
template <class T>
bool IterListe<T>::operator++(int i)
{
    return operator++();
}
template <class T>
IterListe<T>::operator T()
{
    return noeud->getData();
}
template <class T>
T& IterListe<T>::operator&()
{
    return noeud->getData();
}

et je l'utilise comme tel:

Code :

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

1
2
3
4
5
IterListe<FederationAdmin> l2(fede);
    for(l2.reset();!l2.end();l2++)
    {
        cout<<(FederationAdmin)l2<<endl;
    }

J'ai une erreur lors de la compilation sur la ligne en rouge:

error: no match for ‘operator<<’ in ‘std::cout << IterListe<T>::operator T() [with T = TeamMember]()’|

je pensais que la surcharge de l'opérateur "operator T()" gérait ceci mais je n'en suis pas tout a fait sur, ni de sa déclaration réel... si quelqu'un à déjà fait ou sait comment faire ça m'aiderait vachement parce qu'il ne me reste plus que cette classe a faire

[Emmanuel Deloget]

De manière générale, on évite la surchage d'un opérateur T() générique, car les effets de bord peuvent être étranges et dangereux. On préfèrera une fonction explicite (ou, dans ton cas, l'opérateur *() qui fait très bien ce boulot). C++1x change la donne, avec son opérateur de cast marqué du mot clef explicit qui permet de ne pas avoir les effets de bords qu'on connait.

Pour ton problème spécifique : la syntaxe est correcte (encore qu'on puisse spécifier une version const de l'opérateur, mais c'est autre chose). J'ai du mal à voir le rapport entre FederationAdmin et TeamMember - l'un est un type alias sur l'autre, je présume. Dans ce cas, il doit manquer quelque chose quelque part. Tu peux essayer de mettre l'expression complète entre parenthèses, la précédence de << étant relativement élevée. Tu devrais vérifier aussi l'existence d'une fonction

Code :

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

std::ostream& operator<<(std::ostream& o, const TeamMember& tm);

Attention : ne pas écrire cette fonction sous la forme d'un template, tu aurais des surprises.

[kranagard]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
cout<<"Administrateur:"<<endl<<"-admin"<<endl<<"Chef d'équipe:"<<endl;
    IterListe<TeamMember> l1(equipe);
    for(l1.reset();!l1.end();l1++)
    {
        cout<<(TeamMember)l1<<endl;
    }
 
    cout<<"Chef Fédération:"<<endl;
    IterListe<FederationAdmin> l2(fede);
    for(l2.reset();!l2.end();l2++)
    {
        cout<<(FederationAdmin)l2<<endl;
    }

j'ai copié la mauvaise erreur, relative a la suite du code qui est identique au précèdent.
je regarde pour le reste
merci.


[edit] j'ai bien

friend std::ostream& operator<<(std::ostream& s,TeamMember& comp);

pas de soucis de ce cotes la

j'ai aussi essayé de mettre tout entre parenthèse mais rien ne change.
pour le "const", j'y ai pensé mais cela change quoi dans ce cas précis?

sinon pour "operator *()", je vois pas. le compilateur me dit que je dois spécifier un type...
et operator T*() me retourne cette erreur:

error: no matching function for call to TeamMember::TeamMember(IterListe<TeamMember>&)’|

il essayerait de faire une copie en prenant mon iterateur comme objet de base?

en tout cas merci, ça me motive à trouver de savoir que c'est pas si simple que ça ^^

[kranagard]

en utilisant l'operateur &()

Code :

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

        T& operator& (void);

en changeant l'appel comme suit:

Code :

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

1
2
3
4
5
IterListe<TeamMember> l1(equipe);
    for(l1.reset();!l1.end();l1++)
    {
        cout<<(TeamMember&)l1<<endl;
    }

cela a l'air de fonctionner (j'ai juste un petit soucis avec mon pointeur de nœud qui n'est pas bien initialisé a première vu.
mais cela reste étrange je trouve...

[kranagard]

ah bah non, en fait il tente de passer dans l'operateur:

ostream& operator<<(ostream& s, TeamMember& comp)

sans passer par mon opérateur de l'iterateur, du coups j'ai un iterateur dans une référence de TeamMember et ça plante, logique...
j'en reviens au problème de mon operator T().



je viens de tester mon iterateur avec des int tel que:

Code :

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

1
2
3
4
5
6
7
8
    ListeTrie<int> liste;
    liste.setComparateur(compareInt);
    liste.add(1);liste.add(4);liste.add(19);liste.add(40);
    IterListe<int> l(liste);
    for(l.reset();!l.end();l++)
    {
        cout<<(int)l<<endl;
    }

et cela fonctionne parfaitement... cela viendrait de mes classes teamMember ou FederationAdmin??

au passage ceci fonctionne:

Code :

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

1
2
3
4
5
6
7
8
9
10
cout<<"Administrateur:"<<endl<<"-admin"<<endl<<"Chef d'équipe:"<<endl;
    for(int i = 0; i< equipe.size; i++)
    {
        cout<<equipe[i]<<endl;
    }
    cout<<"Chef Fédération:"<<endl;
    for(int i = 0; i< fede.size; i++)
    {
        cout<<fede[i]<<endl;
    }

[kranagard]

le code:

Code :

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

1
2
TeamMember t((TeamMember)l1);
cout<<t<<endl;

fonctionne parfaitement mais toujours pas:

Code :

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

cout<<(TeamMember)l1<<endl;

je vais le laisser comme ça pour le moment mais bon...

[Invité]

Ton operateur de sortie(operator<<) prend TeamMember par référence non-constante(voir la signature indiquée par Emmanuel Deloget)

Aprèes le code de ton iterateur ne me parait pas très bon : comme dit précedemment utilisent plutot l'operateur*() que l'operateur de cast, les operateurs d'incrémentation n'ont pas la bonne sémantique et la surcharge de l'operateur& ne me semble pas très appropriée