RangeError: недопустимая длина массива при распространении массива

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

Это функция:

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));
}

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

Любая идея, что и как могло вызвать это?

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

georg 28.03.2019 11:13

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

Miroslav Jonas 28.03.2019 11:15

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

Miroslav Jonas 28.03.2019 11:37
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
4
3
3 494
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 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 && 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

Ответ принят как подходящий

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

Другие вопросы по теме