Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvements #22

Open
mr47 opened this issue Apr 2, 2018 · 1 comment
Open

Improvements #22

mr47 opened this issue Apr 2, 2018 · 1 comment

Comments

@mr47
Copy link

mr47 commented Apr 2, 2018

Привет, крайне интересный проект.

Есть пару идей для улучшения кода и просторы работы:

  • React Router - очень просто его использовать
  • Redux - очень просто и очень удобно (single source of thuth самое оно для такого проекта)
  • Отказаться от singleton класса т.к получаешь супер полиморфтные обьекты тыц
  • Разделить компоненты и логику т.к все очень в каше
  • Использовать Styled-components
  • Использовать scss aka sass т.к проще и удобнее.
  • Рефакторнуть евенты с сокетами, window не лучшая идея.
  • Lodash решит тысячи проблем что есть в коде (костыли с мерджингом обьектов и прочим )
  • Отдельно про forceUpdate, ну не нужен он вам :)
  • window не друг вам :)

PS: Если не планируешь много переписывания то могу сделать пару ПРов. Но они будут очень жесткие...

Удачи!

@mr47 mr47 changed the title Improvments Improvements Apr 2, 2018
@DEgITx
Copy link
Owner

DEgITx commented Apr 2, 2018

Ну я там особо сильно не планировал переписывать в клиентской части, только думал насчет нормального форматера кода какого общего - но это может потом.
Если есть желание можешь взяться за что-то, тем более я не люблю визуальную составляющую, только постарайся через четкие пул реквесты желательно с максимальным раделением задач, и желательно на каждое за что берешься по тикету, чтобы я был уверен что не пересечется. Я понимаю что они скорее всего по половине таксков выйдут большие, к примеру по тому же редуксу, но если есть возможность - сокращай и дели, будет проще вливать и читать. Если будут вопросы, меня легко найти по degitx нику.
В стайлед компонентах помню какие-то траблы могут быть на уровне билда релизной версии, по крайнемере в том пакете что юзали, в одном из проектов помню натыкались, нужно еще и это чекать.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants