RSS Le Daily WTF
Suivre
CodeSOD : une question de compréhension
Un développeur a écrit un morceau de code inefficace et mal rédigé qui a été appelé « Googlesauce » parce qu'il semble avoir été copié-collé à partir d'une recherche Google. Le code utilise un StringBuffer pour construire une requête SQL simple, ce qui n'est pas nécessaire, et inclut une clause WITH au mauvais endroit. La requête est ensuite préparée en tant que déclaration, mais elle ne contient aucun paramètre, ce qui rend cette étape inutile. Le développeur tente ensuite de définir un paramètre inexistant, ce qui lancerait une exception. Au lieu d'utiliser une déclaration SELECT COUNT, le code effectue une boucle sur les résultats pour compter les lignes. Le code est incorrect sur le plan syntaxique et ne fonctionnerait pas comme prévu. Le plus surprenant est que ce code ait été intégré à la base de code sans avoir été détecté ni corrigé. Le développeur qui a écrit le code semblait ne pas comprendre ce qu'il faisait et a quand même enregistré le code non fonctionnel. Cela met en évidence un manque de protection en place pour empêcher la validation d'un code mal écrit. La tâche de nettoyer ce code incombe désormais à des développeurs plus compétents. La présence d'un tel code dans la base de code soulève des inquiétudes quant à la qualité globale et à la maintenabilité du logiciel.