Discussion :

[eric_marciano]

Je suis en train de m'arracher les cheveux à comprendre le code d'un autre développeur.
Il a définit une classe ou tout est privé, si bien qu'on ne peut rien à faire, ni l'instancier, ni en appeler une méthode. Je ne vois pas bien l'intérêt d'une telle classe... Il semble bien qu'il s'agisse d'un singleton, géré par un "smart pointer" qui compte le nombre de références.

Pour être plus clair, voici ce que j'ai:

1) La classe de pointeur intelligent (que j'ai réussi à comprendre)

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
 
template <class T> class SmartPtr
{
public:
	SmartPtr();
	SmartPtr(T* ptr);
	SmartPtr(const SmartPtr<T>& reference);
	~SmartPtr();
 
	T* operator->() const;
	T& operator*() const;
	operator const T*() const;
	operator T*() const;
	operator const void*() const;
	operator void*() const;
 
	SmartPtr<T>& operator=(const SmartPtr<T>& reference)
	SmartPtr<T>& operator=(T* ptr);
 
	// Manipulation du compteur de références
	SINT32 AddReference();
	SINT32 Release();
	SINT32 GetReferenceCount() const;
 
 
protected:
	T* myPointer;
};

2) La fameuse classe entièrement privée (simplifiée bien sûr)

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
 
class MyClass
{
	static void* operator new(unsigned int);
	static void operator delete(void*);
	static class SmartPtr<MyClass> New();
 
	int AddReference();
	int Release();
	int GetReferenceCount();
	//
	// Autres méthodes...
	//
	MyClass(class MyClass&);
	MyClass();
 
	virtual ~MyClass();
	virtual class SmartPtr<MyClass> Clone();
};

Comme ni public, ni protected, ni private n'est précisé, c'est "private" qui est pris par défaut.
Il y a bien une méthode static qui semble être là pour créer une nouvelle instance, mais comme elle est privée, elle est inaccessible.

Par ailleurs, j'ai accès de façon globale à une fonction:

Code :

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

1
2
 
class SmartPtr<MyClass>  MyClass_New();

Mais là encore, je ne peux pas l'utiliser car le compilo n'est pas content, il dit que le destructeur de "MyClass" n'est pas accessible.

Donc je nage... Si quelqu'un peut m'expliquer comment instancier "MyClass", ou mieux, comment exploiter l'instance existante, je suis preneur.

Merci !!
Eric

[AurelienTran]

Ca a aucune intérêt car comme tu le dis, on ne peut pas l'utilisée. C'est à se demander comment une telle classe a peu être tout au moins tester...

Sauf si tout est gérer par le constructeur et le destructeur??? faudrait voir l'implémentation pour être sur.

EDIT: constructeur et destructeur est aussi private donc ca ne peux pas compiler non plus. doit juste manquer le mot clé public quelque part.

[JolyLoic]

Il n'y a pas un "friend" quelque part ?

[eric_marciano]

Aurélien,
C'est bien ce que je pensais, il doit bien manquer un public quelque part, et pourtant ça compile. Mais je n'ai que les .h, je n'ai pas le code source complet.

Loic,
Ta question est intéressante. Je ne vois de friend nul part (j'ai pourtant cherché...) Mais est-ce que je pourrais créer une classe friend de MyClass pour accéder aux membres privés?

(en presque 20 ans de C++, je n'ai JAMAIS utilisé le mot-clé "Friend"...)

Merci de votre aide en tout cas !!
Eric

[hiura]

Mais est-ce que je pourrais créer une classe friend de MyClass pour accéder aux membres privés?

Oui. Une classe/fonction amie peut accéder à tout.

[méphistopheles]

de toute évidence, ça ressemble à une classe destinée à être mère: on voit d'ailleurs des fonctions virtuelles.Cela signifie donc que pour utiliser cette classe, tu doit le faire au travers d'une classe fille.

Cette classe me parait donc tout à fait correcte , elle nécessite juste un héritage.


Bonne chance

[Luc Hermitte]

Même pas, il faudrait au moins des constructeurs protégés -- à moins d'un "#define class struct" qui traine quelque part.

[JolyLoic]

Ta question est intéressante. Je ne vois de friend nul part (j'ai pourtant cherché...) Mais est-ce que je pourrais créer une classe friend de MyClass pour accéder aux membres privés?

Pas sans modifier la définition de MyClass. Plus le temps passe, plus je me demande, comme Luc, s'il n'y a pas un hack à l'aide de #define class struct qui traîne dans le .cpp qui visiblement ne t'est pas fourni.

[méphistopheles]

Même pas, il faudrait au moins des constructeurs protégés -- à moins d'un "#define class struct" qui traine quelque part.

Oups, oui, je suis complètement à coté... et je peux même pas mettre ça sur le compte de la fatigue (on est le 8 mai ). Bon, on va dire que j'étais trop détendu.

[eric_marciano]

En effet, l'héritage ne marche pas, j'ai déjà essayé.
Quant au "#define class struct", ça signifierait qu'on essaierait de me faire croire que c'est une classe, alors qu'en fait c'est une structure, c'est bien ça?
Mais dans ce cas, comment peut-on avoir des fonctions, et même des fonctions virtuelles?? On peut avoir ce genre de chose dans une struct?

Je crois que non, mais j'attends peut-être les limites de mes connaissances...

Eric

[10_GOTO_10]

Il n'y a pas un "friend" quelque part ?

S'il était quelque part, il serait dans la définition de MyClass. Une autre classe ne peut pas se déclarer arbitrairement amie de celle-ci (heureusement !).

[10_GOTO_10]

J'ai l'impression qu'il s'agit d'une classe prévue pour se dupliquer, se détruire elle-même, mais à condition d'en avoir déjà une instance (une "graine"), qui viendrait d'ailleurs (un void * renvoyé d'une fonction, par exemple). Protection anti-copie ?

[hiura]

Mais dans ce cas, comment peut-on avoir des fonctions, et même des fonctions virtuelles?? On peut avoir ce genre de chose dans une struct?

la seule différence entre les struct et les class en C++ est :
Par défaut le contenu et l'héritage dans une class est private.
Par défaut le contenu et l'héritage dans une struct est public.
(plus un troisième détail)

Voir http://cpp.developpez.com/faq/cpp/in...e_class_struct

[méphistopheles]

En effet, l'héritage ne marche pas, j'ai déjà essayé.
[...]
Mais dans ce cas, comment peut-on avoir des fonctions, et même des fonctions virtuelles?? On peut avoir ce genre de chose dans une struct

On peut, mais par contre, je me demande bien à quoi peuvent servir des fonctions virtuelles privées...

[eric_marciano]

J'ai l'impression qu'il s'agit d'une classe prévue pour se dupliquer, se détruire elle-même, mais à condition d'en avoir déjà une instance (une "graine"), qui viendrait d'ailleurs (un void * renvoyé d'une fonction, par exemple). Protection anti-copie ?

Oui, j'ai ça sur une autre classe du même type, une fonction globale du type:
void* get_MyClassInstance();

Le nom de la fonction en question est suffisamment explicite pour voir qu'elle est bien censée retourner une instance de ma classe mystérieuse.
Je ne sais pas comment l'utiliser, mais je sens qu'on est près du but.

Merci encore à tous pour votre aide !!

Eric

[EDIT] J'ai fait un nouvel essai: j'appelle la fonction, je récupère un void*, je le cast en "MyClass*" et j'essaie d'appeler une méthode de la classe. Bien sûr, ça ne marche pas car toutes les méthodes sont private...

[méphistopheles]

J'ai l'impression qu'il s'agit d'une classe prévue pour se dupliquer, se détruire elle-même, mais à condition d'en avoir déjà une instance (une "graine"), qui viendrait d'ailleurs (un void * renvoyé d'une fonction, par exemple). Protection anti-copie ?

