Spring Data JPA中的动态查询。重构

时间:2016-07-22 17:21:38

标签: java spring oop spring-mvc spring-data-jpa

使用Spring数据jpa和规范,我需要在spring mvc中实现过滤器/搜索功能。后端接收一个对象(ReportTemplateBean),该对象基本上是一个bean,其中一些字段代表前端的过滤器。

public class ReportTemplateBean implements Serializable {

private static final long serialVersionUID = -3915391620260021813L;

private Long id;

private String property;

private String city;

private String state;

private String zipCode;

private String propertyStatus;

private String realEstateRep;
//more code

我们有控制器

@RequestMapping(value = "/search", method = RequestMethod.GET)
@ResponseBody
public ReportBean search(@AuthenticationPrincipal ActiveUser activeUser,
    @ModelAttribute("templateForm") ReportTemplateBean template,
    Pageable pageable) throws GenericException {
LOGGER.info("Pulling report requested");

ReportBean report = reportService.searchProperties(template,
    pageable.getPageNumber(), pageable.getPageSize());

return report;
}

服务

@Override
@Transactional(readOnly = true, timeout = 20)
public ReportBean searchProperties(ReportTemplateBean template,
    Integer pageNumber, Integer pageSize) throws GenericException,
    TransactionTimedOutException {
LOGGER.info("searchProperties({})", template);

try {
    // pageNumber = (pageNumber == null ? 0 : pageNumber);
    // pageSize = (pageSize == null ? 10 : pageSize);
    ReportTemplate t = reportTemplateMapper.beanToEntity(template);
    List<PropertyBean> beans = new ArrayList<PropertyBean>();
    PropertySpecification spec = new PropertySpecification(t);
    Page<Property> properties = propertyRepository.findAll(spec,
        new PageRequest(pageNumber, pageSize, Sort.Direction.ASC,
            "name"));

然后它动态构建查询,但使用我不喜欢的长IF链。这是规范。

@SuppressWarnings("unchecked")
@Override
public Predicate toPredicate(Root<Property> root, CriteriaQuery<?> query,
    CriteriaBuilder cb) {
Path<String> propertyName = root.get(Property_.name);
Path<String> city = root.get(Property_.city);
Path<String> state = root.get(Property_.state);
Path<String> zipCode = root.get(Property_.zipCode);
final List<Predicate> orPredicates = new ArrayList<Predicate>();
final List<Predicate> andPredicates = new ArrayList<Predicate>();
if (template.getProperty() != null
    && template.getProperty().length() > 0) {
    andPredicates.add(cb.equal(propertyName, template.getProperty()));
}
if (template.getCity() != null && template.getCity().length() > 0) {
    andPredicates.add(cb.equal(city, template.getCity()));
}
if (template.getState() != null && template.getState().length() > 0) {
    andPredicates.add(cb.equal(state, template.getState()));
}
if (template.getZipCode() != null && template.getZipCode().length() > 0) {
    andPredicates.add(cb.equal(zipCode, template.getZipCode()));
}
if (template.getRealEstateRep() != null) {
    Join<Property, User> pu = null;
    if (query.getResultType().getName().equals("java.lang.Long")) {
    pu = (Join<Property, User>) root.fetch(Property_.createdBy);
    } else {
    pu = root.join(Property_.createdBy);
    }
    Path<Long> userId = pu.get(User_.id);
    andPredicates.add(cb.equal(userId, template.getRealEstateRep()));
}
if (template.getProjectType() != null
    && template.getProjectType().length() > 0) {
    Join<Property, Project> pp = null;
    if (query.getResultType().getName().equals("java.lang.Long")) {
    pp = root.join(Property_.projects);
    } else {
    pp = (Join<Property, Project>) root.fetch(Property_.projects);
    }
    Path<String> projectType = pp.get(Project_.projectName);
    andPredicates.add(cb.equal(projectType, template.getProjectType()));
}
//more IF's
return query.getRestriction();
}

你可以注意到规范似乎很难看,而且SONAR抱怨这种方法的循环复杂性(这很好)。

所以问题是,如何重构规范(IF&#39; s)以获得更多的OO代码? 提前致谢。 更新 - 我想在Spring Data JPA中使用/实现类似新功能(Query by Example)似乎如果你传递一个bean ExampleMatcher 类将忽略bean字段中的null值,这几乎就是我要查找的内容。忽略空值和空值。

1 个答案:

答案 0 :(得分:0)

我写了我的解决方案给你另一种选择,但正如我在评论中所说,我没有使用规范,我很想知道是否有人知道在春季jpa进行动态查询的另一种方式。

您可以在@Query界面中使用@Repository注释编写自己的查询。 在您的情况下(假设 ReportTemplateBean 是您的实体且其主键是 Long 类型),它将类似于:

@Repository
public interface ReportTemplateRepo extends JpaRepository<ReportTemplateBean, Long>{

    @Query("SELECT rb FROM ReportBeanTemplate rb JOIN ExampleTable et WHERE et.idTemplate = rb.id AND (:id is null OR :id = rb.id) AND (:city is null OR :city = rb.city) AND (:state is null OR :state = rb.state)")
    public List<ReportTemplateBean> findTemplates(@Param("id") Long id, @Param("city") String city, @Param("state") String state);
}

您可以添加所需的所有参数,在调用方法时将其传递为null。

方法调用示例(在您的服务类中):

@Autowire
ReportTemplateRepo templateRepo;

public void invocation(ReportTemplateBean template){
    List<ReportTemplateBean> templateRepo.findTemplates(
        template.getId(), template.getCity(), template.getState());
    }

这是我发现进行此类查询的唯一方法。