한 줄 요약
newsfeed 프로젝트 내가 맡은 부분 추가 도전까지 구현(120%), 튜터님 전체 코드 리뷰
newsfeed
오늘도 역시 깃허브 주소를 남기겠습니다.
오늘 구현한 부분: 페이징 적용, 사용자 정의 예외처리로 예외처리 변경하기, 리팩토링
추가로 도전할 부분: 연관 관계 부모가 삭제가 되면 자식도 함께 삭제되는 기능 공부하고 적용
깃허브 주소: https://github.com/jihwanprogramer/NewSfeed
코드 리뷰
튜터님이 말씀하신 개선사항
-
페이지 조회랑 전체 조회가 같은 기능이기 때문에 하나로 통합하는게 좋을 것 같다.
-
세터 사용 주의해야 할 것 같다.
-
엔티티에서 제약조건 사용하여 다른 개발자가 쉽게 보드의 제약조건을 볼 수 있도록 하는 게 좋을 것 같다.
-
코드와 구조의 통일감이 없다( 누구는 인터페이스를 써서 개발하고 누구는 클래스만을 써서 개발하고 메서드명도 제각각)
-
엔티티 생성 가독성, 엔티티 역할 부여 new로 생성하지말고 객체에서 직접 생성을 다루기 (정적 팩토리 매서드 형태)
-
도메인 중심으로 로직 흐르게 하기
-
엔티티가 가진 값은 엔티티가 다루기
-
커서 방식의 페이지 연구해보기
-
옵셔널 자체 함수 사용해보기
-
생성 수정 삭제시
@transactional붙여주기 -
연관관계가 있는 삭제시 @SQLdelete 같은 소프트 딜리트 어노테이션 활용, 연관관계가 끈끈하면 캐스케이드 활용, 오펀리무버도 고려해보면 좋음
느낀 점
팀 프로젝트는 확실히 개인 프로젝트할 때 보다 고려할 부분이 많다는 것을 느꼈고 나름대로 팀 프로젝트만의 느낌은 색다르기 때문에 좋은 경험이 되고 있는 것 같다. 거의 다 끝나가고 있는 와중에 리팩토링할 것이 생겨서 또 내일 열심히 달려야 될 것 같다.👍👍
