전원희씨 rest api 피드백

개발자 : 전원희

 

프로젝트 : ReDeal - 중고 거래 플랫폼

 

 

1. 로그아웃시 알림 창이 다수 발생.

 -> 시연 도중 발생한거라 캡쳐는 없지만 다수 발생하는것을 확인 했습니다.

 

2. 엔티티 사용시  createdDate과 updatedDate는 공통 엔티티로 뺄수 있을것 같습니다.

 

 

 

 

3. status 사용시 string보다 enum을 사용하는게 좋을 것 같습니다.

 

4. 추가적으로 status는 "판매중" , "완료" 같은 한글보다 영어나 코드로 저장하는것이 더 나을것 같습니다.

 

 

5. 삭제시 이미 한번 조회를 해온 후라 굳이 ID기반으로 삭제할 필요는 없을 것 같습니다.

 

 

 

의문점

1.로그인 / 회원가입시 미리 데이터가 입력되어있는 부분들이 있었는데 왜 그런건지? 

 

 

2. 현재 git 상 엔티티에 OneToMany나 ManyToOne등의 연관관계 매핑이 안보이는데 의도된 사안인지?