vk-apps-launch-params icon indicating copy to clipboard operation
vk-apps-launch-params copied to clipboard

Улучшения в фронтенд коде

Open zardoy opened this issue 4 years ago • 1 comments
trafficstars

@tsivarev знаю, что это не очень популярная демка, поэтому и запрашиваемые изменения неполны.

  • Думаю не очень хорошей практикой будет показывать функцию, которая входит в стандарт ECMAScript, а именно URLSearchParams, хотя я вообще написал отдельный модуль для этого и это как мне кажется, самый удобный способ для работы с параметрами (особенно для новичков).
  • Также думаю будет хорошо показать необычный паттерн для устранения дубликации кода (один <Group> вместо несколько). Это очень сильно упрощает рефакторинг.
  • Совсем не понимаю зачем нужен пример с хэш параметрами?
  • Зависимости не обновлялись более двух лет, проект сейчас просто не запустится, однако я не стал трогать эту часть.
  • Думаю, было бы не плохо изменить расширения на tsx ну или jsx уж как минимум. Стандарт же...
  • Ну и для упрощения было бы не плохо превратить App в функциональный компонент. Совсем не понимаю, зачем React еще держит документацию по классовым...

zardoy avatar Jan 12 '21 06:01 zardoy

Извиняюсь за предоставление информаци в настолько неструктурированном виде, надеюсь, что всё более менее понятно 😃

zardoy avatar Jan 12 '21 06:01 zardoy