Question

我有一个Spring MVC应用程序，其中存储库（@Repository）直接注入某些控制器（@Controller），因此有时会绕过服务层。

例如：我的存储库中有几个findByXXX方法，我没有找到使用Spring Roo在我的服务中自动生成自定义方法的包装方法的方法。

因此我陷入两难境地：绕过服务层（坏）或花费大量时间（坏）在我的自定义存储库方法的Service层中创建简单的包装器方法。

有没有人能解决这个难题？

编辑1 ：这是@Nabil要求的控制器之一：

@Controller
@RequestMapping("/signup")
public class SignupController {

    @Autowired
    private SignupService signupService;

    @Autowired
    private SigninService signinService;

    @Autowired
    private MemberRepository memberRepository;

    @Autowired
    private PreferenceService preferenceService;

    @RequestMapping(method = RequestMethod.GET, produces = "text/html")
    public String signupForm(@ModelAttribute SignupInfo signupInfo, Model model) {
        populateForm(model, signupInfo);
        return "signup";
    }

    @RequestMapping(method = RequestMethod.POST, produces = "text/html")
    public String signup(@ModelAttribute @Validated({ Validation.Signup.class }) SignupInfo signupInfo, BindingResult bindingResult, Model model) {
        if (!preferenceService.isEmailAvailable(signupInfo.getMember().getEmail())) {
            bindingResult.rejectValue("member.email", "controller.signup.email_already_used");
        }

        if (bindingResult.hasErrors()) {
            populateForm(model, signupInfo);
            return "signup";
        }

        signupService.signupMember(signupInfo.getMember(), signupInfo.getAddressReference());
        signinService.signin(memberRepository.findByEmail(signupInfo.getMember().getEmail()));
        return "redirect:preference/email";
    }

    private void populateForm(Model model, SignupInfo signupInfo) {
        model.addAttribute("signupInfo", signupInfo);
        model.addAttribute("roles", Arrays.asList(Role.ROLE_BASIC_CHILDMINDER, Role.ROLE_BASIC_FAMILY));
    }
}

Answer 1

解决方案是将整个业务逻辑纳入服务。控制器的职责应该是将HTTP请求转换为单一服务调用，而不是将结果或异常转换为HTTP响应。该服务应调用执行业务操作所需的所有存储库和其他服务。

至于几个findByXXX，代表团并不是坏事。但是，我会在存储库层中使用通用单一方法findAll(Predicate p)。即比如Spring Data JPA docs建议，甚至更好地使用QueryDSL。然后服务层将构造谓词，而不是仅委托给下层。

从Spring MVC控制器调用存储库的问题

1 个答案: