Cette méthode fais une centaine de ligne et mélange toutes les stratégies de sérialisation. AMHA elle a besoin de refactoring pour séparer chacune de ses stratégies dans des fonctions séparée et cette méthode-ci ne garder que la logique de « comment on sélectionne la bonne stratégie ? ». Cela réduira considérablement sa complexité cyclomatique et permettra de tester chaque stratégie de manière séparée. Mais on va fatalement écrire plus de chose en faisant ça.
Tous les contenus que j'écris ici sont sous licence CC0 (j'abandonne autant que possible mes droits d'auteur sur mes écrits)
[^] # Re: Moi aussi !
Posté par barmic . En réponse au journal Lutter contre l'overengineering. Évalué à 6.
Je ne suis pas d'accord et j'ai un exemple : https://github.com/wordpress-mobile/WordPress-Android/blob/develop/WordPress/src/main/java/org/xmlrpc/android/XMLRPCSerializer.java#L69
Cette méthode fais une centaine de ligne et mélange toutes les stratégies de sérialisation. AMHA elle a besoin de refactoring pour séparer chacune de ses stratégies dans des fonctions séparée et cette méthode-ci ne garder que la logique de « comment on sélectionne la bonne stratégie ? ». Cela réduira considérablement sa complexité cyclomatique et permettra de tester chaque stratégie de manière séparée. Mais on va fatalement écrire plus de chose en faisant ça.
Tous les contenus que j'écris ici sont sous licence CC0 (j'abandonne autant que possible mes droits d'auteur sur mes écrits)