10
août
2012
Code review: comment ne pas écrire une méthode simple
août
2012
Un article de Philippe Vialatte
Pour une fois, ce n’est pas du « lu dans mon dépôt svn », mais du « lu sur le web ».
Dans un post sur les méthodes d’extension, l’auteur donne en exemple la méthode suivante:
public static bool IsEven(this int num){
if (num == 0){
return false;
}else if (num % 2 == 0){
return true;
}
return false;
}
Il y’a deux problèmes avec cette méthode, un bug fonctionnel, et un souci plus général d’écriture.
Allez, qui s’y colle ?
2 Commentaires + Ajouter un commentaire
Articles récents
Archives
- janvier 2014
- septembre 2013
- août 2013
- mai 2013
- avril 2013
- janvier 2013
- août 2012
- juin 2012
- mai 2012
- avril 2012
- mars 2012
- novembre 2011
- septembre 2011
- août 2011
- juillet 2011
- juin 2011
- mai 2011
- avril 2011
- février 2011
- janvier 2011
- novembre 2010
- octobre 2010
- septembre 2010
- août 2010
- juillet 2010
- juin 2010
- mai 2010
- avril 2010
- mars 2010
- février 2010
- janvier 2010
- décembre 2009
- novembre 2009
- octobre 2009
- septembre 2009
- août 2009
- juillet 2009
- juin 2009
- mai 2009
- avril 2009
- mars 2009
- février 2009
- janvier 2009
tout pile
C’est marrant, j’ai remarqué le même code il y a deux jours
J’aurais écris tout simplement ceci :
public static bool IsEven(this int num)
{
return num % 2 == 0;
}
Du point de vue fonctionnel, le développeur a considéré que 0 n’était pas pair alors q’il l’est (si ma mémoire est bonne).