-
Notifications
You must be signed in to change notification settings - Fork 113
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
부산대 BE_이풍헌 6주차 과제(step3) #260
base: canyos
Are you sure you want to change the base?
Conversation
canyos
commented
Aug 2, 2024
- point기능을 팀원들과 명세를 맞춰 추가했습니다.
…nto step3 # Conflicts: # src/main/java/gift/dto/wish/WishPageDTO.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다! 마지막 리뷰겠군요! 마지막이니 코드에 대한 세밀한 리뷰 보다는 앞으로 코드나 개발 하실 때 방향성을 얘기해주는 게 더 좋을 거 같네요 ㅎㅎ
저는 리뷰를 하면서 제일 중요하게 생각하는 건 지금 당장 풍헌님의 코드를 넘겨 받았을 때 내가 최대한 빨리 이해하고 이걸 짤 수 있는 가? 를 고려해서 리뷰를 드렸습니다. 결국 개발을 업으로 삼았을 때 제일 중요한 건 다른 사람도 내 코드를 사용할 수 있다는 걸 고려해야 한다 라는 거라고 생각해요. 그래서 세부 구현은 entity 내부에서 처리하고 service layer에서 기능 단위의 메소드만 적는게 중요합니다. 그렇게 되었을 때, 그 이후에나 유닛테스트, DB 성능 최적화, 멀티 스레드를 고려한 코드를 고민하면 풍헌님의 포트폴리오가 점점 강화될 거라 생각합니다. 그리고 그런 테스트나 최적화를 하면 할 수 록 하나의 명세서처럼 보이게 될 거에요. 풍헌님에게 개인적으로 드리고 싶은 하나의 조언은 내가 개발하는 기능을 1. A를 한다. 2. B를 한다. 처럼 적어보고 이걸 각각 메소드화 하는 연습인 거 같아요. 그래야 테스트 코드가 나오고 가시성이 높아지거든요. 그래도 많은 발전이 있었다고 생각합니다!
6주간의 리뷰가 얼마나 도움이 되었는지는 잘 모르겠지만 ㅎㅎ 그래도 다른 사람이 이 걸 보면 이해가 될까? 라는 고민을 하면서 만드시면 좋은 개발자가 되실거라 생각합니다. 굳이 개발자가 아니더라도 말이죠 ㅎㅎ 고생 많으셨습니다.