Discussion :

[scrouet]

Bonjour.

Je fais analyser mon code (PHP) par SonarQube depuis peu. Je me prends tout un paquet de "Code smells" "Define a constant instead of duplicating this literal "xxx" n times", où "xxx" est en général le nom d'une colonne de ma base de données.
Auriez-vous des bonnes pratiques pour éviter ce genre d'alertes ? J'hésite à définir une constante dans mon code pour les noms de toutes les tables et toutes les colonnes de la base.

[SQLpro]

le code d'une requête n'a rien à voir avec le code d'un langage de dev. L'optimiseur se chargera de factoriser... !

A +

[scrouet]

En fait ce n'est pas tant le code de la requête que SonarQube pointe, mais l'utilisation de chaînes de caractères pour récupérer les résultats.
Par exemple, si je récupère les nom et prénom (champs Per_Nom et Pre_Prenom) de personnes via une requête SQL (ici en PHP)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
$sRqtPer = "SELECT * FROM personnes WHERE Per_Id=:iIdListe";
$oPdsResPer = $oPdoCnxBD_L->prepare($sRqtPer, array(PDO::ATTR_CURSOR => PDO::CURSOR_FWDONLY));
$oPdsResPer->execute(array(':iIdListe' => $iIdListe)
$aUtilisateurs = $oPdsResPer->fetchAll();

SonarQube râle si j'utilise plus de 2 fois $aUtilisateurs['Per_Nom'] ou $aUtilisateurs['Per_Prenom'].
La règle en question : https://rules.sonarsource.com/php/ty...earch=constant

[Celira]

L'idée sous-jacente de la règle dans SonarCube est que si tu utilises une valeur constante, c'est qu'il y a une notion fonctionnelle (ou éventuellement technique) derrière. Utiliser des constantes te permet de différencier par exemple entre "20" le nombre maximum d'éléments à afficher dans un tableau et "20" la longueur maximale d'un pseudo. ça facilite aussi grandement l'existence si la valeur constante en question doit changer : tu changes à un endroit et pouf ! magie ! toute l'application est à jour. En reprenant l'exemple précédent, si tu dois modifier la taille max d'un tableau de 20 à 25 et que tu as écrit en dur "20" dans chaque script qui construit un tableau, ça va vite devenir casse-pied...

Dans ton cas, il s'agit d'un nom de colonne dans la base. Utiliser une constante te permet donc de gérer le cas où la colonne est renommée en base (et accessoirement d'éviter de te prendre le chou parce que tu as écrit $aUtilisateurs['Pre_Nom'] et que tu ne comprends pas pourquoi le nom reste vide )

Bon évidemment, si tu renommes les colonnes en base, tu vas de toute façon devoir te farcir les fichiers pour modifier le code des requêtes proprement dites (sauf si tu utilises un ORM type Doctrine qui écrit les requêtes pour toi, mais ceci est une autre histoire)

[scrouet]

Merci pour cette réponse. Je pense configurer toutes mes constantes liées à la base de données dans un seul fichier.