작성
·
44
0
@Controller
@RequestMapping("/validation/v2/items")
@RequiredArgsConstructor
@Slf4j
public class ValidationItemControllerV2 {
private final ItemRepository itemRepository;
private final ItemValidator itemValidator;
@GetMapping("/add")
public String addForm(Model model) {
model.addAttribute("item", new Item());
return "validation/v2/addForm";
}
@PostMapping("/add")
public String addItemV6(
@Validated
@ModelAttribute Item item,
BindingResult bindingResult,
RedirectAttributes redirectAttributes, Model model) {
//검증에 실패하면 다시 입력 폼으로 이동
if(bindingResult.hasErrors()) {
log.info("errors = {}", bindingResult);
return "validation/v2/addForm"; //TODO 이거는 왜 리다이렉트 안하지 ?
}
//에러에 안 걸려서, 이후는 성공 로직
Item savedItem = itemRepository.save(item);
redirectAttributes.addAttribute("itemId", savedItem.getId());
redirectAttributes.addAttribute("status", true);
return "redirect:/validation/v2/items/{itemId}";
}
}
위 코드에서 TODO 부분인데요
검증에 실패했을때 addForm 으로 다시 보내는데, 리다이렉트를 안하더라구요
그러면 브라우저에서 새로고침하면 POST 를 보내게 될텐데
상품 등록 폼으로 다시 보내는게 목적이라면 리다이렉트를 하는게 더 좋아 보이는데요
혹시 리다이렉트 안하는 이유가 있을까요 ?
답변 1
리다이렉트를 안하는 특별한 이유가 있지는 않습니다.
극단적인 예시로 아래와 같이 바인딩 실패 시 리다이렉트로 메인 페이지로 이동시켜도 코드 상으로는 아무 문제가 없습니다.
if(bindingResult.hasErrors()) {
log.info("errors = {}", bindingResult);
return "redirect:/";
}
하지만, 사용자 관점에서 등록 실패 시 마다 메인페이지로 이동하면 불편하겠죠?
사용자 편의성을 고려한 코드를 작성하신 것으로 이해하시면 될 것 같습니다.
리다이렉트를 하는 방법은 알겠습니다만...
강의 예제 코드에서 저 경우에 리다이렉트를 안하는 이유가 궁금합니다