일정 관리 앱 만들기 과제 피드백

[잘한 점]

controller, dto, entity 패키지를 만들어서 각 계층별로 패키지 관리를 잘 하셨습니다!
전체 일정 조회할 때 작성자명과 수정일을 파라미터로 받을 때 해당 부분은 선택적으로 받기 때문에 required=false로 설정을 잘 해주셨습니다!
필요한 Lombok 어노테이션만 사용하시고 entity -> dto로 변환할 때 setter가 아니라 생성자로 잘 하신 것 같습니다. setter를 남용하면 값을 변경한 의도를 파악하기 힘들거나 객체 일관성을 유지하기 힘들기 때문에 지금처럼 지양하시기 바랍니다.

[개선되면 좋을 부분]

제가 직접 서버를 구동시켜서 테스트 해보니 아래 "전체 일정 조회"코드에서 오류가 있습니다. 예를 들어, 일정 중에 작성자 명이 "홍길동", "홍길동2", "홍길동3"인 일정들이 있을 때 파라미터로 ?username="홍길동"을 넘겨주면 for문을 순회하면서 "홍길동2"일 때 for문 내부 2번째 if문((2): 작성자명이 있는 경우)의 내부가 실행됩니다. if문 내부에서 match를 false로 셋팅해주고 아래의 else문에서 false이기 때문에 리턴이 되면서 NOT FOUND로 응답을 하게 됩니다. 일정 데이터 중에 "홍길동"이 있음에도 NOT FOUND가 뜨는 오류가 있습니다. 추가적으로, for문 순회시 "홍길동2"가 "홍길동"보다 먼저 순회가 된다면 빈 배열이 리턴이되고 NOT FOUND가 뜨는 오류 현상도 발견했습니다. 해당 부분의 로직은 수정이 필요할 것 같습니다. https://github.com/beunyeong/Spring_Schedule/blob/4e0dba5bbc478c9207e3b7d26390b17767336d76/src/main/java/com/example/schedule/controller/ScheduleController.java#L44-L78
보통 REST API를 설계할 때 url의 경우 소문자를 이용합니다. 그래서 아래 코드처럼 /Schdules가 아닌 /schedules로 수정하시면 더욱 Restful한 API 설계가 될 것 같습니다. REST API 관련 블로그 글을 아래 첨부할테니 참고하시기 바랍니다. https://github.com/beunyeong/Spring_Schedule/blob/4e0dba5bbc478c9207e3b7d26390b17767336d76/src/main/java/com/example/schedule/controller/ScheduleController.java#L15-L18
- 참고자료) https://medium.com/tech-pentasecurity/restful-api-%EB%84%A4%EC%9D%B4%EB%B0%8D-7c81bdb9da63
이번 과제에서는 lv0에서 작성한 ERD와 SQL을 토대로 직접 DB에서 데이터를 조회하거나 저장을 해야하는데 이 부분은 아직 적용을 못하신 것 같아 아쉽습니다. 다음 과제에서는 꼭 DB연결까지 진행해보시기 바랍니다. DB연결까지 진행을 하게되면 지금처럼 controller만 있는 것이 아니라 repository단, service단이 추가가 될 것입니다!

[고민했거나, 어려웠던 부분]

이 부분은 많이 고민하신게 느껴지네요. 작성 해주신 것처럼 @JsonFormat(pattern = "yyyy-MM-dd")을 사용하셔서 LocalDateTime의 타입을 가지지만 반환할 때는 지정해준 패턴으로 반환하도록 잘 하셨습니다! 물론 @JsonFormat(pattern = "yyyy-MM-dd")을 Schedule 클래스에서 바로 적용해서 할 수도 있겠지만 Schedule 클래스의 경우 entity클래스이기 때문에 지금 처럼 responseDto 클래스를 만들어서 잘 하셨습니다. 그리고 @JsonFormat(pattern = "yyyy-MM-dd")을 사용하지 않고 진행하고 싶으시다면, updateDate 변수 타입을 LocalDate로 하셔서 생성자 부분에서 LocalDateTime을 toLocalDate라는 메서드를 통해서 간편하게 LocalDate 타입으로 변경이 가능하니 참고하시기 바랍니다. https://github.com/beunyeong/Spring_Schedule/blob/4e0dba5bbc478c9207e3b7d26390b17767336d76/src/main/java/com/example/schedule/dto/FindAllResponseDto.java#L10-L19
- 예시 코드
```
@Getter
public class FindAllResponseDto {

private Long id; // 고유 식별자 ID(필수)
private String title; //일정 제목
private String content; //할일 (필수)
private String username; //작성자명(필수)
@JsonFormat(pattern = "yyyy-MM-dd HH:mm:ss")
private LocalDateTime createdDate; // 작성일
private LocalDate updatedDate;// 수정일

public FindAllResponseDto(Schedule schedule) {
  this.id = schedule.getId();
  this.title = schedule.getTitle();
  this.content = schedule.getContent();
  this.username = schedule.getUsername();
  this.createdDate = schedule.getCreatedDate();  // 작성일
  this.updatedDate = schedule.getUpdatedDate().toLocalDate();  //수정일
}
}
```

beunyeong / Spring_Schedule

일정 관리 앱 만들기 과제 피드백 #1

[잘한 점]

[개선되면 좋을 부분]

[고민했거나, 어려웠던 부분]