Je dois faire un FizzBuzz pour un entretien d'embauche (du moins je sais qu'ils vont me poser cette question), donc j'ai fait ça :

Première technique : "entreprise"

const FizzBuzz = (rule, iterations) => {
	for (let i = 0; i <= iterations; i++) {
		let output = ""
		for (let r in rule)
			if (i % r === 0) { output += rule[r] }
		console.log(output ? output : i)
	}
}

const rule = {
	3:'Fizz',
	5:'Buzz'
}

FizzBuzz(rule, 100)

Deuxième technique : "simple et propre"

for (let i = 0; i <= 100; i++) {
	let output = ""
	if (i % 3 === 0) { output += "fizz" }
	if (i % 5 === 0) { output += "buzz" }
	console.log(output ? output : i)
}

Est-ce que vous avez des améliorations à proposer et quelle technique pensez-vous est la meilleure et laquelle présenter lors d'un entretien ?

la plus simple toujours
et puis commence a i = 1

J'ai pris l'habitude de commencer à compter à partir de zero mais okay. D'autres remarques ?

Le 26 janvier 2018 à 17:34:47 Caecilius a écrit :
J'ai pris l'habitude de commencer à compter à partir de zero mais okay. D'autres remarques ?

Le test demande généralement les nombres de 1 à 100, mais en commençant à 0 et en ayant comme condition i <= 100, tu fais 101 itérations. (Après, ça dépend toujours de comment le problème sera posé.)

Oui tu as raison ce n'est pas logique de demander 100 itérations et d'en avoir 101, merci pour le signaler... D'autres remarques ?

Salut, t'as pas mis le cas où i % 5 == 0 and i % 3 == 0 : print("FizzBuzz")

Le 26 janvier 2018 à 18:54:04 Exacompta a écrit :
Salut, t'as pas mis le cas où i % 5 == 0 and i % 3 == 0 : print("FizzBuzz")

mais non pas besoin

Le 26 janvier 2018 à 18:54:04 Exacompta a écrit :
Salut, t'as pas mis le cas où i % 5 == 0 and i % 3 == 0 : print("FizzBuzz")

tu as survolé le code sans le lire

Le 26 janvier 2018 à 18:59:43 Caecilius a écrit :

Le 26 janvier 2018 à 18:54:04 Exacompta a écrit :
Salut, t'as pas mis le cas où i % 5 == 0 and i % 3 == 0 : print("FizzBuzz")

tu as survolé le code sans le lire

Désolé je connais pas du tout ce langage

Si l'entreprise utilise correctement ce test, elle se servira de ta réponse pour ensuite te poser des questions et voir avec toi pourquoi as-tu fait certains choix.

Pour ce type de test, et de question précise, je ne t'aiderai pas sur la partie technique.
Je te dirai juste :
- faire ce qui est demandé : je préfère quelqu'un un peu faiblard techniquement a quelqu'un qui n'arrive pas à comprendre un énoncé aussi simple. Exemple ici (qui a été relevé, donc je le reprends) : quelqu'un qui n'utilise pas les bonnes bornes car il "a l'habitude" d'autre chose va clairement me faire perdre du temps lorsque je lui fournirai des specs ainsi que sur la recette. C'est une personne qui manque de rigueur et a du mal à réaliser des solutions répondant parfaitement au besoin.
- penser à la maintenance : "développe comme si la personne qui allait reprendre ton code est un psychopathe qui connait ton adresse"
- pouvoir justifier tes choix
- faire un clair simple, facile, lisible (ça rejoint "penser à la maintenance")

- penser à la maintenance : "développe comme si la personne qui allait reprendre ton code est un psychopathe qui connait ton adresse"

ça par contre c'est totalement con, tu ne peux pas te rabaisser au plus petit dénominateur commun quand tu dev

C'est pas mal mal ça fait pas très "corporate"
Je te conseil d'étudier en détail ce projet qui va te permettre d'intégrer tous les design patterns : https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition

@Caecilius : je ne comprends absolument pas ta remarque. Peux-tu l'expliciter s'il te plaît ?

Le 27 janvier 2018 à 18:00:11 ACanOfPickIes a écrit :
C'est pas mal mal ça fait pas très "corporate"
Je te conseil d'étudier en détail ce projet qui va te permettre d'intégrer tous les design patterns : https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition

Le développement entreprise bordel

C'est pas mal mal ça fait pas très "corporate"

Je te conseil d'étudier en détail ce projet qui va te permettre d'intégrer tous les design patterns : https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition

production ready