Discussion :

[17mounir]

Je commence à utiliser le plugins CheckStyle pour essayer d'homogéiniser un peu le code et vérifier qu'il respecte la norme de base...

Mais je trouve plusieurs Warning et je ne sais pas si ça doit être traîté ou pas :

- La méthode xxx n'est pas conçue pour être dérivée il faut la déclarer abstraite, final ou la laisser vide... (je fais comment ?)

- Les paremètre doivent être final...

- ...

Est ce qu'il faut normalement respecter toutes les consignes de la config de Sun, pour dire qu'on code proprement (niveau styling) ou pas ?

[gailuris]

Salut,

Les régles de sun sont (à mes yeux et ça peut se discuter longuement) une sorte d'idéal à atteindre en terme de conventions de codes. Après selon les contraintes du projets, les "goûts" de l'équipe, tout peut se discuter. Lorsque j'ai utilisé checkstyle, j'ai sacrément diminué les contraintes car à mon "goût" trop importantes.
Par contre pour un bon travil en équipe, il faut évidement se mettre d'accord sur ces régles.

[17mounir]

Merci pour ta réponse...

Oui c'est vrai que l'idéal est difficile à obtenir, ce n'est pas ce que je cherche non plus vu le coûp que ça engendre... mais j'avoue que j'apprends des choses sur Java en utilisant le plugin (exe : paramètres en final,...)

Je me pose par contre la signification du Warning (La méthode X n'est pas conçue pour être dériver, il faut la déclarere abstraite, final ou la laisser vide...

Est ce que toutes les methodes qui ne sont pas redéfinie devront être final; ça risque de devenir pénible... on gagne grand chose niveau perf (grace au final) ?

[gailuris]

J'ai trouvé ce lien intéressant (à rpiori j'ai pas tout lu) la dessus :

http://checkstyle.sourceforge.net/co...gnForExtension

En gros ils estiment que dans le cadre d'une api (ce qui est presque toujours le cas pour sun), une méthode ne peut être surchargée que si elle est soit abstraite soit vide, sinon elle ne peut pas et doit donc etre déclarée finale. Le désavantage de cette régle etant que tu ne peux prévoir un comportement par défaut pour la classe mère...
=> Encore une règle controversée.

Par contre je ne pense pas que le final te fasse gagner quoi que ce soit au niveau perf, c'est vraiment une question de design de ton appli.

[®om]

En gros ils estiment que dans le cadre d'une api (ce qui est presque toujours le cas pour sun), une méthode ne peut être surchargée que si elle est soit abstraite soit vide, sinon elle ne peut pas et doit donc etre déclarée finale. Le désavantage de cette régle etant que tu ne peux prévoir un comportement par défaut pour la classe mère...

Donc la méthode toString() de Object ne respecte pas cette règle...

[gailuris]

Donc la méthode toString() de Object ne respecte pas cette règle...

Oui Et il doit y en avoir des tas. Preuve que "c'est surtout une ligne de conduite" (© Pirate des Caraïbes, l'avatar de 17mounir va bien avec ).

[17mounir]

lol, merci les gas pour ces réponses...

Finallement je crois que je ne vais pas utiliser cette règle pour le moment, ça fera moins de jaune sur l'écran.