C++, propriété statique

Je rencontre des problèmes lors de l'accès à une propriété statique dans une classe. Je reçois l'erreur suivante:C++, propriété statique

shape.obj : error LNK2001: unresolved external symbol "public: static class TCollection<class Shape *> Shape::shapes"

La définition de la classe est:

class Shape { 

public: 
    static Collection<Shape*> shapes; 

    static void get_all_instances(Collection<Shape*> &list); 
};

et la mise en œuvre de la méthode statique étant:

void Shape::get_all_instances(Collection<Shape*> &list) { 
    list = Shape::shapes; 
}

Il semble que le La propriété shapes n'est pas en cours d'initialisation.

Source

2010-10-24 Louis

Wow merci pour le retour rapide. J'ai pris des heures pour faire de la recherche et environ 1 minute sur StackOverflow. – Louis

Je pense que vous regretterez beaucoup d'avoir à la fois une variable statique et une variable publique. As-tu réellement besoin de cela? Pourquoi ne pas simplement passer une const Collection & objet partout où vous avez besoin d'une liste de toutes les formes? Cela encouragera les autres développeurs à appeler Shape :: get_all_instances() dès qu'ils auront besoin de la liste, au lieu de la passer explicitement. Cela conduira à toutes sortes de problèmes, lorsque vous voulez tester ou que vous voulez travailler sur des sous-formes de formes. –

Ouais c'est public en ce moment à cause d'un débogage furieux. – Louis

Vous avez raison puisque la variable statique ne sont déclarés dans la classe et non définie.

devez les définir aussi, ajoutez simplement la ligne suivante dans le fichier où se trouve votre implémentation.

Collection<Shape*> Shape::shapes;

Et ça devrait faire l'affaire.

Source

2010-10-24 07:56:13 Keynslug

Oui. Vous devez ajouter

Collection<Shape*> Shape::shapes;

dans l'un des fichiers .cpp pour définir le membre statique.

Source

2010-10-24 07:53:47 kennytm

la déclaration est dans la classe.

la définition doit être placé dans exactement un fichier cpp:

Collection<Shape*> Shape::shapes;

Source

2010-10-24 07:54:52 justin

Vous avez déclaré shapes mais n'ont pas définir.

Ajouter la définition dans le fichier de mise en œuvre

Collection<Shape*> Shape::shapes; //definition

Source

2010-10-24 07:54:59

Pour le code tel quel, vous devez fournir une définition de shapes, comme (dans un fichier de mise en œuvre)

Collection<Shape*> Shape::shapes(whatever constructor args);

Mais au contraire, vous voudrez peut-être envisager une fonction membre qui retourne une référence à une locale statique Collection<Shape*>.

Salutations & hth.

Source

2010-10-24 07:56:33

Répondre

Questions connexes