search

eliotjang / Movolleh-Movie-Website

use TMDB Open API for team project
0 stars 3 forks source link

feedback #103

Open ggomma opened 6 months ago

ggomma commented 6 months ago

Feedback

총평

개선점

사용하지 않는 코드 최소화

사용하지 않는 코드는 삭제하는 것이 가독성에 좋습니다.

https://github.com/eliotjang/Movolleh-Movie-Website/blob/4f1813e84e5c32720ec5bd8d60960ed928dde65c/src/main.js#L3C27-L3C40 https://github.com/eliotjang/Movolleh-Movie-Website/blob/4f1813e84e5c32720ec5bd8d60960ed928dde65c/src/render.js#L18-L39

중복 코드 최소화

특별한 이유가 있는게 아니라면 아래 경우에는 updateState를 3번 호출하기보다는 한번에 호출하는 편이 좋습니다.

https://github.com/eliotjang/Movolleh-Movie-Website/blob/4f1813e84e5c32720ec5bd8d60960ed928dde65c/src/main.js#L20-L22

중복 코드 최소화

유사한 코드를 여러곳에서 사용하고 있다면 이를 별도의 함수로 구현하는 것을 추천합니다.

https://github.com/eliotjang/Movolleh-Movie-Website/blob/4f1813e84e5c32720ec5bd8d60960ed928dde65c/src/render.js#L103-L111