Odeur de code - affectation locale inutilisée-js tutoriel-php.cn

Odeur de code - affectation locale inutilisée

Mary-Kate Olsen

Libérer： 2025-01-30 02:31:08

original

723 Les gens l'ont consulté

utilisez-vous la valeur renvoyée?

TL; DR: Évitez d'attribuer des valeurs que vous n'utilisez jamais.

Problèmes

Code mort
Intention claire
Offre de maintenance
complexité cachée
Débogage des restes
Mauvaise portée
Code commenté
mutabilité

Solutions

supprimer les affectations inutilisées.
Essayez d'éviter les effets secondaires.

Contexte

Lorsque vous attribuez une valeur à une variable locale mais que vous ne l'utilisez jamais, vous créez un encombrement inutile dans votre code.

Cela peut confondre les autres et rendre le code plus difficile à maintenir.

Parfois, lors du débogage, vous pouvez attribuer des variables temporelles pour une meilleure inspection.

Cela se produit également lorsque vous attribuez l'exécution à une propriété d'objet mais il est plus difficile à suivre.

C'est aussi un signe d'un objet mutant car si vous supprimez l'affectation, il ne reste que les effets secondaires.

Les objets mutés peuvent provoquer des effets secondaires inattendus, ce qui rend plus difficile le suivi des changements.

Exemple de code

Faux

function updateUserName(user, newname) {
  user.name = newname;
  return user;
}

function performMaintenance(existingUser) {
  let updatedUser = updateUserName(existingUser, "Bobby Peru");
  // Other tasks
}
// The variable updatedUser is never used

Copier après la connexion

Droite

function updateUserName(user, newname) {
  user.name = newname;
  // Just side effects without explicit return
}

function performMaintenance(existingUser) {
  updateUserName(existingUser, "Bobby Peru");
  // Other tasks
}

Copier après la connexion

Détection

[x] automatique

Vous pouvez détecter cette odeur à l'aide d'outils d'analyse statique ou de revues de code qui vérifient les variables inutilisées après l'attribution.

La plupart des liners signalent cela comme un problème.

Balises

Bloatiers

Niveau

[x] débutant

Pourquoi la bijection est importante

Il reste propre et efficace lorsque votre code reflète avec précision la logique du monde réel.

Les affectations inutilisées rompent cette connexion, ce qui rend plus difficile de comprendre l'intention et de maintenir le code.

Génération d'IA

Les outils AI génèrent rarement des affectations de variables inutilisées lorsqu'ils comprennent l'intention.

Détection d'IA

La refactorisation assistée par AI avec des instructions claires peut signaler les variables inutilisées et suggérer des déménagements, mais il ne comprend pas toujours si la valeur de retour aurait dû être utilisée.