FizzBuzz жду критики

Курсы Основы JavaScript
1

Правильно ли все сделал, или можно идеальнее?

var number = 15;
var taskResult;
if((!number % 3 == 0) && (!number % 5 == 0)){
taskResult = number;
}
if(number % 3 == 0){
taskResult = ‘Fizz’;
}
if(number % 5 == 0){
taskResult = ‘Buzz’;
}
if(number % 5 == 0 && number % 3 == 0){
taskResult = ‘FizzBuzz’;
}

2

можно лучше, в конце прикреплю вам мемес на эту тему:

  • не надо делать 3 независимых проверки, лучше
    одну громоздкую конструкцию через else if
  • сравнивать с нулем в предикате попахивает индусятиной
  • можно дефолтно присвоить результату number, чтобы не делать лишнего ветвления

мемес про предикат

image
image.jpg570×736 85.2 KB

открывать в случае полового бессилия 
var number = 15;
var taskResult = number;

if (!(number % 3) && !(number % 5)) {
  taskResult = 'FizzBuzz';
} else if (!(number % 3)) {
  taskResult = 'Fizz';
} else if (!(number % 5)) {
  taskResult = 'Buzz';
}

а так, многие придерживаются мнения - работает - ну и ладно.

1 лайк
3

Если бы я хотел учиться по способу - работает и “ок”.Я бы сюда не заходил даже, я хочу чтобы работало ПРАВИЛЬНО, но я не знаю как узнать это “правильно”.мб сайт learn.js мне в помощь

4

да, неплохой выбор, с учетом того, что недавно они актуализировали учебник под es8