Trop de variables globales
Ça serait plus propre de faire un point une structure/objet et le définir en paramètre des fonctions qui en ont besoin

Le 05 décembre 2022 à 13:31:14 :
Trop de variables globales
Ça serait plus propre de faire un point une structure/objet et le définir en paramètre des fonctions qui en ont besoin

Et trop de commentaire évident. Un commentaire est là pour fournir de la documentation a l'extetieur d'un module. Ou pour expliquer pourquoi une ligne fonctionne, pourquoi elle est nécessaire parceque ce n'est pas naturellement évident.

Mais un commentaire:

Int x; // declare une variable entière appelle x

n'est pas un commentaire utile.

Trop de commentaires.
Utiliser les var dans JavaScript c'est super hein ?
Les fonctions sont trop fragmentées, vaut mieux les organisers selon les fonctionnalités pour avoir quelques dizaines de lignes dans une fonction.
Mélanger les déclarations et les instructions c'est stylé ? Pourquoi ne mettez pas toutes les déclarations avant la première instruction ?
Mauvaise indentation.
Faut savoir coder avec fonctions d'ordre supérieur pour éliminer les redondantes. Ou bien tu veux coder comme tu es à l'aise du Java ?

C'est du code généré par OpenAI ?

Pour moi ce code a été généré par une IA vu la verbosité des commentaires.

Un petit exemple pour illustrer ce qui va pas :

// Affichage du message d'erreur
console.error(message);
}

Le code est un langage (parlé par le programmeur). Dès lors, pas besoin de sous-titrer en français ce que fait le code. C'est redondant. C'est comme dire 2 fois la même chose.

Généralement, un code bien écrit n'a même pas besoin de commentaires. Si tu as besoin de commenter du code, c'est généralement un red flag qui t'indique qu'il faut le simplifier. (Je parle ici d'un commentaire qui décrit une implémentation bas-niveau, pas d'un commentaire qui décrit ce que fait une fonction et qui servira à générer de la documentation pour la librairie.)

trop de commentaires

  // Affichage du message d'erreur
  console.error(message);