Oui, mais il y a un problème: même en supposant que la classe soit instanciée dans un autre process, elle ne pourrais pas être recopiée sans au minimum un constructeur de copie public....

Sinon, une possibilité serait qu'elle soit juste déclarée pour être pointée par le process qui ne l'instancierais pas et qui se contenterais de manipuler son pointeur. le problème est que dans ce cas, un simple class MyClass; suffirait...

[Florian Goo]

Il est rigolo ce topic

Ça va paraitre idiot, mais peut-être que cette classe ne sert à rien ?
Peut-être que le type qui a développé ça était novice en C++ (en tout cas j'espère pour lui), d'où le symptomatique MyClass.
Il aurait écrit cette classe pour se faire la main sur la surcharge de new. Une fois qu'il aurait saisi le fonctionnement, il aurait alors eu l'idée de rendre tous les membres de cette classe privés histoire de tester que le reste du code n'en faisait plus usage.
Enfin, il aurait oublié de virer le reste de la classe.

Nous n'avons pas d'autre explication.

Plus sérieusement, tu devrais trouver un morceau du code dans le projet qui fait usage de cette classe. Fais une recherche globale.

[JolyLoic]

Quant au "#define class struct", ça signifierait qu'on essaierait de me faire croire que c'est une classe, alors qu'en fait c'est une structure, c'est bien ça?

Non, ça signifierait qu'on ferait croire à la moitié du programme que les fonctions sont publiques (car struct <=> class + public), alors qu'on dirait à une autre moitié du programme qu'elles sont privées. Bien entendu, tenir ce genre de discours schizophrénique à un compilateur n'est pas autorisé par le norme. Mais il y a des chances que ça passe avec pas mal de compilateurs. A noter que j'ai plutôt rencontré ce truc dans l'autre sens : L'utilisateur d'une classe qui voulait contourner les droits mis en place par l'auteur de la classe, plutôt que par l'auteur de la classe pour restreindre les accès, comme ça a peut-être l'air d'être le cas ici.

On peut, mais par contre, je me demande bien à quoi peuvent servir des fonctions virtuelles privées...

Elles sont très utiles. La quasi totalité de mes fonctions virtuelles sont privées. Les autres sont protected, ou public quand je fais du code à la va vite que je regrette systématiquement par la suite.
Une fonction virtuelle privée ne peut pas être appelée par les classes dérivées, mais rien n'empêche les classes dérivées de la redéfinir. Pour plus de détails, recherche programmation par contrat sur ce forum, ou bien lis http://www.gotw.ca/publications/mill18.htm qui résume bien la situation.

[Mat007]

Salut,

Soit il y a du code qui utilise cette classe (production ou test) et dans ce cas il suffit de regarder comment ça la manipule, soit il n'y en a pas et cette classe ne sert à rien dans le système existant.
Code mort => poubelle sans l'ombre d'une hésitation !

MAT.

[eric_marciano]

Merci à tous.
Avc toutes les informations que j'ai là, je devrais bien réussir à faire quelque chose, ou à conclure que je ne peux rien faire.
Pour clarifier la situation, je précise que c'est moi qui ai renommé la class "MyClass" et que l'auteur n'est surement pas un débutant. En revanche, le logiciel qu'il a écrit est suffisamment ancien pour mélanger du C et du C++, c'est peut-être là l'explication. Je suis désolé de ne pas pouvoir en dire plus pour des raisons de confidentialité.

Eric