Discussion :

[emmesse]

Bonjour,

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

Code cpp :

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
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

[dalfab]

Bonjour,

C'est plutôt cohérent.

Juste la séquence:

Code :

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

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 :

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

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.)

[emmesse]

Oui en effet, ça fonctionne aussi.
Merci pour la suggestion

[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.