Create Vadim_Moshev.js #10
Conversation
|
Issue #8 |
| @@ -0,0 +1,7 @@ | |||
| // Solution by Vadim Moshev special for Anton Code It channel | |||
| function sum(...args) { | |||
There was a problem hiding this comment.
На мой взгляд, в данной задаче нет смысла смешивать объявление функций с помощью function и стрелочных функции. Если выбрать какой-то один вариант можно сделать код немного чище.
There was a problem hiding this comment.
По условию задачи в функцию всегда приходит не больше двух аргументов. Кажется, нет смысла собирать их в args, а можно просто объявить, например, как x и y и дальше работать с ними. Так мы всегда будем знать какие и сколько аргументов нам приходит.
| @@ -0,0 +1,7 @@ | |||
| // Solution by Vadim Moshev special for Anton Code It channel | |||
| function sum(...args) { | |||
| return ((args[1] !== undefined) | |||
There was a problem hiding this comment.
Скобки вокруг всего выражения лишние. Их можно убрать и ничего такого не произойдёт.
There was a problem hiding this comment.
Можно инвертировать условие, а ветки поменять местами. Иногда их проще читать.
Here is my solution for your Argument Trouble task