조서영 4주차 기능 구현

[seoyoung-dev]

제 작업 환경에서 빌드했을 때 에러가 발생하진 않지만, 혹시나 지난 번 같은 일이 발생할 수 있기 때문에 weekly에 merge하시고 코드 run 한 번 해 주시면 감사하겠습니다 🙇‍♂️

기능 구현 항목

• 태스크 생성 기능
• 유효성 검사

코드 수정 사항

• TaskController에서 createTask의 반환값 추가(SingleResult로 삭제한 id같은걸 반환)(피드백 사항)
• createTask의 이름 변경
• @Valid추가
• 유효성 검사 조건 설정
• Long project_id, Long task_id 보다 카멜 케이스를 사용하는 것이 좋습니다.(피드백 사항)
• TaskService의 생성자에 @Autowired적용
• deleteTask삭제시 존재 여부 및 예외처리 코드 필요(객체 조회하면서 존재 검증 후 taskRepository.delete(entity); 로 삭제)

이슈 사항

• isDelete 변수 네이밍이 문법적으로 맞지 않은 것 같아(be동사+일반동사 불가능) isDeleted로 수정하였습니다.
• 코드리뷰 피드백에서 isDeleted는 boolean을 사용하는 것이 적절할 것 같다고 하셨는데 다른 분들도 isDeleted를 Integer 형식으로 정의하신 것 같아서 이부분에 대한 통일이 필요할 것으로 보입니다.
• Guest 기능이 아직 개발되지 않은 것 같아 mock data를 만들어 넣어보았습니다. Guest 엔티티에 join_number라는 항목이 있는데, 어떤 유형의 데이터를 저장하는 건지 궁금합니다. Guest 기능은 순호님이 개발하실 예정인지도 궁금합니다!

GuestEntity guestEntity = new GuestEntity("test@gmail.com", "1", 0, projectEntity);

• 현재 코드에서 User 엔티티와 Member 엔티티가 공존하는 것 같은데, 어떤 차이가 있는지 궁금합니다.
• 동일한 projectId를 가지는 태스크를 조회하는 기능을 추가해야 할 것 같은데, 코드 작성에 어려움이 있습니다. Controller 코드는 작성하였는데, 이에 대한 Service 코드와 Repository 코드를 어떻게 작성하는 게 좋을지 모르겠습니다. 아래의 코드는 제가 임의로 작성한 것인데, 이게 올바른 방법인지 잘 모르겠습니다. 같이 얘기 나눠보면 좋을 것 같습니다.

    // TaskController.java
    @GetMapping("/{projectId}/task")
    public PageResult<TaskEntity> getTaskList(@PathVariable Long projectId) {
        return PageMapper.toPageResult(taskService.getTaskList(projectId));
    }

    // TaskRepository.java 
    @Repository
    public interface TaskRepository extends JpaRepository<TaskEntity, Long> {

        Page<TaskEntity> findByProjectEntityId(Long projectId, Pageable pageable);
}

미완성 항목

• swagger tag 설정(Task CRUD 기능 개발 후 작성 예정)

[Kdonghs]

kakao-tech-campus-2nd-step3:seoyoung에 반영