Discussion :

[Balyn]

Bonsoir, voici mon soucis, j'essaye d'utiliser une classe CollectionEmploye qui contient un tableau d'employe (vector). Le constructeur doit construire un vector vide et la méthode ajouterEmployé ajouter un Employé.


Voici mon collection.h

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
#ifndef COLLECTION_H
#define COLLECTION_H
#include "projetEmploye.h"
#include "Commercial.h"
#include "Manutentionnaire.h"
#include "Vendeur.h"
#include "Representant.h"
#include "Technicien.h"
#include <vector>
 
class CollectionEmploye  {
   protected :
   vector<Employe*> listeEmploye;
 
   public :
   CollectionEmploye();
   void ajouterEmploye(Employe &e);
}

ainsi que le cpp qui va avec

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#include "Collection.h"
 
CollectionEmploye::CollectionEmploye(){
 
listeEmploye = new vector<Employe*> listeEmploye;
 
}
 
void ajouterEmploye(Employe &e) {
 
listeEmploye.push_back(&e);
 
}

Est-ce que quelqu'un peut m'expliquer d'où vient mon erreur? Je débute avec les vector et en c++ et suis un peu perdu. Merci beaucoup.

[bacelar]

Vous avez vu les pointeurs nus avant les std::vector ?

"std::vector" c'est la base de tout code C++ moderne et les pointeurs nus, c'est un sujet "avancées".
Vous avez du apprendre avec un cours "historique" qui pue la naphtaline.

Vous avez de la chance, vous n'avez qu'à reprendre le début d'un cours moderne sur le C++ qui vous expliquera très top comment vous servir de std::vector et des pointeurs intelligents, et aussi la notion d'ownership.

Parce que votre code, au niveau "ownership", c'est une catastrophe.

[Bousk]

Qu'est sencé vouloir dire ceci ? listeEmploye = new vector<Employe*> listeEmploye; ?
- Syntaxe inexplicable et totalement fausse
- Utilisation de new alors qu'il ne s'agit pas d'un pointeur

[Balyn]

Non, je n'avais jusqu'à il y a 15 minutes aucune idée de ce que pouvait être un pointeur nu.
Concernant le constructeur

Code :

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

listeEmploye = new vector<Employe*> listeEmploye;

, je voudrais initialiser un vector vide afin de le remplir par la suite avec la méthode ajouter.
Je comprends bien que ce n'est pas du tout comme ça que je suis sensé faire mais ça ne m'avance pas beaucoup plus de le savoir.

[bacelar]

Il faut absolument que vous ayez les bases avant de commencer à faire des choses non triviales.
Et "l'ownership", c'est la base de la base.
Votre code montre que vous n'avez pas la moindre idée de ce concept.
Reprenez un simple cours d'introduction au C++ moderne.

On ne peut pas corriger votre code car la gestion de l'ownership n'apparait pas dans votre code.
C'est l'une des faiblesses majeures du vieux code C++, il fallait avoir tout le code de la classe ET des codes utilisateur de cette classe pour la (l'ownership) déterminer (ou correctement la documenter).

Si le "CollectionEmploye" est le propriétaire des éléments et que Employe est une classe "valeur" (et non entité), ce qui est peu probable (mais ça me simplifie le 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
class CollectionEmploye  {
   protected :
   vector<Employe> listeEmploye;
 
   public :
   CollectionEmploye(){};
   void ajouterEmploye(Employe e);
}
 
void ajouterEmploye(Employe e) {
 
listeEmploye.push_back(e);
 
}