Algèbre booléenne ? Qu’est ce donc que ceci ? Je pense que mon collègue n’a pas bien suivi ses cours en école d’ingé. Il n’a pas du assimiler le concept du « ET » et du « OU ». Il nous comble de joie en nous proposant de belles fonctions de plusieurs dizaines de lignes se résumant par une suite de if…
Il n’a pas non plus compris je pense le concept de l’imbrication des conditions.
Je vous laisse admirer :
-
protected Boolean isFoo_Excepte_06(final Child child) {
-
Boolean valide = Boolean.FALSE;
-
String code = child.getCode();
-
if (code.equals(MyConsts.CONST_1)) {
-
valide = Boolean.TRUE;
-
}
-
if (code.equals(MyConsts.CONST_2)) {
-
valide = Boolean.TRUE;
-
}
-
if (code.equals(MyConsts.CONST_3)) {
-
valide = Boolean.TRUE;
-
}
-
if (code.equals(MyConsts.CONST_4)) {
-
valide = Boolean.TRUE;
-
}
-
if (valide) {
-
if (child.getOtherCode().equals(MyConsts.CONST_5)) {
-
return Boolean.FALSE;
-
} else {
-
return Boolean.TRUE;
-
}
-
}
-
return Boolean.FALSE;
-
}
La première simplification serait bien évidement d’utiliser l’opérateur « OU » proposé nativement par java et d’imbriquer les conditions :
-
protected Boolean isFoo_Excepte_06(final Child child) {
-
String code = child.getCode();
-
if (code.equals(MyConsts.CONST_1)
-
|| code.equals(MyConsts.CONST_2)
-
|| code.equals(MyConsts.CONST_3)
-
|| code.equals(MyConsts.CONST_4)) {
-
-
if (child.getOtherCode().equals(MyConsts.CONST_5)) {
-
return Boolean.FALSE;
-
} else {
-
return Boolean.TRUE;
-
}
-
}
-
return Boolean.FALSE;
-
}
Une seconde simplification serait de ne pas faire « if(…) {return true;}else{return false;} » mais faire directement une instruction return et même soyons fous, d’utiliser l’opérateur « ET »…
-
protected Boolean isFoo_Excepte_06(final Child child) {
-
String code = child.getCode();
-
return ((code.equals(MyConsts.CONST_1)
-
|| code.equals(MyConsts.CONST_2)
-
|| code.equals(MyConsts.CONST_3)
-
|| code.equals(MyConsts.CONST_4))
-
&& (child.getOtherCode().equals(MyConsts.CONST_5)));
-
}
Elle est pas belle la vie ?