' requested

Version imprimable

Erreur : conversion from 'std::vector<Vehicule*>*' to non-scalar type 'std::vector<Vehicule*>' requested

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:

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:

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

Citation:

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

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:

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:

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

12/03/2018, 01h30
foetus
Citation:

Envoyé par koala01

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 ?

Citation:

Envoyé par koala01

ton constructeur devrait plutôt ressembler à quelque chose comme

Ou bien, si ton attribut est un pointeur :mrgreen:
Code:

1 2 3 4 5 Garage::Garage() :Vehicule(), vhcl_garage(new std::vector<Vehicule>()) { // ... }
12/03/2018, 10h41
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.
12/03/2018, 14h19
monsrhf
Citation:

Envoyé par foetus

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:

vector<Vehicule*> vhcl_garage;

il n'est pas de cette forme

Code:

std::vector<Vehicule> * vhcl_garage

donc correction pour mon code il est comme suite :
Code:

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?!
12/03/2018, 14h49
bacelar

Citation:

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.
12/03/2018, 15h55
monsrhf

oui je l'ai supprimer et elle m'indique deux erreurs :
Undefined reference to 'vtable for Garage'
error: ld returned 1 exit status
12/03/2018, 16h06
bacelar

Bon, je suis à peu près sûr que vous ne compilez pas le code que vous nous indiquez ou que vous oubliez de signaler d'autres erreurs en amont.

Déjà, vous l'avez défini et implémenter la méthode "nbrRoues" de la classe Garage ? (si vous trouvez cela stupide, relisez la réponse de @Bousk).

je n'ai pas mis tout le code car j'ai juger de n'afficher que le code concerné par l'erreur mais pour toute fin utile voilà le code complet :

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
#ifndef VEHICULE_H
#define VEHICULE_H
#include <vector>
using namespace std;
class Vehicule
{
    public:
    Vehicule();             ///constructeur par default
    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 Voiture: public Vehicule
{
public:
    Voiture(int , int );
    //Construit une voiture dont on fournit le prix et le nombre de portes
    virtual void affiche() const;
    virtual int nbrRoues();
    virtual ~Voiture();
 
    private:
    int m_portes;
 
};
class Moto : public Vehicule
{
    public:
    Moto(int , double );
    //Construit une moto d'un prix donné et ayant une certaine vitesse maximale
    virtual void affiche() const;
    virtual int nbrRoues();
    virtual ~Moto();
 
    private:
    double m_vitesse;
};
 
class Garage: public Vehicule
{
private:
    vector<Vehicule*> vhcl_garage;
public:
    Garage();
    void ajouter();
    void supprimer();
    virtual void affiche();
    ~Garage();
};
#endif // VEHICULE_H

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
#include "Vehicule.h"
#include <iostream>
#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 !
{}
 
Voiture::Voiture(int prix, int portes)
    :Vehicule(prix), m_portes(portes)
{}
 
void Voiture::affiche() const
{
    cout << "Ceci est une voiture avec " << m_portes << " portes et coutant " << m_prix << " euros." << endl;
}
 
int Voiture::nbrRoues()
{
    return 4;
}
 
Voiture::~Voiture()
{}
 
Moto::Moto(int prix, double vitesseMax)
    :Vehicule(prix), m_vitesse(vitesseMax)
{}
 
void Moto::affiche() const
{
    cout << "Ceci est une moto allant a " << m_vitesse << " km/h et coutant " << m_prix << " euros." << endl;
}
 
int Moto::nbrRoues()
{
    return 2;
}
 
Moto::~Moto()
{}
Garage::Garage()
:Vehicule()
{
}

12/03/2018, 16h56
bacelar

Citation:

Déjà, vous l'avez défini et implémenter la méthode "nbrRoues" de la classe Garage ?

Donc non, donc Garage est une classe abstraite, vous ne pouvez donc pas l'instancier.
12/03/2018, 17h09
monsrhf

C'est la classe Vehicule qui est absraite pas Garage. En plus le main ne contient rien je n'ai pas crée d'instance de la classe Garage. J'ai crée des objets Vehicule, alors qu'elle abstraite, mais ces objets sont des pointeurs. Donc c'est légitime de le faire, ou bien?!
12/03/2018, 17h46
bacelar

Citation:

C'est la classe Vehicule qui est absraite pas Garage.

"Garage" dérive de "Vehicule".
"Vehicule" est une classe abstraite car la méthode "int nbrRoues()" est virtuelle pure.
Toutes les classes qui dérivent de "Vehicule" et qui ne définissent pas "int nbrRoues()" sont aussi abstraites.
Donc "Garage" est abstraite. :weird:

Citation:

En plus le main ne contient rien je n'ai pas crée d'instance de la classe Garage.

Alors pourquoi le compilateur a besoin d'une vtable de "Garage" ? :weird:

Citation:

Undefined reference to 'vtable for Garage'

Donc montrez-nous votre "main" et les messages d'erreurs complets, numéros de ligne compris.

Citation:

J'ai crée des objets Vehicule, alors qu'elle abstraite

Non, vous avez créé des objets donc la classe est une classe fille "Vehicule".
Pointeur ou pas, cela ne change rien.

Citation:

mais ces objets sont des pointeurs.

Les pointeurs nus que vous utilisez (et les pointeurs nus, c'est pas pour les débutants) ne sont pas des objets, contrairement aux pointeurs intelligents, que je vous conseille chaudement d'utiliser à la place de ces pointeurs nus tout moisis.
23/03/2018, 11h10
EL-jos

Salut tout le monde,

il y a beaucoup des chose à modifier
1) On utilise jamais un espace de nom(using namespace) dans un fichier .h
2)L'utilisation de using namespace dans un fichier .cpp déprécier car cela est la cause des conflits de nom des variables
3)L'utilisation de pointeur nu(*pointeur) est déconseiller à la place utilise un unique_ptr<Vehicule> pointeur
4)L'utilisation de l'opérateur new est déconseiller, utilise un make_unique
5)Un garage n'est pas un Vehicule donc revoie ton cours sur l'héritage
6)Change de livre, trouve toi un livre sur C++ moderne car ton code c'est juste du C avec des classes( C with class )
Merci pour la compréhension