Je viens de rentrer d'un entretien d'embauche et l'intervieweur m'a demandé d'écrire un programme:
Il devrait, compter de 1 à 100, et imprimer ...
S'il était multiple de 3, "ping"
S'il était multiple de 5, "pong"
Sinon, imprimez le numéro.
S'il était multiple de 3 AND 5 (comme 15), il devrait afficher "ping" et "pong".
J'ai choisi le langage Javascript et je suis arrivé avec ceci:
for (x=1; x <= 100; x++){
if( x % 3 == 0 ){
write("ping")
}
if( x % 5 == 0 ){
write("pong")
}
if( ( x % 3 != 0 ) && ( x % 5 != 0 ) ){
write(x)
}
}
Actuellement, je suis très mécontent de ma solution, mais je ne peux pas en trouver une meilleure.
Est-ce que quelqu'un connaît une meilleure façon de faire cela? .
Votre solution est tout à fait satisfaisante à mon humble avis. Dur, comme la moitié des nombres ne sont pas multiples de 3 ni 5, je commencerais par l’inverse:
for (var x=1; x <= 100; x++){
if( x % 3 && x % 5 ) {
document.write(x);
} else {
if( x % 3 == 0 ) {
document.write("ping");
}
if( x % 5 == 0 ) {
document.write("pong");
}
}
document.write('<br>'); //line breaks to enhance output readability
}
Notez également que tout nombre autre que 0
et NaN
sont des valeurs de vérité. J'ai donc supprimé le != 0
inutile et certaines paires de parenthèses.
Voici une autre version, elle ne fait pas deux fois la même opération de module mais nécessite de stocker une variable:
for (var x=1; x <= 100; x++) {
var skip = 0;
if (x % 3 == 0) {
document.write('ping');
skip = 1;
}
if (x % 5 == 0) {
document.write('pong');
skip = 1;
}
if (!skip) {
document.write(x);
}
document.write('<br>'); //line breaks to enhance output readability
}
Voici mon one-liner:
for(var x=1;x<101;x++)document.write((((x%3?'':'ping')+(x%5?'':'pong'))||x)+'<br>');
J'utilise des opérateurs ternaires pour renvoyer une chaîne vide ou 'ping'/'pong'
, en concaténant le résultat de ces opérateurs, puis en effectuant un OR (si le nombre n'est pas divisible par 3 ou 5, le résultat du la concaténation est ''
qui est FALSEY en javascript). Lorsque les deux cas sont vrais, le résultat de la concaténation est 'pingpong'
.
Donc, fondamentalement, cela revient à
'' || x // returns x
'ping' || x // returns 'ping'
'pong' || x // returns 'pong'
'pingpong' || x // returns 'pingpong'
Voici une solution qui permet une liste dynamique de multiples sans ajouter plus de conditions.
// List of outputs
var outputs = [
{mult: 3, str: 'ping'},
{mult: 5, str: 'pong'}
// {mult: 10, str: 'play'} ex: [30] => 'pingpongplay'
];
// Loop 100 times
for (var i = 1, j = 100; i <= j; i += 1) {
// Set empty vars
var result, string = '';
// Loop through the listed output objects
outputs.forEach(function (output) {
// If listed multiple, concat string
if (i % output.mult === 0) {
string += output.str;
}
});
// Set result
if (string.length) {
result = string;
} else {
result = i;
}
// print result
document.body.innerHTML += result + '<br>';
}
Et comme une fonction qui passe jslint:
/*jslint browser: true */
var printOutputs = function (array, iterations) {
'use strict';
var i = 1;
var outputResults = function (arr, idx) {
var result;
var str = '';
arr.forEach(function (item) {
if (idx % item.mult === 0) {
str += item.str;
}
});
if (str.length) {
result = str;
} else {
result = idx;
}
return result;
};
while (i < iterations) {
document.body.innerHTML += outputResults(array, i) + '<br>';
i += 1;
}
};
var outputs = [
{mult: 3, str: 'ping'},
{mult: 5, str: 'pong'}
];
printOutputs(outputs, 100);
Et pour le plaisir, une version ES6 simplifiée:
const pO=(arr,itr)=>{let i=1,o=(a,idx)=>{let r,s='';a.map(v=>{if(idx%v.mult===0)s+=v.str});s.length?r=s:r=idx;return r};while(i<itr){document.body.innerHTML+=`${o(arr,i)}<br>`;i++}};
pO([{mult:3,str:'ping'},{mult:5,str:'pong'}], 100);
La meilleure solution que j'ai proposée est la suivante:
for (var i = 1; i <= 100; i++) {
var message = '';
if (i%3 === 0) message += 'ping';
if (i%5 === 0) message += 'pong';
console.log(message || i);
}
//create a for loop to count from 0 to 100
for (let num = 0; num <= 100; num++){
/**As the count increments, if the number is divisible by 3 and divisible by 5
print FizzBuzz, I have concatenated the number with FizzBuzz for clarity.
Use % instead of \ to ensure it returns an int instead of float.**/
if ((0 == num % 3) && (0 == num % 5)){
console.log ("FizzBuzz" + " " + num);
//otherwise, if the number is divisible by 5 print Buzz
} else if (0 == num % 5) {
console.log("Buzz" + " " + num);
//Also, if the number is divisible by 3 print Fizz
} else if (0 == num % 3){
console.log("fizz" + " " + num);
} else {
//meanwhile, still print all the numbers that don't satisfy the conditions above
console.log (num);
}
}
J'ai écrit quelques variations à ce sujet (en utilisant fizz
et buzz
) comme point de repère pour envisager différentes manières d'itérer la logique conditionnelle.
while
a encore gagné:
// Iterate using a recursive function
// firing a callback once per iteration
function f(s,n) {
if(++n >= 102) return;
s === '' ? console.log(n-1) : console.log(s);
!!(n % 3)
? !!(n % 5)
? f('',n) : f('Buzz',n)
: !!(n % 5)
? f('Fizz',n) : f('FizzBuzz',n);
}
// Iterate using a `while` loop
// firing a callback after satisfying a condition
function b(n) {
var i = n;
$:
while(++i) {
if(i % 3)
if(i % 5)
console.log(i);
else
console.log('Buzz');
else if(i % 5)
console.log('Fizz');
else
console.log('FizzBuzz');
if(i >= 100)
break $;
}
return;
}
// Iterate using a `for` loop
// firing a callback once per iteration
function F(n) {
var i = n, f = 'Fizz', b = 'Buzz', o = '';
for (; i <= 100; i++) {
o = !(i % 3)
? !(i % 5)
? f + b : f
: !(i % 5)
? b : i;
console.log(o);
}
return;
}
// Iterate using a `for` loop
// firing a callback after satisfying a condition
function B(n) {
var i = n;
var fiz = 'Fizz';
var buz = 'Buzz';
for(; i <= 100; i++)
if(!(i % 3))
if(!(i % 5))
console.log(fiz + buz);
else
console.log(fiz);
else if(!(i % 5))
console.log(buz);
else
console.log(i);
return;
}
f('', 1); // recursive
b(0); // `while`
F(1); // `for`
B(1); // `for
Benchmark: http://jsperf.com/fizzbuzz-mod