Déclaration des variables dans une fonction : bonnes pratiques

Question

Ok, mais pourquoi déclarer les variables hors de la fonction ?
La fonction est une brique élémentaire qui doit se suffire à elle-même, du coup, elle doit contenir toutes les informations nécessaires à sa vie.
Ce qui permet une plus grande lisibilité aussi, du moins je crois, je peux me tromper. Mon code :
function imc(poid, taille){
poid = Number(prompt('Entrez votre poids'));
taille = Number(prompt('Entrez votre taille en cm'));
let tailleMetre = taille / 100;
let imcResult = poid / (Math.pow(tailleMetre, 2));
return imcResult.toFixed(2);
};
alert(imc());
Idem, peut-être une erreur de ma part, mais je ne crée pas de variables let poid ni let taille , car ce sont les deux arguments que je reçois dans ma fonction, ils se suffisent à eux-mêmes (ça fonctionne) mais c'est peut-être pas juste non plus.

Louis-Nicolas Leuillet · Accepted Answer

Hello François,
Je comprends ta logique mais elle n'est pas tout à fait correcte : une fonction est faite pour ne pas se répêter .
Autrement dit, elle n'est pas forcément obligée de se suffire à elle-même. D'ailleurs, tu remarqueras vite que c'est impossible.
La raison ? Parce qu'on utilise souvent la même variable dans plusieurs fonctions.
Autrement dit avec ton raisonnement actuel c'est impossible par la suite de réutiliser poid ou taille . On est obligé de redemander une nouvelle fois à l'utilisateur ces informations : bah oui les données n'existent que pour la fonction imc ! 👀
Alors qu'en passant les paramètres (donc les variables), on peut ensuite décider n'importe quand de réutiliser le poids et la taille (peut-être pour afficher un graphique ou que sais-je).
Et pour ta seconde interrogation, ton code est faux : tu ne reçois jamais poid ou taille dans ta fonction. La preuve du appelle imc() directement sans faire ceci :
alert(imc(poid, taille));
Pour cause si tu fais ça tu auras un problème plus tard car tes variables poid et taille n'existent pas à l'extérieur de ta fonction donc tu ne peux pas les utiliser.
Ton code avec ton envie d'intégrer les variables ressemblerait normalement à ça :
function imc(){
const poid = Number(prompt('Entré votre poid'));
const taille = Number(prompt('entrée votre taille en cm'));
let tailleMetre = taille / 100;
let imcResult = poid / (Math.pow(tailleMetre, 2));
return imcResult.toFixed(2);
};
alert(imc());

Francois P · Answer

PS: Grrrr j'ai un IMC de 32.45 exercice traumatisant 😱😁

Francois P · Answer

Merci, ça fait tellement longtemps que je ne code plus que je n’avais même pas vu mon erreur (ma bêtise 😮).

Déclaration des variables dans une fonction : bonnes pratiques

Formation JavaScript (legacy)