• 카테고리

    질문 & 답변
  • 세부 분야

    백엔드

  • 해결 여부

    미해결

상품 수정이 되지 않는데 이유를 모르겠습니다.

24.02.26 17:27 작성 조회수 91

0

학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

1. 강의 내용과 관련된 질문을 남겨주세요.
2. 인프런의 질문 게시판과 자주 하는 질문(링크)을 먼저 확인해주세요.
(자주 하는 질문 링크: https://bit.ly/3fX6ygx)
3. 질문 잘하기 메뉴얼(링크)을 먼저 읽어주세요.
(질문 잘하기 메뉴얼 링크: https://bit.ly/2UfeqCG)

질문 시에는 위 내용은 삭제하고 다음 내용을 남겨주세요.
=========================================
[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예/아니오)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예/아니오)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예/아니오)

[질문 내용]
상품 수정페이지 에서 상품에 대한 값 변경 시 데이터베이스에 반영이 되지 않는데 이유를 모르겠습니다. 다음은 코드입니다.

ItemController.java

package jpabook.jpashop.Contorller;

import jpabook.jpashop.domain.item.Book;
import jpabook.jpashop.domain.item.Item;
import jpabook.jpashop.service.ItemService;
import lombok.Getter;
import lombok.RequiredArgsConstructor;
import org.springframework.stereotype.Controller;
import org.springframework.ui.Model;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.PostMapping;

import java.util.List;

@Controller
@RequiredArgsConstructor
public class ItemController {

    private final ItemService itemService;

    @GetMapping("/items/new")
    public String createForm(Model model) {
        model.addAttribute("form", new BookForm());
        return "items/createItemForm";
    }

    @PostMapping("/items/new")
    public String create(BookForm form) {
        Book book = new Book();
        //set안쓰고 Book생성자의 파라미터로 넣는게 더 좋은 설계임
        book.setName(form.getName());
        book.setPrice(form.getPrice());
        book.setStockQuantity(form.getStockQuantity());
        book.setAuthor(form.getAuthor());
        book.setIsbn(form.getIsbn());

        itemService.saveItem(book);
        return "redirect:/";
    }

    @GetMapping("/items")
    public String list(Model model) {
        List<Item> items = itemService.findItems();
        model.addAttribute("items", items);
        return "items/itemList";
    }

    @GetMapping("items/{itemId}/edit")
    public String updateItemForm(@PathVariable("itemId") Long itemId, Model model) {
        Book item = (Book) itemService.findById(itemId); // 캐스팅이 별로 좋진 않음

        BookForm form = new BookForm();
        form.setId(item.getId());
        form.setName(item.getName());
        form.setPrice(item.getPrice());
        form.setStockQuantity(item.getStockQuantity());
        form.setAuthor(item.getAuthor());
        form.setIsbn(item.getIsbn());

        model.addAttribute("form", form);
        return "items/updateItemForm";
    }

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

//        Book book = new Book(); 컨트롤러에서 어설프게 엔티티 생성하지 말기
//        book.setId(form.getId());
//        book.setName(form.getName());
//        book.setPrice(form.getPrice());
//        book.setStockQuantity(form.getStockQuantity());
//        book.setAuthor(form.getAuthor());
//        book.setIsbn(form.getIsbn());
//
//        itemService.saveItem(book);

        itemService.updateItem(itemId, form.getPrice(), form.getName(), form.getStockQuantity());
        return "redirect:/items";
    }
}

ItemService.java

package jpabook.jpashop.service;

import jpabook.jpashop.domain.item.Item;
import jpabook.jpashop.repository.ItemRepository;
import lombok.RequiredArgsConstructor;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import java.util.List;

@Service
@Transactional(readOnly = true)
@RequiredArgsConstructor
public class ItemService {

    private final ItemRepository itemRepository;

    @Transactional
    public void saveItem(Item item) {
        itemRepository.save(item);
    }

    public List<Item> findItems() {
        return itemRepository.findAll();
    }

    public Item findById(Long itemId) {
        return itemRepository.findOne(itemId);
    }

    public void updateItem(Long itemId, int price, String name, int stockQuantity) {
        Item findItem = itemRepository.findOne(itemId);
        findItem.setPrice(price);
        findItem.setName(name);
        findItem.setStockQuantity(stockQuantity);
    }
}

 

HTML파일은 모두 복사붙여넣기 하였습니다.

답변 2

·

답변을 작성해보세요.

0

David님의 프로필

David

2024.02.26

안녕하세요. 김형준님, 공식 서포터즈 David입니다.

서비스에서 트랜잭션이 읽기전용으로 설정되어 있습니다.

특정 메서드에만 쓰기가 가능한 트랜잭션을 설정해 주시거나 읽기전용 트랜잭션에 대한 설정을 변경하셔야 합니다.

감사합니다.

0

인프런 AI 인턴님의 프로필

인프런 AI 인턴

2024.02.26

안녕하세요, 인프런 AI 인턴입니다. 김형준님.

상품 수정 기능에서 데이터베이스에 변경사항이 반영되지 않는 문제가 있으셨군요. 이와 유사한 문제를 겪은 다른 수강생의 질문 및 답변들과 더불어, @Transactional영속성 컨텍스트에 대한 이해를 기반으로 한 강의 내용의 해석에 도움되는 정보를 제공해드릴 수 있습니다.

유사 질문 링크:

  1. updateItem질문입니다. - Controller에서 엔티티의 변경사항을 반영하지 않는 문제에 대한 질문입니다. 해당 답변에서는 스프링의 OSIV(Open Session In View) 기본 설정과 트랜잭션 외부에서의 더티 체킹 동작 여부에 대한 설명이 있습니다. 다만, 이 질문에서의 상황과 정확히 일치하는지 확인이 필요합니다.

  2. 정적 팩토리 메소드로 변경감지 - Setter를 사용하지 않고 엔티티의 데이터를 변경하는 방법에 대한 질문입니다. 서비스 계층에서 업데이트 로직을 명확히 구현하는 예시가 포함되어 있습니다.

위의 링크들은 상품 수정 관련 처리를 하면서 발생할 수 있는 다양한 시나리오들을 다루고 있으며, 특히 영속성 컨텍스트와 트랜잭션 범위 내에서의 엔티티 상태 관리에 초점을 두고 있습니다. 질문자님이 제공하신 코드를 바탕으로 상품 수정이 제대로 반영되지 않는 이유를 파악하는 데 도움이 될 수 있습니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.