Discussion :

[alias2015_29]

Bonjour à tous,

Voici code :

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
 
#include <iostream>
#include <string.h>
using namespace std;
 
class Point  //Déclaration Classe
{
     private: //Attribut
     int x,y;
     char couleur[10];
 
     public: //méthodes + Constructeur + Destructeur
        Point();
        ~Point();
        //surcharge d'opérateur + & = & <<
        Point operator+(const Point &);
        Point operator=(const Point &);
        Point operator<<(const Point &);
 
};
 
Point Point::operator<<(const Point &P)
{
   cout << "Point = (" << x <<" , "<< y <<" ) "<< couleur << endl;
return (x,y);
}

est ce que c'est juste ou faux??

[bacelar]

42

Manque l'implémentation des opérateurs + et =.

Mise en page pas cohérente : le public devant les champs, séparation header/cpp => "using namespace" qui faut pas .

L'opérateur << méthode de la classe, c'est une connerie, revoyez le chapitre sur les flux et en particulier la signature de ce type d'opérateur.

...

[ternel]

As-tu seulement essayé de le compiler, ce code?

[alias2015_29]

oui j'ai essayé de compilé il m'affiche erreur

j'ai pas su comment le corriger

En regardant le FAQ dans ce Forum:

j'ai corrigé ma faute

Code :

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

1
2
friend  ostream& operator<<(ostream&,const Point &);
      friend  istream& operator>>(istream&,const Point &);

et implémentant les deux méthodes

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
ostream& operator<<(ostream& os, const Point& p)
{
      os << p.x << ' ' << p.y << '\n'
           << p.couleur ;
      return os;
}
 
istream& operator>>(istream& ist, const Point& p)
{
      ist >> p.x >> p.y;
      return ist;
}

En compilant il affiche

[Bousk]

Tu essayes de modifier un paramètre passé par const référence.
Tu as bien copié/collé la version <<, pas >> ..

[Médinoc]

Franchement, je ne vois pas l'utilité de les mettre en friend, il suffit de déclarer des fonctions membres appropriées:

Code C++ :

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
//Point.hpp
#include <iostream>
 
class Point  //Déclaration Classe
{
private: //Attribut
	int x,y;
	char couleur[10];
 
public: //méthodes + Constructeur + Destructeur
	Point();
	~Point();
	//surcharge d'opérateur + & = & <<
	Point operator+(const Point &) const;
	Point operator=(const Point &);
 
	void output(std::ostream&) const;
	void input(std::istream&);
};
 
std::ostream& operator<<(std::ostream&,const Point &);
std::istream& operator>>(std::istream&, Point &); //pas de const ici

Code C++ :

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
//Point.cpp
#include "Point.hpp"
using namespace std;
 
void Point::output(ostream& os) const
{
	os << x << ' ' << y << '\n'
		<< couleur ;
	return os;
}
 
void Point::input(istream& ist)
{
	ist >> x >> y;
}
 
ostream& operator<<(ostream& os, const Point& p)
{
	p.output(os);
	return os;
}
 
istream& operator>>(istream& ist, Point& p)
{
	p.input(os);
	return ist;
}

[ternel]

c'est marrant, mais j'utilise souvent les opérateurs inversés comme membres.

Code :

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

1
2
3
4
5
6
7
8
9
 
class Bidule {
public:
	std::ostream& operator>>(std::ostream& os) const {return os << "Bidule";}
	std::istream& operator<<(std::istream& is) {/*ce que tu veux*/; return is;}
};
 
std::ostream& operator<<(std::ostream& os, Bidule const& b) {return b>> os;}
std::istream& operator>>(std::istream& is, Bidule & b) {return b << is;}

Attention toute fois qu'initialiser depuis la console est souvent un piège, car il faut avoir créé la variable avant.
Ca signifie que l'objet est construit dans un état par défaut avant d'être reconstruit.

[alias2015_29]

est ce que c'est obligatoire de mettre std:: ???

[Médinoc]

Dans le fichier d'en-tête, oui, parce qu'il ne faut pas mettre de using namespace dans un fichier d'en-tête.

[ternel]

En effet, un using provoque de la pollution arbitraire: chaque #include contiendra le using, qui peut interférer avec d'autres, ou avec le code de l'utilisateur.

Un using dans un en-tête annule tous les bénéfices des namespace

Il y a une entrée dans la faq sur le sujet, mais je ne l'ai plus sous la main

[alias2015_29]

j'ai ajouté cette ligne pour surcharger opérateur de lecteur

Code :

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

 friend  istream& operator>>(istream& ,const Point &);

son code est :

Code :

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

1
2
3
4
5
istream& operator>>(istream& ist, const Point& p)
{
      ist >> p.x >> p.y;
      return ist;
}

il m'affiche cette erreur

[Médinoc]

Réfléchis un peu et dis-moi ce qui est mauvais dans: istream& operator>>(istream&, const Point &)

[alias2015_29]

pour Cin >> on doit pas mettre le mot const ???

[Médinoc]

La réponse à cette question est évidente si tu te rappelles à quoi sert le cin >>: Il sert à modifier l'objet.

[Bousk]

http://cpp.developpez.com/faq/cpp/?p...sir-mes-objets