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

[팩토링] 코드 피드백 #86

Open
crongro opened this issue Feb 13, 2023 · 0 comments
Open

[팩토링] 코드 피드백 #86

crongro opened this issue Feb 13, 2023 · 0 comments
Labels
refactor 머지 및 기타 이유에 의한 리팩토링

Comments

@crongro
Copy link

crongro commented Feb 13, 2023

🥞 요약

  • action/reducer 방식시도가 좋네요. 장점이 무엇인지 생각해보시고, 코드에 안쓰는 주석은 삭제를.
  • app.ts의 init은 더 간단하게 표현하면 좋으니 하위함수로 더 분리해보세요.
  • store.subscribe, foreach 등의 고차함수들은 가급적 함수선언을 별도로 하고 그걸 가져와서 쓰세요. 직접 함수를 선언하지 말고요.
  • 역시 불필요한 주석이 여기저거 너무 많아서 코드를 읽는것이 어렵습니다.
  • 전체적으로 일관성 있는 코드패턴이 보여서 좋아요.
@crongro crongro added the refactor 머지 및 기타 이유에 의한 리팩토링 label Feb 13, 2023
@tdmdgh tdmdgh closed this as completed Feb 22, 2023
@tdmdgh tdmdgh reopened this Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 머지 및 기타 이유에 의한 리팩토링
Projects
None yet
Development

No branches or pull requests

2 participants