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

코드 피드백 #94

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

코드 피드백 #94

crongro opened this issue Feb 13, 2023 · 0 comments
Labels
Frontend Frontend

Comments

@crongro
Copy link

crongro commented Feb 13, 2023

  • userService.js
o = {
  () {},(){}
}
  • user.js
    catch 문에서의 에러처리가 꼼꼼해서 좋고요. 그 내용을 별도 함수로 분리해도 더좋을듯

  • makeTimer.js
    함수를 그 안에서 중첩된 표현으로 했는데, 그럴필요는 없어요. 분리해서 선언하셔도 됩니다.
    어짜피 makrttimer 에서 export하지 않으면 접근도 안되고요.
    displayTimeLeft함수보니 classList 추가하는 중복코드가 보이네요. 작은 함수로 분리해서 중복을 없애보세요.

  • pages/*.pages.js
    async로 함수를 처리하는데 장점을 모르겠습니다. 동기함수로 표현해도 될 거 같아요.

  • /scripts 라는 디렉토리보다 유틸리티면 utils 정도의 의미를 부여해보세요.

page, component로 분리해서 모듈을 만든것이 좋아보이네요.

@crongro crongro added the Frontend Frontend label Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Frontend
Projects
None yet
Development

No branches or pull requests

1 participant