DTO사용에 대한 질문

강의 23분18초 부분

updateItem을 DTO로 받는것에 대해 질문 2가지있습니다.

1. 아래와 같이 서비스계층의 DTO를 만들어서 파라미터로 사용하는부분은 이해했는데, 애초에 그냥 form을 넘겨주는것은 잘못된 설계인가요?

@PostMapping("/items/{itemId}/edit")
public String updateItem(@ModelAttribute("form") BookForm form) {
    UpdateItemDto updateItemDto = new UpdateItemDto(form);
    itemService.updateItem(updateItemDto);
    //itemService.updateItem(form);
    return "redirect:/items";
}

2. 강의에선 생략하는 부분인것같은데 만약 엔티티에 change() 함수를 만든다했을때 Item을 상속받는 Book, Album,Movie 각각에 change함수를 만들어주어야하나요..? (Item이라는것을 상속받는 3개의 클래스에대해 어떻게 처리해줘야할지 모르겠어요.)
   
   Item (부모클래스)에 필드들을 private -> protected로 바꾸고, Book의 change함수를 아래와 같이 만들면되나요? (이경우 Album, Movie 사용한다고 가정하면 똑같이 만들어줘야하는지?)

   public void change(UpdateItemDto dto) {
           this.name = dto.getName();
           this.price = dto.getPrice();
           this.stockQuantity = dto.getStockQuantity();
           this.author = dto.getAuthor();
           this.isbn = dto.getAuthor();
       }