Как привести к более красивому виду код? Я реализовал через if else, но выглядит достаточно убого:
const makeRecomendations = (wordsPerMinute, newAverage) => {
if (wordsPerMinute <= 119 && (newAverage >=5 && newAverage <=8) && output.length < 1) return 'Увеличьте скорость речи'
else if (wordsPerMinute <= 119 && newAverage <=5 && output.length < 1) return 'Увеличьте скорость речи\nГоворите громче'
else if (wordsPerMinute <= 119 && newAverage >=8 && output.length < 1) return 'Увеличьте скорость речи\nГоворите тише'
else if (wordsPerMinute <= 119 && (newAverage >=8) && output.length >= 1) return 'Увеличьте скорость речи\nГоворите тише\nКонтролируйте употребление слов паразитов'
else if (wordsPerMinute <= 119 && (newAverage <=5) && output.length >= 1) return 'Увеличьте скорость речи\nГоворите громче\nКонтролируйте употребление слов паразитов'
else if (wordsPerMinute <= 119 && (newAverage >=5 && newAverage <=8) && output.length >= 1) return 'Увеличьте скорость речи\nКонтролируйте употребление слов паразитов'
else if (wordsPerMinute >= 180 && (newAverage >=5 && newAverage <=8) && output.length < 1) return 'Не торопитесь во время доклада'
else if ((wordsPerMinute >= 119 && wordsPerMinute <= 180) && newAverage <=5 && output.length < 1) return 'Говорите громче'
else if ((wordsPerMinute >= 119 && wordsPerMinute <= 180) && newAverage >=8 && output.length < 1) return 'Говорите тише'
else if ((wordsPerMinute >= 119 && wordsPerMinute <= 180) && (newAverage >=5 && newAverage <= 8) && output.length >= 1) return 'Контролируйте употребление слов паразитов'
else if ((wordsPerMinute >= 119 && wordsPerMinute <= 180) && newAverage <=5 && output.length >= 1) return 'Говорите громче\nКонтролируйте употребление слов паразитов'
else if (wordsPerMinute >=180 && newAverage <=5 && output.length >= 1) return 'Не торопитесь во время доклада\nГоворите громче\nКонтролируйте употребление слов паразитов'
else if (wordsPerMinute >=180 && (newAverage >=5 && newAverage <= 8) && output.length < 1) return 'Не торопитесь во время доклада'
else if (wordsPerMinute >=180 && (newAverage >=5 && newAverage <= 8) && output.length >= 1) return 'Не торопитесь во время доклада\nКонтролируйте употребление слов паразитов'
else if (wordsPerMinute <= 119 && (newAverage >=5 && newAverage <= 8) && output.length >= 1) return 'Увеличьте скорость речи'
else return "Вы потрясающий спикер!"
}
else
. У вас после каждого условия естьreturn
, значит в последующие if-ы вы и так не попадете, если какое-либо из условий сработает.