Ошибка диапазона: недопустимая длина массива при разбросе массива

#javascript #arrays #typescript #array-splice

#javascript #массивы #typescript #массив-сращивание

Вопрос:

У меня есть довольно стандартная функция гидратации элемента, которая выдает «Ошибка диапазона: недопустимая длина массива«. К сожалению, это случается очень редко и в процессе производства, поэтому трудно перехватить входные параметры.

Это функция:

 function setProgress(items: SomeType[], id: string, someProperty: string) {
  const index = items.findIndex(item => item.id === id);
  const newItem: SomeType = {...items[index], someProperty };

  return [...items.slice(0, index), newItem, ...items.slice(index   1); 
}
  

И это то, что передается (без искажения):

 function setProgress(items, id, someProperty) {
  var index = items.findIndex(function(e) {
      return items.id === id
    }), 
    newItem = Object.assign({}, items[index], {
      someProperty: someProperty
    });
  return items.slice(0, index).concat([newItem], items.slice(index   1));
}
  

Я пробовал играть с другими значениями, но я не могу воспроизвести эту ошибку.

Есть идеи, что и как могло вызвать это?

Комментарии:

1. Вы уверены, что ошибка в этой функции? «Недопустимая длина» возникает только тогда, когда вы выполняете Array(-1) или .length=-1 , ни то, ни другое не имеет места.

2. Это то, что меня озадачивает. Chrome сообщает об ошибке в строке возврата. И эта функция является частью редуктора, поэтому входные данные довольно строго контролируются.

3. Когда я смотрю на сокращенный код, единственный массив (o) можно найти в reflective-injector от angular и RxJS forkJoin и withLatestFrom .

Ответ №1:

Проблема заключалась в необработанном угловом случае, когда индекс -1 . Вместо замены элемента это удвоило бы размер массива. Это в конечном итоге привело бы к переполнению стека и превышению предела размера массива.

Ответ №2:

У меня возникла эта проблема в следующем сценарии при попытке распространить объект, подобный массиву:

 let borken = [...this.state.data.get("tags"), newTag]

// immutablejs toJS() converts the array-like into an array.
let worken = [...this.state.data.get("tags").toJS(), newTag]
  

Несколько примеров того, как получить эту ошибку:

 // example 1
Array(NaN)

// example 2
let a = []
a.length = a.length  - 1

// example 3 setting .length to > 2^32
let b = [];
b.length = Math.pow(2, 32);
  

В моем случае это был вызов следующего __spreadArrays :

 var __spreadArrays = (undefined amp;amp; undefined.__spreadArrays) || function () {
    for (var s = 0, i = 0, il = arguments.length; i < il; i  ) s  = arguments[i].length;
    for (var r = Array(s), k = 0, i = 0; i < il; i  )
        for (var a = arguments[i], j = 0, jl = a.length; j < jl; j  , k  )
            r[k] = a[j];
    return r;
};

  

Где var r = Array(s) и s было NaN