Pull to refresh

Comments 5

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


Пока этот показался очень удобным решением в несколько строчек, если нет желания и необходимости тащить в проект дополнительные зависимости для denouncing’га.

Маленькое уточнение: в вашей реализации есть небольшой изъян в виде возможной гонки. Скажем пользователь набрал "а", пошёл запрос, пользователь набрал "б", пошёл ещё запрос. Второй запрос пришёл раньше первого, первый пришёл вторым. Получаем ерунду.


Нюанс второй: не учтён вариант когда запрос завершился тогда, когда компонент уже умер. Получаем setState для размонтированного компонента. Warning. Тоже мелочь, но всё же.

Резонные замечания. Могу предположить, что автор оригинальной статьи не хотел перегружать пример другими деталями и сконцентрировать внимание на использовании хука, без других подробностей. Это только предположение, но выглядит логичным.

Sign up to leave a comment.

Articles