Ambigu de la base de l'héritage multiple

Je suis en train d'écrire certaines sous-classes des classes dans une grande bibliothèque. Je suis un ambiguë "de base" de l'erreur. Voici une compilable exemple du problème:

#include <iostream>

//I can't change these because they are in the library:
class InteractorStyle {};
class InteractorStyleCamera : public InteractorStyle {};
class InteractorStyleImage : public InteractorStyle {};

//These are my subclasses (so I can change them):
class PointSelector : public InteractorStyle {};
class PointSelector2D : public InteractorStyleCamera, public PointSelector
{
  //This function has to exist exactly like this (a requirement of the library):
  static PointSelector2D* SafeDownCast(InteractorStyle *o)
  {
    return static_cast<PointSelector2D *>(o);
  } 
};


int main()
{

  return 0;
}

L'erreur est

erreur: ‘InteractorStyle " est ambigu dans la base de ‘PointSelector2D’.

Est-ce que je peux faire dans ce cas?

  • Si vous le savez, l'erreur est parce que les deux InteractorStyleCamera et PointSelector sont dérivées à partir de InteractorStyle. Désolé de ne pas aider beaucoup de bien 🙂
  • Est-il possible de faire PointSelector pas hériter?
InformationsquelleAutor David Doria | 2011-08-25
  1. 17

    Votre problème, c'est que Interacteur style est héritée par deux fois - une fois par PointSelector2D et une fois par InteractorStyleCamera. Cela signifie que vous avez 2 versions de chaque membre de contenus au sein de votre classe.

    Découvrez:

    Comment puis-je éviter le Diamant de la Mort lors de l'utilisation de l'héritage multiple?

    Et essayer de l'héritage virtuel.

    • Je pense que le problème est que la bonne façon d'ajouter de l'héritage virtuel nécessite l'OP modifier le code qu'il ne peut pas le modifier. C'est dommage, parce que le inmodifiable code doit certainement être l'aide d'héritage virtuel.
    • Ah, je pensais qu'il avait la bibliothèque de la source à jouer avec - si il ne veut pas qu'il pourrait être désossées :p Pourriez avoir à aller de contourner le problème et d'essayer d'obtenir les mêmes résultats avec le confinement si c'est le cas.
    • Yep, templatetypedef est bon - j'ai été poster ici pour voir si il y avait d'autres options.
    • w00te - c'est la solution pour garder un objet de PointSelector à l'intérieur de la PointSelector2D classe? Qui semble si difficile, n'est-ce pas?
    • Elle est très maladroite, j'en conviens. J'espère que vous trouverez un moyen d'accéder à chercher à compiler, mais si vous ne pouvez pas confinement est le seul moyen de maintenir la fonctionnalité et de l'étendre. C'est pourquoi des choses comme le TSL a vous faire conteneur des adaptateurs (c'est à dire de la file d'attente) par contenant des conteneurs STL comme deque. Je suis complètement d'accord ses pas quelque chose que vous voulez faire, sauf si vous vous ennuyez si.
    InformationsquelleAutor John Humphreys - w00te
  2. 5

    Vous pouvez superficiellement "résoudre" il par l'utilisation d'un deux-étape de fonte. Par exemple

    static_cast<PointSelector2D *>(static_cast<InteractorStyleCamera *>(o));

    Bien sûr, vous devez garder à l'esprit que cette "corrige" la syntaxe, mais ne résout pas le problème structurel sous-jacent. Votre PointSelector2D a deux InteractorStyle de la base de sous-objets à l'intérieur. En fonction de ce qui InteractorStyle de la base de sous-objets de commencer, la sortie chemin est différent. Et il est très important de prendre le chemin d'accès correct. Ce que j'ai écrit ci-dessus est pour InteractorStyle à l'intérieur de InteractorStyleCamera. Pour les autres la base de la bonne sortie serait

    static_cast<PointSelector2D *>(static_cast<PointSelector *>(o));

    Si vous êtes donné un InteractorStyle * pointeur avec aucune des informations supplémentaires au sujet de base c'est pointant vers, alors vous ne pouvez peut-être résoudre votre problème avec static_cast. Il n'y a aucun moyen de savoir qui sortie chemin à prendre. En prenant le faux on va produire un totalement dépourvue de sens résultat.

    Comme il a été noté précédemment, dynamic_cast peuvent aider dans cette situation, mais il a des exigences supplémentaires (polymorphe de départ type). Vos types ne sont pas polymorphes (au moins dans votre citée en exemple), de sorte dynamic_cast ne les accepteront pas pour upcasts.

    InformationsquelleAutor AnT
  3. 3

    Je crois que vous pouvez résoudre ce problème en utilisant un dynamic_cast au lieu d'un static_cast. Le dynamic_cast pouvez regarder l'objet lors de l'exécution pour déterminer lequel des deux InteractorStyle classes de base est en cours pointé, et à partir de là peut ajuster le pointeur vers le bas pour le type approprié.

    • Vous devriez être en mesure de mettre librement en fonte de type pointeur à droite? Je lance un vector<string>* à un int* alors pourquoi static_cast être le problème?
    • Malheureusement cette fonction est beaucoup plus macro qui est nécessaire pour la bibliothèque, je suis aide à comprendre que c'est en effet l'un de ses objets. Est-il autre chose que je peux faire qui n'implique pas de changer de fonction?
    • Carnegie- static_cast ne fonctionne que si il y a une ambiguïté abattu à exécuter à partir du type de base pour le type dérivé. Dans ce cas, il y a deux objets de base de type InteractorStyle - l'un de PointSelector et un de InteractorStyleCamera - de sorte que le compilateur ne peut pas savoir au moment de la compilation lequel des deux objets est en cours pointé, et donc il ne peut pas ajuster le pointeur sur la base de l'objet correct. À l'aide d'un style C cast ici est dangereux parce qu'il ne serait pas ajuster le pointeur. Seulement dynamic_cast pouvez utiliser les informations d'exécution afin de déterminer comment ajuster le pointeur.
    • ah merci, je pensais que static_cast était essentiellement un style C cast.
    InformationsquelleAutor templatetypedef
  4. 0

    Vous pouvez obtenir votre code à compiler (j'ai reçu votre exemple pour compiler au moins) par l'ajout de InteractorStyle pour les classes qui PointSelector2D hérite. Si c'est une solution à long terme n'est pas connu par moi.

    • Tu veux dire le modifier: classe PointSelector2D : public InteractorStyleCamera, public PointSelector, public InteractorStyle ? n'est toujours pas compiler pour moi.
    • bizarre, essayez de mettre InteractorStyle en premier dans la liste. Autre que cela, je suppose que ça doit être mon compilateur. Lequel utilisez-vous? Aussi êtes-vous de la compilation de votre code d'exemple ou le vrai projet?
    • nope, même le premier de la liste, il ne fonctionne pas. J'utilise g++ 4.5 et la compilation de l'exemple de code que j'ai posté ici.
    InformationsquelleAutor Seth Carnegie