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

Meet 컴포넌트 관심사 분리 리팩토링 필요 #28

Open
ksmfou98 opened this issue Oct 2, 2021 · 1 comment
Open

Meet 컴포넌트 관심사 분리 리팩토링 필요 #28

ksmfou98 opened this issue Oct 2, 2021 · 1 comment
Labels
👨‍💻 작업 중 ! 현재 작업 중입니다. 🔨 리팩토링 코드 리팩토링이 필요해요 !

Comments

@ksmfou98
Copy link
Owner

ksmfou98 commented Oct 2, 2021

지금 소켓 관련된 모든 로직이 Meet 컴포넌트 안에 있다.
채팅, 화상, 유저목록 등등

그러다보니 하나만 바껴도 전부다 랜더링이 된다

컴포넌트를 좀더 잘게 쪼개서 관심사를 분리시켜야 될 거 같다

이렇게 가다간 성능 완전 망가질듯

성능 최적화를 한번 쫙 해주고 개발하는게 나을 거 같다.

@ksmfou98 ksmfou98 added 🔨 리팩토링 코드 리팩토링이 필요해요 ! 🚑 급해요 급해요 빨리 부탁드려요! labels Oct 2, 2021
@ksmfou98
Copy link
Owner Author

ksmfou98 commented Oct 3, 2021

우선 Chat side bar 부분 함수추출하기로 코드 리팩토링 성공. -> 랜더링 최적화됌

@ksmfou98 ksmfou98 added 👨‍💻 작업 중 ! 현재 작업 중입니다. and removed 🚑 급해요 급해요 빨리 부탁드려요! labels Oct 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👨‍💻 작업 중 ! 현재 작업 중입니다. 🔨 리팩토링 코드 리팩토링이 필요해요 !
Projects
None yet
Development

No branches or pull requests

1 participant