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

FE 코드피드백 #154

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

FE 코드피드백 #154

crongro opened this issue Feb 14, 2023 · 0 comments
Assignees
Labels
🐱front-end 프론트엔드 관련 🔨 Refactor 코드 리팩토링

Comments

@crongro
Copy link

crongro commented Feb 14, 2023

  • async 함수를 사용하면 await 를 조금더 사용해보시고, catch 절 안에 복잡한 로직은 분리해서 선언해보세요.
  • for 문사용할때 고차함수로 표현가능할지도 보세요.
  • 드문드문 console.log 삭제.
  • forEach처럼 고차함수에 전달되는 함수를 그 안에서 표현하다보니 중첩된 코드가 있네요. 그 함수를 밖에서 정의해서 사용해보세요.
  • 전체적으로 TS적용 시도가 좋군요. interace를 활용해서 파라미터나 class를 좀더 정의해보면 더 좋고요.
@crongro crongro added 🔨 Refactor 코드 리팩토링 🐱front-end 프론트엔드 관련 labels Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐱front-end 프론트엔드 관련 🔨 Refactor 코드 리팩토링
Projects
None yet
Development

No branches or pull requests

1 participant