vk-apps-launch-params
vk-apps-launch-params copied to clipboard
Улучшения в фронтенд коде
trafficstars
@tsivarev знаю, что это не очень популярная демка, поэтому и запрашиваемые изменения неполны.
- Думаю не очень хорошей практикой будет показывать функцию, которая входит в стандарт ECMAScript, а именно
URLSearchParams, хотя я вообще написал отдельный модуль для этого и это как мне кажется, самый удобный способ для работы с параметрами (особенно для новичков). - Также думаю будет хорошо показать необычный паттерн для устранения дубликации кода (один
<Group>вместо несколько). Это очень сильно упрощает рефакторинг. - Совсем не понимаю зачем нужен пример с хэш параметрами?
- Зависимости не обновлялись более двух лет, проект сейчас просто не запустится, однако я не стал трогать эту часть.
- Думаю, было бы не плохо изменить расширения на
tsxну илиjsxуж как минимум. Стандарт же... - Ну и для упрощения было бы не плохо превратить
Appв функциональный компонент. Совсем не понимаю, зачем React еще держит документацию по классовым...
Извиняюсь за предоставление информаци в настолько неструктурированном виде, надеюсь, что всё более менее понятно 😃