Discussion :

[kali38]

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 :

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

[oodini]

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

[kali38]

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

[SkyZoThreaD]

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

[kali38]

Ok donc je doit definir une classe de ce type

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
class Commercial{
public:
Commercial(........, const int var) : ......., var(var) {}
.......
protected:
const int var;
 
};

[SkyZoThreaD]

Pour moi c'est ok.

[oodini]

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

[JolyLoic]

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.