Le code définit des fonctions pour convertir entre des booléens et un type "YesNoType" personnalisé, qui représente essentiellement une valeur booléenne stockée en tant que chaîne de caractères.
La fonction "toYesNo(String)" mappe "Y" à "Yes" et d'autres valeurs à "No", introduisant une inconsistance potentielle puisqu'elle renvoie toujours "Yes" indépendamment de la chaîne d'entrée.
La fonction "toYesNo(boolean)" ne peut pas directement construire un "YesNoType" à partir d'un booléen, ce qui est considéré comme un défaut.
La fonction "fromYesNo(YesNoType)" vérifie si le "YesNoType" correspond à "Yes" (insensible à la casse) et renvoie "Yes" en conséquence.
La fonction "isYesNo(YesNoType)" convertit le "YesNoType" en booléen, mais déclare inutilement une variable locale et pourrait être simplifiée.
Le code qui repose sur un "YesNoType" personnalisé au lieu d'utiliser des booléens est considéré comme un choix de conception douteux.
La fonction "toYesNo(String)" permet de définir un "YesNoType" avec "Y" mais renvoie toujours "Yes", créant des valeurs incohérentes.
La fonction "toYesNo(boolean)" ne peut pas créer directement un "YesNoType" à partir d'un booléen, ce qui est un inconvénient significatif.
La fonction "fromYesNo(YesNoType)" semble inutile avec une conception appropriée.
Dans l'ensemble, l'utilisation de "YesNoType" comme représentation en chaîne de caractères des booléens dans le code est considérée comme une mauvaise décision de conception.
thedailywtf.com
CodeSOD: Yes, No, NO NO NO NO
Create attached notes ...
