Вихарев Вячеслав #25
Вихарев Вячеслав #25
Conversation
|
🍅 Пройдено тестов 14 из 18 |
|
🍏 Пройдено тестов 18 из 18 |
|
@chipolinka обрати внимание решено доп. задание |
|
|
||
| // Constants | ||
| // | ||
| var FUNC_ORDER = [ |
There was a problem hiding this comment.
Название не совсем корректно отображает содержимое
| ]; | ||
|
|
||
|
|
||
| // Functions-helpers |
There was a problem hiding this comment.
Если хочешь писать доки -- у тебя есть пример, как их оформлять)
There was a problem hiding this comment.
Это не доки, это просто разделение на логические части, просто для того чтоб удобнее было ориентироваться
There was a problem hiding this comment.
Мешают ориентироваться в коде
Штуки полезные, понимаю
|
|
||
| // Functions-helpers | ||
| // | ||
| var _functionSorter = function (one, another) { |
There was a problem hiding this comment.
Можешь объяснить, почему у тебя названия функций начинаются с земли?)
There was a problem hiding this comment.
Это функции, которые не предполагается использовать где-то еще, кроме как в этом коде, тогда как функции из следующего блока можно использовать где-то еще
There was a problem hiding this comment.
И по правилам название функции должно начинаться с глагола.
There was a problem hiding this comment.
Ну не всегда
"Правила" эти довольно условны
Кажется логичнее применять "сортировщик" к коллекции, чем применять к ней "отсортировать"
There was a problem hiding this comment.
Я бы всё равно избавилась на твоем месте от земли.
| */ | ||
| exports.select = function () { | ||
| return; | ||
| var _functionApplyer = function (obj, f) { |
There was a problem hiding this comment.
Я ее передаю как аргумент в <список функций>.map с присвоенный объектом
Так для каждой функции она будет применена к объекту
| var key = keyValuePair[0]; | ||
| var value = keyValuePair[1]; | ||
|
|
||
| var newObject = Object.assign({}, obj); |
There was a problem hiding this comment.
В ES5.1 еще нет Object.assign, так что нельзя использовать готовый вариант)
|
В целом код 👍 , но вот есть немного замечаний) |
|
🍅 |
3 participants
No description provided.