Écouter ESLint — exercice corrigé de JavaScript : rendre le Web interactif (école d'ingénieur) (chapitre 11)

Écouter ESLint

Erreurs, stockage et qualité du code · niveau 1 (application)

Énoncé

Pour chacun des cinq messages qu'ESLint donne sur le brouillon du cours, expliquer le risque réel et corriger. Lequel des cinq aurait fait planter le programme, et quand ?

Corrigé

no-var (ligne 4) : var a une portée de fonction et remonte (chapitre 4) ; remplacer par let ou const.
no-unused-vars (total) : une variable morte, souvent le signe d'un code inachevé ; la supprimer.
eqeqeq (ligne 10) : == convertit ("9782956300014" == 9782956300014 est vrai) ; écrire ===.
no-unused-vars (ancien) : la fonction voulait sans doute renvoyer l'ancien panier ; c'est return ancien; qui manquait.
no-undef (pannier) : faute de frappe. C'est elle qui aurait fait planter le programme, par une ReferenceError, et seulement au premier appel de vider(), peut-être des semaines après la mise en ligne.

Les deux avertissements sur des variables inutilisées montrent l'intérêt de les lire : l'un d'eux cachait le vrai bogue.