[bug]: useQueue

[wmoooid]

Писал тесты для useQueue и нашел баг:
Функция remove возвращает undefined вместо удаленного значения.

Текущая реализация:

  const remove = () => {
    let removed;
    setQueue(([first, ...rest]) => {
      removed = first;
      return rest;
    });
    return removed as Value;
  };

Возможный фикс:

  const remove = () => {
    const [first, ...rest] = queue;
    setQueue(rest);
    return first;
  };

При такой реализации бага нет, но смущает, что берем очередь напрямую из стейта, а не из сетстейта. Что думаете?

[wmoooid]

Как вариант можно ещё что-то такое сделать:

  const remove = () => {
    setQueue(([, ...rest]) => rest);
    return queue[0];
  };

[debabin]

@wmoooid Забавный баг, с учетом того, что такая реализация у многих, баг понятен return выполняется первеe чем setState

Такая же бага uidotdev/usehooks#276

[debabin]

Тут вероятнее всего такая же бага https://github.com/streamich/react-use/blob/master/src/useQueue.ts#L17

[debabin]

Все твои варианты с огрехом, если вызывать по 2 remove подряд, то получаем баг

[debabin]

Залил фикс, прошу закрыть issue после проверки

[wmoooid]

Проверил, работает как надо