Discussion :

[rdtech]

bonjour,
je me suis inspiré d'exemples dans la doc boost serialization pour coder en c++.
J'ai donc une classe bus_route

Code :

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

1
2
3
4
5
6
7
8
9
 
class bus_route
{
private:
    bus_stop* stops[10];
public:
    bus_route(){}
    bus_stop* operator[](int);
};

J'instancie un objet de cette classe:

Code :

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

1
2
 
bus_route* ligne2a = new bus_route;

Et j'aimerai instancier des objets de la classe bus_stop pour les stocker dans le tableau de pointeurs correspondant dans l'objet ligne2a en écrivant:

Code :

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

1
2
3
 
bus_stop bs1(g10, g11);    //g10 et g11 sont des coordonnées
*(ligne2a[0]) = &bs;

J'ai donc surcharger l'opérateur [ ] dans bus_route de cette manière:

Code :

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

1
2
3
4
5
 
bus_stop** bus_route :: operator[](int index)
{
    return &stops[index];    //retourne l'@ du pointeur au rang index
}

En écrivant *(ligne2a[0]) = &bs; voire (*ligne2a)[0] = &bs; je pensais affecter le contenu de l'adresse pointé et donc le pointeur. A priori, je n'ai pas à surcharger l'opérateur d'affectation puisque je ne retourne pas un type classe.

Je prend toutes les idées.
Merci.

[Iradrille]

Hello,

Tout d'abord, tu devrais utiliser un vector à la place d'un tableau de pointeur.
Ça simplifie le code et ça t'autorises à avoir un nombre quelconque d'arrets (plus de limite à 10).

Code :

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

1
2
bus_stop bs1(g10, g11);    //g10 et g11 sont des coordonnées
*(ligne2a[0]) = &bs;

Fais attention avec ce genre de code, tu stock l'adresse d'un objet bus_stop, mais quand l'objet sera supprimé, l'adresse stockée sera invalide.
ex:

Code :

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

1
2
3
4
if(/*...*/) {
   bus_stop bs1(g10, g11);    //g10 et g11 sont des coordonnées
   *(ligne2a[0]) = &bs1;
} // suppression de bs1

Pour l'operator[], retourner une référence est préférable à un pointeur sur pointeur.

Tu devrais avoir un truc dans le genre.
Si tu dois partager des arrêts entre plusieurs lignes, utilise un vector<std::shared_ptr<bus_stop>>, vaut mieux éviter les pointeurs nus.
Comme ça c'est la ou les lignes qui contiennent l'arrêt qui sont responsables de sa durée de vie.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class bus_route {
    std::vector<bus_stop> stops;
 
public:
    bus_route(){}
 
    void add(const bus_stop& bs) { stops.push_back(bs); }
 
    bus_stop& operator[](size_t ind) { return stops.at(ind); }
    const bus_stop& operator[](size_t ind) const { return stops.at(ind); }
};

[rdtech]

Merci beaucoup,
remarques très intéressantes, spécialement sur la partage des arrêts.
J'avais gardé le tableau de pointeurs comme dans le source original mais ça me compliquait la vie plus qu'autre chose...

Cordialement.