인프런 커뮤니티 질문&답변

@Controller

@PostMapping("/items/{itemId}/edit")
public String update(@PathVariable("itemId")Long itemId, @ModelAttribute("form") BookForm form){

    itemService.updateItem(itemId, form.getName(), form.getPrice(), form.getStockQuantity(), form.getAuthor(), form.getIsbn());

    return "redirect:/items";
}

 

 

@Service

/**
 * 다운캐스팅 없이 따로 findBook 메소드를 만들어줌 
 */
@Transactional
public void updateItem(Long id, String name, int price, int stockQuantity, String author, String isbn) {

    Book findBook = itemRepository.findBook(id);
    findBook.updateBook(name, price, stockQuantity, author, isbn);
    log.info("update실행");
}

 

 

Book엔티티

@Entity
@DiscriminatorValue("B")
@Getter
@NoArgsConstructor(access = AccessLevel.PRIVATE)
@Slf4j
public class Book extends Item {

    private String author;

    private String isbn;


    private Book(String name, int price, int stockQuantity, String author, String isbn) {
        super(name, price, stockQuantity);
        this.author = author;
        this.isbn = isbn;
    }


    /**
     * 정적 팩토리 메소드 생성 
     */
    public static Book createItem(String name, int price, int stockQuantity, String author, String isbn) {
        Book book = new Book(name, price, stockQuantity, author, isbn);

        return book;
    }


    /**
     * update 변경 로직 
     */
    public void updateBook(String name, int price, int stockQuantity, String author, String isbn) {
        addItem(name, price, stockQuantity);
        this.author = author;
        this.isbn = isbn;

    }

}

 

 

Item엔티티

/**
 * update 변경 로직
 */
protected void addItem(String name, int price, int stockQuantity) {
    this.name = name;
    this.price = price;
    this.stockQuantity = stockQuantity;
}

 

안녕하세요! 2회독하면서 영한센세가 말씀하신대로 setter는 최대한 쓰지않고 개발하고있습니다.

 

따로 DTO는 만들지않고 정적 팩토리 메소드를 활용해서 변경감지 로직을 짜봤는데 제대로 짠건지 잘 모르겠네요. 여기서 더 수정 하자면 어떤부분을 고치면 좋을까요??