ai-je eu raison de faire ceci?

Version imprimable

Bonjour,

est-ce ainsi que l'on surcharge l'opérateur>> ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
class nombre{
   nombre& entree(std::istream &flux);
};
 
std::istream& operator>> (std::istream& flux, nombre &nb);
 
nombre::nombre(std::string const &valeur_init){
...
}
 
nombre& nombre::entree(std::istream &flux){
  std::string nombre_str;
  flux>>nombre_str;
  nombre ceci(nombre_str);
  *this=ceci;
  *this=this->normalise();
  return *this;
}
 
 
std::istream& operator>>(std::istream& flux, nombre &nb){
  nb.entree(flux);
  return flux;
}

je suis preneur de toute suggestion

Bonjour,

C'est plutôt cohérent.

Juste la séquence:
Code:

1 2 3 4 nombre ceci(nombre_str); *this=ceci; *this=this->normalise(); return *this;
pique un peu les yeux. La normalisation devrait être appelée dès la construction, et il ne resterait alors que: return *this = nombre(nombre_str);.

Autre possibilité: On peut raccourcir en définissant l'opérateur>> en une fonction friend et inline, plutôt qu'en 2 fonctions. Ça donne:
Code:

1 2 3 4 5 6 7 8 9 10 class nombre{ public: friend std::istream& operator>>( std::istream& flux, nombre& nb ) { std::string nombre_str; flux >> nombre_str; // n'est peut-être pas si simple (on suppose que le texte qui suit est séparé par un caractère non imprimable.) nb = nombre{ nombre_str }; // nb.normalise(); appelé par le constructeur (après création, le 'nombre' est directement opérationnel) return flux; } };
Attention, le corps de la fonction doit être dans la class nombre sinon la fonction amie risque de ne pas être retrouvée par le compilateur (c'est dans des cas complexes mais autant prendre cette habitude.)

05/12/2024, 15h40
emmesse

Oui en effet, ça fonctionne aussi.
Merci pour la suggestion
05/12/2024, 16h23
deedolith

Salut,

A mon avis, par respect du SRP (https://en.wikipedia.org/wiki/Single...lity_principle), ta classe représente une nombre, la gestion des entrées n'est pas de sa responsabilité, cette fonctionnalité doit être faite ailleur.