#good pratice

[spicynoodlesdoodles]

To those of whom are interested in asking fir requests please direct them in a simple manner over to my ask box please!

[aritylabs]

Good practice: bannir `send` pour les messages dynamiques

Quand je passe en revue du code Ruby, je tombe souvent sur des utilisations de send à éviter. C'est une méthode présente sur tous les objets, permettant d'envoyer un message (aka. appeler une méthode) en passant en paramètre le nom de la méthode voulue.

Cet usage est fautif et à bannir si on y a recourt pour effectuer des appels dynamiques : envoyer un message sans savoir jusqu'au dernier moment lequel (nom reçu en paramètre, itération sur plusieurs noms de méthodes, etc...). 

Pour cause, l'autre usage notoire de send n'est autre que de violer à dessein l'interface publique des objets, pour appeler des méthodes privées ou protégées.

Exemple pour inclure un module dans un autre sans réouvrir celui-là (ìnclude` est une méthode de classe privée) :

ActiveRecord::Base.send :include, MyFancyModule

L'alternative est d'utiliser public_send, qui n'invoke, comme son nom l'indique, que les méthodes publiques de vos objects.

Vous serez ainsi certain de ne pas appeler par mégarde une méthode privée qui peut disparaitre ou changer d'interface à tout moment. Rappelons qu'en POO, seule l'interface publique d'une classe est censée être conservée.

Cheers