hyeonjinchoi/schedule-management

일정 관리 앱 서버

1. Use Case Diagram

2. API 명세서

3. ERD

schedule-management's Issues

[ 프로젝트 피드백 ]

안녕하세요~ :)

각 단계별 기능을 동작할 수 있도록 잘 구현해주셨습니다.
DTO 도 잘 활용해주셨고 예외도 잘 던지도록 처리해주신것 칭찬드립니다.
각 기능별로 개선되면 좋을 점들을 피드백 드리겠습니다.

[ 관심사 분리 관련 ]
현재 Controller 에 모든 코드가 들어있다보니 코드 길이가 매우 길어졌습니다.
관심사 분리를 통해 코드를 분산하고 각 레이어별로 최적화가 가능하도록 수정해주세요.

[ 기능 관련 ]
스케쥴 삭제시 password 를 요청 파라미터로 받고 있는데요.
요청 파라미터는 uri 에 노출되는 값이기 때문에 보안에 취약합니다.
password 는 보안 데이터이니까 requestBody 에 담아서 처리해주세요.

[ Table ID 관련 ]
Schedule 테이블 객체에 새로운 레코드를 추가할때 id 값을 직접 size 조회해서 +1 한 후 넣어주고 있는데요.
이렇게 하게되면 클래스 변수인 schedules 변수가 너무 커질 수 있고 어플리케이션 종료시 삭제되게 됩니다.
테이블의 ID 는 Schedule 필드위에 @id 를 사용해서 아래와 같이 구성해주시면 됩니다.
@id
@GeneratedValue(strategy = GenerationType.IDENTITY)
private Long id;

고생 많으셨습니다! 포기하지말고 한단계씩 파이팅 하세요.