Conversation
tasks/basics/nodeTask.js
Outdated
| class ArrayAdvanced extends Array { | ||
| constructor(array) { | ||
| super(); | ||
| this.array = array.slice(); |
There was a problem hiding this comment.
Конструктор нужно переписать. Он должен иметь тот-же контракт, что и обычный Array. Сейчас - это объект со свойством array
tasks/basics/nodeTask.js
Outdated
| this.array = array.slice(); | ||
| } | ||
| specialSort(collection) { | ||
| if (collection.hasOwnProperty('sort')) collection.sort(this.array); |
There was a problem hiding this comment.
Функция сортировки, а так же все остальные функции из опционального объекта - функции, которые перебирают один элемент массива. Они не должны сортировать сразу весь массив
There was a problem hiding this comment.
как функция сортировки будет сортировать один элемент массива?
There was a problem hiding this comment.
Почитай про колбек в Array.prototype.filter: https://www.ecma-international.org/ecma-262/8.0/#sec-array.prototype.filter
machendos
left a comment
machendos
left a comment
There was a problem hiding this comment.
Всё ок, только куда делся фильтр?)
|
Как тебе такое Илон Маск? Функциональность и подход к задаче |
|
Переименуй класс в |
|
методы в unsideTransform для неизменяющего массив метода, метод не изменяет массив в классе? он возвращает новый измененный массив? |
|
Он не изменяет массив, однако возвращает массив-результат: первый и/или последний элемент массива, или вообще пустой массив |
|
Можно реализовать только один, но назови его правильно |
|
А файл в |
|
|
2 participants
Гит Игнор не ставил на лог файл, а так сделал как понял, жду вразумительных ревьюх.