asd95 / tt-adminui

0 stars 0 forks source link

Reducing code sections #14

Open aandrey1993 opened 3 years ago

aandrey1993 commented 3 years ago

Некоторые участки кода можно сократить без потери функциональности. Например:

изображение

изображение

Ещё пример упрощение проверок переменной на пустоту:

data.idnp === undefined || data.idnp === "" -> !data.idnp

asd95 commented 3 years ago

data.idnp === undefined || data.idnp === "" -> !data.idnp Вот за это отдельное спасибо. Не подумал, но понимаю как работает. Что касается вверхнего кода. useEffect(() => { let canceled = false; const fetchService = () => { service .getMonitoringData(dateRange) .then((fetchData) => !canceled && setMonitoringData(fetchData)); }; fetchService(); return () => (canceled = true); }, [dateRange, service]); Я хотел написать все с помощью одного хука и функция нужна была чтобы вызывать ее при интервале и при первоначальном рендеринге. Сделал грубо говоря чтобы не копипастить код. Результат не тот. Легаси код так и остался не стал переделывать, и так все работало)