Le bout de code du jour nous est envoyé par un contributeur qui souhaite rester anonyme. C'est délicieux de voir qu'à partir d'un bon sentiment, un développeur peut créer un horrible monstre qui aura des répercussions pendant plusieurs années. Le WTF serait-il lui aussi pavé de bonnes intentions?

Le code suivant provient d'un "expert" .net employé dans une SSII. Il est exécuté lors de l’import d’un fichier plat dans une base de données. Le développeur prévoyant pense au cas où le fichier CSV ne comporterait pas assez de colonnes par rapport à la table cible. Il l'explique d’ailleurs dans son commentaire: il remplacera les colonnes manquantes par la valeur "null". Ou presque ...

//    Si il n'y a pas assez de colonne alors on ajoute les colonnes avec une valeur null
for (; j < champs.Length - 5; j++)
    this._commande.Parameters.Add(champs[j].nom, champs[j].type).Value = "rien";

 


Sauve un projet, mange un prestataire !