Козлов Александр #63
Козлов Александр #63
Conversation
|
🍏 Пройдено тестов 15 из 15 |
| var changedCollection = collection.map(function (el) { | ||
| return Object.assign({}, el); | ||
| }); | ||
| functions.sort(function (first, second) { |
There was a problem hiding this comment.
Для функции сортировки можно параметры называть a и b
| var fieldsToChoose = [].slice.call(arguments); | ||
|
|
||
| return function select(collection) { | ||
| return collection.map(function (el) { |
There was a problem hiding this comment.
el не несет никакой семантики, лучше подойдет название вроде person
|
|
||
| return function select(collection) { | ||
| return collection.map(function (el) { | ||
| return fieldsToChoose.reduce(function (newCollection, property) { |
There was a problem hiding this comment.
Первый аргумент назвал newCollection, но ведь это не коллеция, а объект с информацией о человеке
| console.info(property, values); | ||
|
|
||
| return; | ||
| return function filterIn(collection) { |
There was a problem hiding this comment.
Снова el никак не отражает того, что лежит внутри (в общем везде el нужно поменять)
|
|
||
| return; | ||
| return function sortBy(collection) { | ||
| var changedCollection = collection.sort(function (first, second) { |
There was a problem hiding this comment.
Зачем еще одну переменную создавать? Можно просто collection.sort()
| return function sortBy(collection) { | ||
| var changedCollection = collection.sort(function (first, second) { | ||
|
|
||
| return (first[property] > second[property]); |
There was a problem hiding this comment.
Функция сравнения должна возвращать 1 (если first > second по некоторому критерию), 0 (если равны) и -1 (если меньше)
|
🍅 |
3 participants
No description provided.