Использование метода .splice в подклассе массива в javascript?
Я пытаюсь создать подкласс массива javascript. Я хочу инициировать подкласс с аргументом типа массива и добавить метод для удаления элемента из массива (подкласса).
Мой код выглядит так:
class CustomArray extends Array {
constructor(array) {
console.info('Initiating array:', array)
super(...array);
}
remove(element) {
let index = this.indexOf(element);
if (index > -1) {
return this.splice(index, 1);
}
return [];
}
}
var a = ['a', 'b', 'c', 'd', 'e'];
var list = new CustomArray(a)
console.info('list:', list);
console.info('remove:', list.remove('c'));
console.info('list:', list);Проблема в том, что когда я вызываю .splice (), он удаляет элемент из массива, но также возвращает массив удаленных элементов, на самом деле он возвращает новый экземпляр моего подкласса CustomArray, который должен быть инициирован с аргументом типа массива, но .splice () инициирует его с аргументы целочисленного типа.
Вот пример того, что, как мне кажется, происходит, когда я вызываю .splice ():
Допустим, у нас есть экземпляр list класса CustomArray, инициированный аргументом ['a','b','c','d','e'], а затем мы вызываем метод list.remove('c'). (Как в фрагменте кода) .
Метод remove класса CustomArray проверяет, что индекс c находится в массиве ['a','b','c','d','e'], который является 2, а затем вызывает method this.splice(2,1), который должен удалить 1 элемент с индексом 2 в массиве ['a','b','c','d','e']. Метод splice удаляет элемент c формы массива, но также возвращает что-то вроде new CustomArray(1), потому что один элемент был удален из массива, поэтому он пытается создать массив длиной 1, но это терпит неудачу, потому что класс CustomArray является массивом expectiong.
Я хочу, чтобы метод splice не инициировал новый экземпляр класса CustomArray и вместо этого возвращал обычный массив (экземпляр объекта Array).
CustomArray или CostomArray?
Спасибо за исправление на ++index, но я использовал цикл вместо while. И это CustomArray, я редактировал это в вопросе.
![Безумие обратных вызовов в javascript [JS]](https://i.imgur.com/WsjO6zJb.png)
Ответы 2
I want to prevent
splicemethod from initiating a new instance ofCostomArrayclass and instead return normal array (an instance ofArrayobject).
Затем вам нужно создать другой метод с другим именем. Семантика splice: четко и точно определено; они составляют контракт на тип Array. Если ваш CostomArray нарушит этот контракт, это будет означать, что это больше не Array, это что-то подобный массиву и не должно расширять Array.
Поскольку ваш метод называется remove, все в порядке; если вы хотите, чтобы remove возвращал Array, а не CostomArray, вам просто нужно реализовать логику самостоятельно:
remove(element) {
let index = this.indexOf(element);
if (index > -1) {
const newLength = this.length - 1;
while (index < newLength) {
this[index] = this[index + 1];
++index;
}
this.length = newLength;
return [element];
}
return [];
}
В качестве альтернативы, конечно, заставьте конструктор CostomArray работать правильно при вызове различными методами Array.prototype. (Тот, который вы указали в вопросе, работает нормально, за исключением регистрации чего-то, чего вы не ожидаете, с помощью console.info.)
Возможно, splice вернет стандартный массив без вызова конструктора. Это происходит путем изменения @@species недвижимость вашего пользовательского класса, который определяет, какой конструктор будет использоваться. Но имейте в виду, что это повлияет не только на splice, но и на другие методы все, которые будут создавать новый экземпляр, включая map, filter, slice, ...
Вы можете изменить свойство @@species, перезаписав соответствующий статический геттер:
class CustomArray extends Array {
static get [Symbol.species]() { return Array; } // <-----------
constructor(array) {
console.info('Initiating array:', array)
super(...array);
}
remove(element) {
let index = this.indexOf(element);
if (index > -1) {
return this.splice(index, 1); // Now calls Array constructor, not CustomArray
}
return [];
}
}
var a = ['a', 'b', 'c', 'd', 'e'];
var list = new CustomArray(a)
console.info('list:', list);
console.info('remove:', list.remove('c'));
console.info('list:', list);
// Some examples of effects on other methods
console.info(list.map(x => x) instanceof CustomArray); // false
console.info(list.filter(x => 1) instanceof CustomArray); // false
console.info(list.slice() instanceof CustomArray); // false
console.info(list.concat() instanceof CustomArray); // false
// Other methods, that do not return a new instance, are not affected:
console.info(list.reverse() instanceof CustomArray); // true
console.info(list.sort() instanceof CustomArray); // true
FWIW, я только что заметил ошибку в коде в моем ответе, который я исправил (отсутствует строка
++index;).