Commentaires

1. Le vendredi 8 juin 2012 à 13:54, par Pascal MARTIN

Hello,

Plus ou moins de manière générale, et forcément avec des exceptions :

* J'ai tendance à mettre ce qui est public "en haut" du fichier ; et ce qui est protected/private, et donc moins susceptible d'être utile aux lecteurs de monde code, "en bas".
* Le constructeur étant quasi-toujours public, il va "en haut" ; et vu que c'est la première méthode appelée, la mettre en tout premier dans la déclaration de la classe est assez logique.
* L'ajout de nouvelles méthodes : en bas de fichier si la méthode que j'ajoute n'est pas fonctionnellement liée à une méthode existante ; sinon, sous la méthode à laquelle elle est fonctionnellement liée (pour que les méthodes "qui vont ensemble" fonctionnellement parlant soient "ensemble" en lisant le code)
* Setters / Getters, en général, vont par couple : setA, getA puis setB et getB, et ainsi de suite ; ce qui colle avec l'idée de mettre proches les unes des autres les méthodes qui sont fonctionnellement en rapport les unes avec les autres
* Pas de tri par ordre alphabétique : l'ordre alphabétique n'a pas de sens "logique" à la lecture du code (la méthode qui suit, dans l'ordre alphabétique, celle que je suis en train de lire n'a que peu de chance d'être liée à celle-ci). Si je veux consulter les méthodes par ordre alphabétique, la vue "structure / outline" de mon IDE fait ça très bien.

Après, "réorganiser" une classe en déplaçant des méthodes, je ne fais que rarement :

* De toute façon, quand je veux accéder à une méthode, j'utilise les fonctionnalités offertes par mon IDE (que ça soit "taper un bout du nom de la méthode pour rechercher", "ctrl+clic pour suivre", ou la vue "outline / structure"), donc l'ordre importe peut.
* Déplacer du code dans un fichier, c'est un coup à complexifier le suivi de modifications et l'analyse de diff, quand on compare les versions successives du-dit fichier (le gestionnaire de versions gère bien les insertions/suppressions ; mais pas forcément aussi bien les déplacements)

Cela dit, j'utilise toujours un IDE, et jamais un "simple éditeur de texte" -- ce qui peut certainement expliquer une partie de mes choix.

2. Le vendredi 8 juin 2012 à 14:23, par Moosh

Déjà merci pour cette réponse

En fait c'est justement parce que dans 2 branches différentes, les méthodes n'ont pas la même place que je voulais de quoi réorganiser pour que mes suivis de modification redeviennent possible.

Mais par la même occasion c'est pour corriger les non respects de règles de dev (comme évoquées ci-dessus)

Ajouter un commentaire

Nom ou pseudo :

Email (facultatif) :

Site Web (facultatif) :

Commentaire :

Le code HTML sera affiché comme du texte.

Se souvenir de mes informations