Partage et revue de code pour un calculateur d'IMC en JavaScript

Question

Bonjour,
Je vous partage mon code, il fonctionne. Nicolas, aurais-tu un commentaire à me faire dessus stp ?
let poids = Number(prompt("Quel est votre poids en kg ?"));
let taille = Number(prompt("Quel est votre taille en cm ? "));
function calculateurImc(poids, taille) {
let tailleEnMetre = taille / 100;
let resultat = poids / (Math.pow(tailleEnMetre, 2));
return resultat;
}
alert("Votre IMc est de " + calculateurImc(poids, taille));

Louis-Nicolas Leuillet · Accepted Answer

Bonjour Roxanne,
Merci d'avoir partagé ton code ici haha.
Bon, ton code fonctionne et la logique est bonne, donc bravo pour ça ! J'ai quand même quelques petites remarques pour t'aider à progresser.
Première chose : c'est une bonne pratique de récupérer les entrées utilisateur en dehors de la fonction, comme tu l'as fait. Par contre, tu pourrais arrondir le résultat pour éviter d'afficher un nombre avec 15 décimales. Un petit toFixed(2) sur le résultat ferait l'affaire. Aussi, pense à gérer le cas où l'utilisateur entre une valeur invalide (texte, valeur négative, etc.), sinon tu risques d'obtenir NaN ou Infinity dans l'alerte.
Deuxième point, plus cosmétique : Math.pow(tailleEnMetre, 2) fonctionne très bien, mais en JavaScript moderne, tu peux aussi écrire tailleEnMetre ** 2 avec l'opérateur d'exponentiation, c'est un poil plus lisible. Voici à quoi ça pourrait ressembler avec ces ajustements :
let poids = Number(prompt("Quel est votre poids en kg ?"));
let taille = Number(prompt("Quelle est votre taille en cm ?"));
function calculateurImc(poids, taille) {
let tailleEnMetre = taille / 100;
let resultat = poids / (tailleEnMetre ** 2);
return resultat.toFixed(2);
}
alert("Votre IMC est de " + calculateurImc(poids, taille));
Bravo c'est du bon travail, continue comme ça !

Roxanne Coquet · Answer

Ok super je corrige et je note ! Merci pour ton retour !

Partage et revue de code pour un calculateur d'IMC en JavaScript

Formation JavaScript (legacy)