Новоселова Наталья#57
Conversation
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройдено тестов 15 из 15 |
| exports.isStar = true; | ||
| exports.isStar = false; | ||
|
|
||
| var PRIORITET = ['filterIn', 'sortBy', 'select', 'limit', 'format']; |
There was a problem hiding this comment.
Прочитай гайды по оформлению, пожалуйста. Нельзя называть переменные транслитом
|
|
||
| var PRIORITET = ['filterIn', 'sortBy', 'select', 'limit', 'format']; | ||
|
|
||
| function copyCollection(array) { |
There was a problem hiding this comment.
array -- плохое название переменной
оно не отражает сути
|
|
||
| var PRIORITET = ['filterIn', 'sortBy', 'select', 'limit', 'format']; | ||
|
|
||
| function copyCollection(array) { |
There was a problem hiding this comment.
Ты тут возвращаешь значение, названия таких функций принято начинать со слова get
| exports.query = function (collection) { | ||
| return collection; | ||
| var copy = copyCollection(collection); | ||
| var func = [].slice.call(arguments, 1); |
| */ | ||
| exports.query = function (collection) { | ||
| return collection; | ||
| var copy = copyCollection(collection); |
There was a problem hiding this comment.
copy? Что можно понять по переменной, которая называется копия?
| exports.sortBy = function (property, order) { | ||
| console.info(property, order); | ||
| return function sortBy(collection) { | ||
| var newCollection = copyCollection(collection); |
There was a problem hiding this comment.
Новая коллекция? Почему она новая? Чем отличается от старой?)
| console.info(property, formatter); | ||
| return function format(collection) { | ||
| return collection.map(function (element) { | ||
| var copy = Object.assign({}, element); |
There was a problem hiding this comment.
С copy та же самая история, что и выше
|
🍅 |
|
🍏 Пройдено тестов 15 из 15 |
|
🍏 |
|
🚀 |
|
|
||
| function getCopyCollection(collection) { | ||
| return collection.map(function (element) { | ||
| return Object.assign({}, element); |
There was a problem hiding this comment.
Object.assign -- это ES2015. Мы пока остаёмся в рамках ES5, так что эту функцию использовать нельзя
| }) | ||
| .forEach (function (query) { | ||
| copyCollection = query(copyCollection); | ||
| }); |
There was a problem hiding this comment.
В качестве упражнения можете этот кусочек переделать на reduce, тогда можно будет переменную copyCollection не изменять. Но тут я не настаиваю.
| } | ||
|
|
||
| return; | ||
| return first[property] < second[property]; |
There was a problem hiding this comment.
sort ожидает, что компаратор возвращает число. А у вас возвращается логическое значение. И все значения, для которых условие не выполняются, преобразуются в 0, означающее, что значения равны. Так что функция сортировки корректно у вас работать не будет.
| console.info(property, formatter); | ||
| return function format(collection) { | ||
| return collection.map(function (element) { | ||
| var copyCollection = Object.assign({}, element); |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Пройдено тестов 12 из 15 |
|
🍅 Пройдено тестов 12 из 15 |
|
🍅 Пройдено тестов 11 из 15 |
4 participants
No description provided.