4

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

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

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

有没有人解决这个困境？

编辑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));
    }
}
4

1 回答 1

4

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

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

于 2013-09-10T09:15:57.920 回答