Discussion :

[rusol]

Bonjour,

je cherche à overloader l'opérateur >>

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
struct Point
{
   int x, y;
 // methodes ...
};
struct Tablepoint
{
   // methodes ....
   //attributs
   vector<Point> table;
  //
};
istringstream tmp4 ( "(0,1) < (1,0) " );

Je cherche à overloader l'opérateur >> pour pouvoir rajouter des points dans mon vecteur de point. Mais je dois d'abord vérifier que les points soient valides (ex: pas de valeur négatives, inférieurs à 10) et ensuite si tous les points du string sont valide, je les ajoute dans le tableau.

Code :

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

1
2
3
4
5
std::istream& operator >>(std::istream &is, Tablepoint &t)
{
//
return is;
}

Mais je ne vois pas comment écrire le code.

[darkman19320]

Salut,

Je pense qu'une bonne idée serait de décomposer ton problème en plusieurs étapes:
- lire tes points,
- de les comparer,
- faire tes tests
- et enfin de les ajouter ou non à ton tableau.

[rusol]

Merci de ta réponse
J'ai déjà codé mes fonctions de test, mais je n'arrive pas à overloadé l'opérateur...

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
    std::istream& operator>>(std::istream& is, Tablepoint c)
    {
    string str =is.str();
    vector<Point> v;
    bool test=testeur(str, v)
    if (test)
    {
      for (int i=0;i<v.size();i++)
        c. Add(v[i]);
    }
    /*else
    {
      failbit...
    }*/
    return is;
    }

error: ‘std::istream& Tablepoint::operator>>(std::istream&, Tablepoint)’ must take exactly one argument

[Bousk]

http://en.wikipedia.org/wiki/Operators_in_C_and_C%2B%2B

Tel que je vois l'erreur, que tu aurais pu fournir dès le début, ton opérateur >> est membre de la classe.
Alors la syntaxe est operator>>(istream&); et non operator>>(istream&, class&);, on a déjà le this, c'est redondant.

[rusol]

Merci beaucoup

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 std::istream& operator>>(std::istream & is)
    {
    string str;
    is>>str;
    vector<Point> v;
    cout<<str<<endl;
    bool test=testeur(str, v);
    if (test)
    {
      for (int i=0;i<v.size();i++)
        table.push_back(v[i]);
    }
    return is;
    }

[Médinoc]

http://en.wikipedia.org/wiki/Operators_in_C_and_C%2B%2B

Tel que je vois l'erreur, que tu aurais pu fournir dès le début, ton opérateur >> est membre de la classe.
Alors la syntaxe est operator>>(istream&); et non operator>>(istream&, class&);, on a déjà le this, c'est redondant.

Euh, c'est le contraire de ce que je vois dans le tableau de ton lien (R K::operator <<(S b); pour R operator <<(K a, S b);).
Ça me paraît bizarre aussi, car c'est contraire à tous les autres opérateurs (où la classe représente l'opérande de gauche). Est-ce un cas spécial pour les opérateurs de décalage de bits, le fait que la classe représente l'opérande de droite?

[Bousk]

Non le lien est correct (je lui fais toute confiance^^), ma réponse un peu rapide et son erreur double

[Médinoc]

En fait, ça voudrait dire qu'on ne peut pas surcharger ostream<< et istream>> à l'intérieur d'une classe.

Personnellement, je fais toujours ainsi:

Code C++ :

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

1
2
3
4
5
6
7
class UneClasse
{
public:
	writeTo(ostream&) const;
};
 
inline ostream& operator<<(ostream& os, UneClasse const&obj) { obj.writeTo(os); return os; }

Et pareil pour la lecture d'un istream si j'ai besoin que mon objet puisse être ainsi lu.