Pourquoi mon switch casse tout le code alors que mes fonctions marchent

Question

Je voulais comprendre pourquoi on est obligés de faire 4 fonctions (même si je comprends l'intérêt des fonctions) et non pas un switch comme ça par exemple (les variables sont déjà déclarées en let au-dessus) :
Le seul souci c'est que dès que je mets ce switch , plus rien ne marche, même pas les étapes précédentes qui marchent si j'enlève ce switch ?? Merci
switch(choix){
case 1:
result = nombreUn + nombreDeux;
return result;
break;
case 2:
result = nombreUn * nombreDeux;
return result;
break;
case 3:
result = nombreUn - nombreDeux;
return result;
break;
case 4:
result = nombreUn / nombreDeux;
return result;
break;
default:
throw new Error('une erreur est survenue');
}

Louis-Nicolas Leuillet · Accepted Answer

Ah, je vois ce qui ne va pas dans ton code ! En fait, le problème principal vient du fait que tu utilises return en dehors d’une fonction. En JavaScript, return ne peut être utilisé que dans une fonction , sinon ça plante direct. Donc si ton switch est dans le scope global ou dans un endroit où return n’est pas attendu, ça fait tout foirer.
Autre point : t’as mis des break après chaque return . En réalité, ça sert à rien parce que dès que return s’exécute, la fonction s’arrête immédiatement. Donc les break sont juste du code mort, ils ne sont jamais atteints.
Du coup, la solution, c’est de mettre ton switch dans une fonction . Un truc comme ça :
function calculer(choix, nombreUn, nombreDeux) {
switch (choix) {
case 1:
return nombreUn + nombreDeux;
case 2:
return nombreUn * nombreDeux;
case 3:
return nombreUn - nombreDeux;
case 4:
if (nombreDeux === 0) throw new Error('Division par zéro impossible');
return nombreUn / nombreDeux;
default:
throw new Error('Choix invalide');
}
}
Maintenant, ton code va fonctionner parce que tout est bien encapsulé dans une fonction, et return peut s’exécuter normalement.
Après, pourquoi faire 4 fonctions au lieu d’un switch ? Ben en général, quand on veut un code plus clair et maintenable, on préfère séparer les responsabilités. Ça permet de réutiliser les fonctions facilement et de tester chaque opération indépendamment. Mais si ton but est juste de faire un petit script rapide, ton switch dans une fonction, ça passe très bien.

Mireille Emonot · Answer

Je ne peux plus enlever les return dans le code exposé (c'est un reste d'essai avec les fonctions !) mais le switch ne marchait pas sans le return

Pourquoi mon switch casse tout le code alors que mes fonctions marchent

Formation JavaScript (legacy)