我一直在试图弄清楚使用 spring 提交表单的最佳实践是什么，以及实现这一目标的最小样板是什么。

我认为以下是最佳实践特征

• 启用验证并在验证失败时保留表单值
• 禁用重新提交表单F5(即使用重定向)
• 防止模型值出现在重定向之间的 URL 中 (model.clear())

到目前为止，我已经想出了这个。

@Controller
@RequestMapping("/")
public class MyModelController {

    @ModelAttribute("myModel")
    public MyModel myModel() {
        return new MyModel();
    }

    @GetMapping
    public String showPage() {
        return "thepage";
    }

    @PostMapping
    public String doAction(
            @Valid @ModelAttribute("myModel") MyModel myModel,
            BindingResult bindingResult,
            Map<String, Object> model,
            RedirectAttributes redirectAttrs) throws Exception {
        model.clear();
        if (bindingResult.hasErrors()) {
            redirectAttrs.addFlashAttribute("org.springframework.validation.BindingResult.myModel", bindingResult);
            redirectAttrs.addFlashAttribute("myModel", myModel);
        } else {
            // service logic
        }
        return "redirect:/thepage";
    }
}

有没有办法用 less 样板代码做到这一点，或者这是实现这一目标所需的最少代码量？

最佳答案

首先，我不会违反 Post/Redirect/Get (PRG)模式，这意味着如果表单发布成功，我只会重定向。

其次，我将完全摆脱 BindingResult 样式。这对于简单的情况很好，但是一旦您需要更复杂的通知从服务/域/业务逻辑到达用户，事情就会变得棘手。此外，您的服务的可重用性也不高。

我要做的是将绑定(bind)的 DTO 直接传递给服务，该服务将验证 DTO 并在出现错误/警告时发出通知。通过这种方式，您可以将业务逻辑验证与 JSR 303: Bean Validation 结合起来。 为此，您可以使用 Notification Pattern在服务中。

按照通知模式，您需要一个通用的通知包装器:

public class Notification<T> {
    private List<String> errors = new ArrayList<>();
    private T model; // model for which the notifications apply

    public Notification<T> pushError(String message) {
        this.errors.add(message);
        return this;
    }

    public boolean hasErrors() {
        return !this.errors.isEmpty();
    }

    public void clearErrors() {
        this.errors.clear();
    }

    public String getFirstError() {
        if (!hasErrors()) {
            return "";
        }
        return errors.get(0);
    }

    public List<String> getAllErrors() {
        return this.errors;
    }

    public T getModel() {
        return model;
    }

    public void setModel(T model) {
        this.model = model;
    }
}

您的服务将类似于:

public Notification<MyModel> addMyModel(MyModelDTO myModelDTO){
    Notification<MyModel> notification = new Notification();
    //if(JSR 303 bean validation errors) -> notification.pushError(...); return notification;
    //if(business logic violations) -> notification.pushError(...); return notification;
    return notification;
}

然后你的 Controller 会是这样的:

Notification<MyModel> addAction = service.addMyModel(myModelDTO);
if (addAction.hasErrors()) {
    model.addAttribute("myModel", addAction.getModel());
    model.addAttribute("notifications", addAction.getAllErrors());
    return "myModelView"; // no redirect if errors
} 
redirectAttrs.addFlashAttribute("success", "My Model was added successfully");
return "redirect:/thepage";

虽然 hasErrors() 检查仍然存在，但此解决方案更具可扩展性，因为您的服务可以随着新的业务规则通知继​​续发展。

另一种方法，我将保持非常简短，是从您的服务中抛出一个自定义 RuntimeException，这个自定义 RuntimeException 可以包含必要的消息/模型，并使用 @ControllerAdvice 捕获此通用异常，从异常中提取模型和消息并将它们放入模型中。这样，您的 Controller 只会将绑定(bind)的 DTO 转发给服务。

关于java - 使用最小样板提交 Spring 表单，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/49272904/