Boucle, condition et modulo

**Theoask** · 03/08/2016, 17h04

Bonjour,

Quelqu'un peut m'aider pour cet autre problème Javascript ? J'ai la chance d'avoir un collgègue programmeur expérimenté, mais même lui ne trouve pas...

Le code est bon, mais je crois que c'est l'interface de debugging qui bugge...

INSTRUCTIONS:

Nous allons maintenant jouer à FizzBuzz. Les règles sont simples. Nous voulons compter de 1 à 20. Mais si le nombre est divisible par 3, nous allons afficher "Fizz", et si le nombre est divisible par 5, nous allons afficher "Buzz".

Et si le nombre est divisible par 3 ET par 5 ? Exact ! Nous allons afficher "FizzBuzz" !

Il y a plusieurs moyens de le faire, mais nous voudrions une condition imbriquée dans cet exercice.
Instructions

Affichez les nombres de 1 à 20.
Les règles :
Pour les nombres divisibles par 3, affichez "Fizz".
Pour les nombres divisibles par 5, affichez "Buzz".
Pour les nombres divisibles par 3 et 5, affichez "FizzBuzz".
Sinon, affichez simplement le nombre.

MESSAGE D'ERREUR APRES EXECUTION DE MON CODE:

Oups, merci de réessayer. Vous avez affiché 3 Fizz au lieu de Fizz"

MON CODE:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
for(var i = 1; i < 21; i++ ) {
if (i % 15 == 0) {
console.log(i + " " + "Fizzbuzz");
}
else if (i % 5 === 0 ){
console.log(i + " " + "Buzz");
}
else if (i % 3 === 0 ) {
console.log(i + " " + "Fizz");
}
else {
console.log(i)
}
 
}

**NoSmoking** · 05/08/2016, 10h48

Bonjour,
si on s'en réfère à l'énoncée

Pour les nombres divisibles par 3, affichez "Fizz".
...etc...

tu ne dois afficher que "Fizz" et non "3 Fizz", "6 Fizz" ...etc...

Une remarque sur ton code il eut mieux valu écrire if (i % 3 === 0 && i % 5 === 0) {} plutôt que if (i % 15 == 0) {}trop réducteur.

**Theoask** · 05/08/2016, 10h59

C'est un collègue de travail Programmeur avec des années d'expérience qui m'a recommandé de code comme ça plutôt, car un nombre divisible par 15 est aussi divisible par 3 et par 5, donc optimisation du code et simplification du code, ce qui est le but de la bonne programmation,non?*

Et si je lis bien l'instruction: "Pour les nombres divisibles par 3, affichez "Fizz"."

Cela signifie qu'il y peut-être plus d'un nombre divisible par 3, donc plusieurs "Fizz" logiquement..Sinon je ne comprends pas bien l'intérêt de mettre une boucle "for" et de l'exercice.

**NoSmoking** · 05/08/2016, 11h19

donc optimisation du code et simplification du code, ce qui est le but de la bonne programmation,non?

qu'en est-il si au lieu de chercher jusqu'à 20 on te demande de faire la même chose jusqu'à 100 !?!

Dans la même logique pourquoi ne pas faire si i égal 3 ou 6 ou 9 ...etc.. !?!

Et si je lis bien l'instruction: "Pour les nombres divisibles par 3, affichez "Fizz"."

Voilà ce que l'on te demande d'afficher

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
1
2
Fizz
4
Buzz
Fizz
7
8
Fizz
Buzz
11
Fizz
13
14
FizzBuzz
16
17
Fizz
19
Buzz

et rien d'autre.

**Theoask** · 05/08/2016, 12h01

No Smoking

Et moi je réponds à l'exercice et à rien d'autre.."Et si" , "et si"...Ton raisonnement se vaut, mais le mien aussi je pense.

Le but est de répondre le plus efficacement possible à l'exercice donné, et uniquement à cet exercice,sinon on s'en sort pas...

Mais ce n'est que mon avis...Merci quand même...