Constante statique a la classe

Bonjour,

Je suis en plein apprentissage sur la programmation c++ et aujourd'hui j'ai un soucis que je vous soumet:
En effet, j'ai crée une classe Commercial qui hérite de la classe Employé.
Et 2 autres classes Vendeur et Représentant qui héritent de la classe Commercial et j'ai la méthode double calculer_salaire() const
Cette méthode doit me retourner:

Pour le vendeur: (0.2*chiffre d'affaire) + 400
Pour le représentant: (0.2*chiffre d'affaire) + 800

Ma question est de savoir comment faire pour ne pas avoir ces valeurs de 400 et 800 en dur(qui correspondent à la majoration). J'ai pensé pour cela a définir une variable statique dans les classes vendeur et représentant:

Vendeur.hpp:

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 25 26 27 28

#ifndef VENDEUR_HPP #define VENDEUR_HPP   #include "Commercial.hpp" #include <iostream> #include <string> using namespace std;   class Vendeur : public Commercial { public: Vendeur(string nom, string prenom, double age, unsigned int date, double CA) : Commercial(nom, prenom, age, date, CA) {} string get_nom() const; double calculer_salaire(); ~Vendeur() {} private: static double majoration; };   #endif     double Vendeur::majoration(400);   double Vendeur::calculer_salaire() { return (.2 * chiffre_affaire) + majoration; }

---

est-ce la bonne façon sachant que je fais la même chose pour la classe Représentant ou alors faut il plutot la définir dans la classe Commercial?

Définis ta valeur comme un membre constant de ta classe Commercial, et initialise là via le constructeur de Commercial.
Les constructeurs de Vendeur et Représentant (quitte à utiliser du français, tu peux utiliser des accents dans tes noms de classe... ça peut être un sujet d'amusement :)) appelleront le constructeur de leur parent en donnant les valeurs littérales (400 et 800).

Ok mais par contre je pensé qu'on devais utiliser une variable statique pour ce genre de manoeuvre ou cela n'est pas utile?

Le seul problème de la variable statique dans ton cas c'est que ça revient au même que de mettre la valeur "en dur" dans le code :mrgreen:

Ok donc je doit definir une classe de ce type

Code:

---

1 2 3 4 5 6 7 8 9 10

class Commercial{ public: Commercial(........, const int var) : ......., var(var) {} ....... protected: const int var;   };

---

Pour moi c'est ok.

Il ne sert à rien de déclarer const un argument passé par copie.

Citation:

---

Envoyé par oodini

Il ne sert à rien de déclarer const un argument passé par copie.

---

Ça peut avoir un intérêt en terme d'implémentation : On est certain de ne pas le modifier dans l'implémentation. Mais ça n'a effectivement aucun intérêt pour le code appelant. D'ailleurs, on peut déclarer une fonction int f(int i); et l'implémenter int f(int const i);, il s'agit de la même fonction pour le compilateur.