Discussion :

[FunkyTech]

Bonjour à tous
Voici un exemple de code pour illustrer mon souci

Code :

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

1
2
3
 
//! Returns a reference to the InputManager.
InputManager&	getInputManager()	{ return *m_pInputManager; }

Je ne vois pas trop bien ce que fait ce type de retour, à quoi ça sert et pourquoi ne pas simplement retourner un pointeur ?

Merci d'avance

[loufoque]

Parce que les références ont plus de restrictions d'utilisations, les rendant plus sûres.

[FunkyTech]

Ok merci beaucoup c'est beaucoup plus clair.

[Médinoc]

Mais si le pointeur est nul, on retourne une référence invalide, ce qui n'est pas censé arriver.

[Klaim]

En fait, arriver a ce genre d'accesseur est mauvais signe.

Dans le doute j'aurais quand même mis un pointeur ne serait-ce que pour que l'utilisateur du code soit prévenu qu'il soit potentiellement null.

Si ça ne dois de toutes façons pas arriver, alors il y a un problème autre part....ou alors c'est un objet qui construit une donnée via new ou malloc et qui fournis une référence a l'exterieur. Si c'est le cas, une assertion serait bienvenue je pense.

[loufoque]

Je vois pas où est le problème dans cette interface...
L'InputManager peut utiliser l'héritage et être appartenu par l'objet. Ou alors il peut changer. Ce qui justifie parfaitement d'utiliser un pointeur plutôt qu'une référence en interne.
Il n'est néanmoins pas nécessaire de fournir à l'utilisateur un pointeur, une référence suffit.

Dans le doute j'aurais quand même mis un pointeur ne serait-ce que pour que l'utilisateur du code soit prévenu qu'il soit potentiellement null.

Qu'est-ce qui te permet de dire que le pointeur ne pointera pas toujours vers un objet valide ? C'est un invariant plutôt facile à garantir...