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 코드 피드백 #111

Open
crongro opened this issue Feb 13, 2023 · 1 comment
Open

FE 코드 피드백 #111

crongro opened this issue Feb 13, 2023 · 1 comment
Labels
🙋‍♂️ Question 질문 & 고민

Comments

@crongro
Copy link

crongro commented Feb 13, 2023

  • 함수적 표현 시도가 좋아요.

  • 함수 이름 위주로 코드를 표현하면 더 좋긴함.
    가능할지 모르겠지만 아래와 같은 식으로 함수이름만 노출.
    before
    _.map((input) => inputForm({ ...input, target: '.auth-form' })),

    after
    _.map(inputForm),

  • /auth/*.js 중에 함수 하나만 덩그러니 있는 파일도 있던데 파일을 합쳐도 될 듯.

  • whiteTemp 라는 단어보다 whiteHeaderTpl 이라는 이름이 더 많이 쓰이는거 같네요.(Temp가 temp라는 단어같아서요)

  • /utils/slider 같은 코드는 component 아래 재사용부분에 위치해도 될 듯하네요. 그리고 cardSlider() 함수는 _.go() 안에 너무 많은 부분이 연결되어 있어서 함수형표현이지만 어떤 일이 일어나느지 알기 어려워 보입니다.

  • 전체적으로 페이지/컴포넌트 일관성 있는 것 같아서 좋아요.

  • 구현하신 함수 유틸리티 코드들 잘 이해하고 계시면 좋겠고요.

@crongro crongro added the 🙋‍♂️ Question 질문 & 고민 label Feb 13, 2023
@SUMMERLOVE7
Copy link
Collaborator

피드백 감사합니다😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🙋‍♂️ Question 질문 & 고민
Projects
None yet
Development

No branches or pull requests

2 participants