PUT 과제 코드 리뷰 부탁드립니다.

안녕하세요, 과제를 마쳤는데 더 효율이 좋은 코드가 있을 것 같아서 리뷰 좀 부탁드립니다.

name과 joinDate를 바꿔봤습니다.

public User updateUser(int id, User user) {
    User updateUser = findOne(id);

    if (findOne(id) != null) {
        updateUser.setName(user.getName());
        updateUser.setJoinDate(new Date());
        return updateUser;
    }
    return null;
}

• UserDaoService.java

@PutMapping("/users/{id}")
private User updateUser(@PathVariable int id, @RequestBody User user) {
    User updateUser = service.updateUser(id, user);

    if (updateUser == null) {
        throw new UserNotFoundException(String.format("ID[%s] not found", id));
    }

    return updateUser;
}

• UserController.java

| 설명

1. Controller : Path와 RequestBody에 있는 id와 user 값을 받는다.
2. Controller :UserDaoService에 있는 updateUser()메소드로 id와 user를 파라미터로 넘겨준다
3. Service : findOne() 메소드를 통해 id에 대한 정보를 가져온다.
4. Service : findOne()에 맞는 정보가 없을 때 null이 뜨게 되면 null이 리턴되게 해주고, null이 아니라면 update 로직을 그대로 이어나간다.
5. Controller : 불러온 아이디에 setter를 통해 RequestBody에 있던 user 정보를 담아주고 리턴한다.
6. Controller :updateUser() 메소드에서 리턴해 온 정보가 null이면 NotFoundException을 뱉는다.

| 테스트

http://localhost:8088/users/1

• 테스트는 성공

| 의문

• RequestBody에 요청 정보를 다 적어주면 굳이 PathVariable를 사용해야 하나  싶습니다.
• 지금 이 코드를 사용했을 때 RequestBody에 들어간 id 정보가 필요 없다는 것도 찝찝합니다.

코드가 틀렸거나, 설명을 참고하여 제가 잘 못 알고 있는 점이 있다거나,  개선점이 있다면 알려주시면 감사하겠습니다.

다른 과제 글 찾아보니 JPA로 하라는 말씀도 하시는 것 같은데 JPA말고 수업 중 사용한 본연의 코드 정도만 활용해서 해보고 싶습니다.

감사합니다.!