Discussion :

[monsrhf]

je travail sur un exercice qui gère les véhicules d'un garage. J'ai une classe Vehicule et une classe Garage leurs prototype .h est :

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 VEHICULE_H
#define VEHICULE_H
#include <vector>
using namespace std;
class Vehicule
{
    public:
    Vehicule();
    Vehicule(int );           //Construit un véhicule d'un certain prix
    virtual void affiche() const;
    virtual int nbrRoues()=0;
    virtual ~Vehicule();          //Remarquez le 'virtual' ici
 
    protected:
    int m_prix;
};
class Garage: public Vehicule
{
private:
    vector<Vehicule*> vhcl_garage;
public:
    Garage();
    void ajouter();
    void supprimer();
    virtual void affiche();
    ~Garage();
};
#endif // VEHICULE_H

l'erreur est indiqué dans le fichier.cpp qui est comme suit :

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
#include "Vehicule.h"
#include <iostream>
#include <cstdlib>
#include <vector>
using namespace std;
 
Vehicule::Vehicule()
{}
Vehicule::Vehicule(int prix)
    :m_prix(prix)
{}
 
void Vehicule::affiche() const
//J'en profite pour modifier un peu les fonctions d'affichage
{
    cout << "Ceci est un vehicule coutant " << m_prix << " euros." << endl;
}
 
Vehicule::~Vehicule() //Même si le destructeur ne fait rien, on doit le mettre !
{}
Garage::Garage()
:Vehicule()
{
    std::vector<Vehicule>* vhcl_garage=new std::vector<Vehicule>;
}

Je voudrais savoir où est le problème ?!

[koala01]

Salut,

Attention, std::vector<Vehicule*>, ca correspond à un tableau de pointeurs sur véhicules; std::vector<Vehicule>* ca correspond à un pointeur sur un tableau de véhicules, ce n'est pas du tout pareil!

Dans le premier cas, tu dois créer tes véhicules en ayant recours à l'allocation dynamique de la mémoire, mais le tableau est dans la pile de ta classe, dans le second cas, c'est le tableau que tu crées en ayant recours à l'allocation dynamique de la mémoire, mais tu crées "normalement" les véhicules qu'ils contient...

De plus, même en déclarant vhcl_garage comme étant un std::vector<Vehicule> * le code de ton constructeur serait faux, car, lorsque tu écris

Code :

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

1
2
3
4
5
Garage::Garage()
:Vehicule()
{
    std::vector<Vehicule>* vhcl_garage=new std::vector<Vehicule>;
}

tu déclares, à l'intérieur de ton constructeur, une variable locale (à ton construteur) qui s'appelle vhcl_garage et qui, pas de bol, va cacher durant toute la durée de l'exécution du constructeur la donnée membre qui porte le même nom.

Mais le résultat, c'est que toutes les actions qui pourraient être entreprises au niveau du constructeurs seront entreprises sur la variable locale, et que la donnée membre de ta classe ne serait en aucun cas affectée par ces modifications.

Si donc, tu déclare la donnée membre de ton garage comme étant std::vector<Vehicule> * vhcl_garage (ce que je te déconseille), ton constructeur devrait plutôt ressembler à quelque chose comme

Code :

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

1
2
3
4
5
6
7
8
Garage::Garage()
:Vehicule()
{
    /* nul besoin de DECLARER une variable ici, vu qu'on a déjà une donnée membre
     * qui fait l'affaire... il suffit de DEFINIR une valeur correcte pour cette variable
     */
    vhcl_garage=new std::vector<Vehicule>;
}

[foetus]

comme étant std::vector<Vehicule> * vhcl_garage (ce que je te déconseille)

C'est plutôt quel est l'utilité de déclarer un pointeur d'un tableau dynamique ?

ton constructeur devrait plutôt ressembler à quelque chose comme

Ou bien, si ton attribut est un pointeur

Code :

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

1
2
3
4
5
Garage::Garage()
:Vehicule(), vhcl_garage(new std::vector<Vehicule>())
{
// ...
}

[Bousk]

Y'a déjà un autre problème qui saute aux yeux : class Garage: public Vehicule, cet héritage n'a absolument aucun sens.

[monsrhf]

C'est plutôt quel est l'utilité de déclarer un pointeur d'un tableau dynamique ?

c'était juste une faute de frappe. mon attribut est justement comme suit

Code :

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

vector<Vehicule*> vhcl_garage;

il n'est pas de cette forme

Code :

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

std::vector<Vehicule> * vhcl_garage

donc correction pour mon code il est comme suite :

Code :

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

1
2
3
4
5
Garage::Garage()
:Vehicule()
{
    vector<Vehicule*>vhcl_garage=new std::vector<Vehicule*>;
}

je ne sais pas pourquoi il m'indique l'erreur déjà cité en haut?!

[bacelar]

je ne sais pas pourquoi il m'indique l'erreur déjà cité en haut?!

Vous n'avez absolument compris les réponses de mes voisins du dessus.
Supprimez la ligne car elle ne sert à rien et elle est fausse.

[monsrhf]

oui je l'ai supprimer et elle m'indique deux erreurs :
Undefined reference to 'vtable for Garage'
error: ld returned 1 exit status