Фомин Денис #23
Фомин Денис #23
Conversation
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Пройдено тестов 16 из 18 |
|
🍅 Пройдено тестов 16 из 18 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Пройдено тестов 16 из 18 |
|
🍅 Пройдено тестов 16 из 18 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Пройдено тестов 16 из 18 |
|
🍅 Пройдено тестов 16 из 18 |
|
🍅 Пройдено тестов 16 из 18 |
|
🍅 Пройдено тестов 16 из 18 |
|
🍅 Пройдено тестов 17 из 18 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройдено тестов 15 из 15 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройдено тестов 15 из 15 |
| unorderedCollection = unorderedCollection.map(function (item) { | ||
| var idx = defaultCollection.indexOf(item); | ||
|
|
||
| return Object.assign({ position: idx }, item); |
There was a problem hiding this comment.
object.assign в данной задаче использовать нельзя. Предлагается написать свой
|
Почему не хочешь пользоваться локальным линтером? Это быстрее и удобнее |
|
Остались ещё неисправленные замечания. |
|
🚀 |
There was a problem hiding this comment.
В целом, код непонятный и трудночитаемый. Посмотри гайды (https://github.com/urfu-2016/guides/blob/master/codestyle/js.md) и попробуй упростить.
| @@ -0,0 +1,334 @@ | |||
| <html> | |||
| function cloneCollection(collection) { | ||
| var clonedCollection = []; | ||
| collection.forEach(function (item) { | ||
| var newItem = Object.assign({}, item); |
There was a problem hiding this comment.
В данной задаче не используем Object.assign()
| clonedCollection.push(newItem); | ||
| }); | ||
|
|
||
| return clonedCollection; |
There was a problem hiding this comment.
Всё, что делается в этой функции можно реализовать с помощью .reduce()
| return listOfArrays[0].filter(function (item) { | ||
| return listOfArrays.slice(1).some(function (otherArray) { | ||
| return otherArray.indexOf(item) !== -1; | ||
| }) || listOfArrays.length === 1; |
There was a problem hiding this comment.
Так лучше не писать, это нечитаемый код
| }); | ||
| } | ||
| function getUnion(listOfArrays) { | ||
| if (listOfArrays.length === 0) { |
There was a problem hiding this comment.
Вместо listOfArrays.length === 0 можно просто писать listOfArrays.length
|
🍅 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройдено тестов 15 из 15 |
| return previous.concat(current); | ||
| }, []); | ||
| } | ||
| function cloneObject(obj) { |
There was a problem hiding this comment.
Почему бы в этой функции тоже не использовать reduce?
| return clonedObject; | ||
| } | ||
| function getIntersection(listOfArrays) { | ||
| if (listOfArrays.length === 0) { |
| queryType: QUERY_TYPES.FORMAT, | ||
| query: function (collection) { | ||
| var formatted = []; | ||
| collection.forEach (function (item) { |
There was a problem hiding this comment.
Лишний пробел. Плюс тут тоже можно reduce.
| */ | ||
| exports.sortBy = function (property, order) { | ||
| console.info(property, order); | ||
| var SORT_DIRECTION = { |
There was a problem hiding this comment.
Не понял, зачем здесь эта константа? Учитывая, что поле ASCENDING даже не используется. Я бы избавился от неё.
| queryType: QUERY_TYPES.FORMAT, | ||
| query: function (collection) { | ||
| var formatted = []; | ||
| collection.forEach (function (item) { |
|
|
||
| return; | ||
| return newCollection.sort(function (a, b) { | ||
| return orderMultiplyer * (String(a[property])).localeCompare(String(b[property])); |
There was a problem hiding this comment.
Обзательно ли здесь приведение к строке? Кажется, localeCompare сделаем всё за нас.
|
🍅 |
No description provided.