大家帮看看这种 Java 写 sql 的代码可取吗

@Override public DataListPage getDataListPageByFirstClass(DataListParams dataListParams, String firstClass, String branchStoreId) {

    String mainStoreId = BusinessUtils.getMainStoreId(branchStoreId);

    String listHql = CONSTRUCTOR
            + " FROM TCmsLesson cl, TCmsLessonStoreRel r"
            + " WHERE cl.firstClass = :firstClass"
            + " AND cl.isValid = 1"
            + " AND cl.id = r.lessonId"
            + " AND r.storeId in (:mainStoreId , :branchStoreId , '0000000')";

    String countHql = "SELECT COUNT(cl.id) FROM"
            + " TCmsLesson cl, TCmsLessonStoreRel r"
            + " WHERE cl.firstClass = :firstClass"
            + " AND cl.isValid = 1"
            + " AND cl.id = r.lessonId"
            + " AND r.storeId in (:mainStoreId , :branchStoreId , '0000000')";

    dataListParams.setDateParam("cl.createDate");

    return super.getDataListPageByHql(listHql, countHql, dataListParams, firstClass, mainStoreId, branchStoreId);
}

private List<T> listDataByHql(String hql, DataListParams dataListParams, Object... values) {
	if (StringUtils.isNotEmpty(dataListParams.getBeginTime())
			&& StringUtils.isNotEmpty(dataListParams.getEndTime())) {
		hql = hql + " AND " + dataListParams.getDateParam() + " BETWEEN '" + dataListParams.getBeginTime()
				+ "' AND '" + dataListParams.getEndTime() + "'";
	} else if (StringUtils.isNotEmpty(dataListParams.getBeginTime())) {
		hql = hql + " AND " + dataListParams.getDateParam() + " >= '" + dataListParams.getBeginTime() + "'";
	} else if (StringUtils.isNotEmpty(dataListParams.getEndTime())) {
		hql = hql + " AND " + dataListParams.getDateParam() + " <= '" + dataListParams.getEndTime() + "'";
	}

	if (StringUtils.isNotEmpty(dataListParams.getOrderBy())) {
		hql = hql + " ORDER BY " + dataListParams.getOrderBy();
	}

	if (dataListParams.isDesc()) {
		hql = hql + " DESC";
	}

	if (dataListParams.getPage() != null) {
		return this.listByPage(hql, dataListParams.getPage(), values);
	} else {
		return dataListParams.getSize() == 0
				? this.listByParamsAndPosition(hql, dataListParams.getStartPosition(), values)
				: this.listBySizeAndPosition(hql, dataListParams.getSize(), dataListParams.getStartPosition(),
						values);
	}
}

aguesuka

2020-10-27 17:45:43 +08:00

好不好不评价，给几个建议。String hql=前一句加上 // language=HQL (sql jpaql 同理)。参数使用冒号加参数名的格式，然后用 setparammap 或者 setpropsobject 设置参数的值。参数和返回值加上泛型。
java 的 orm 包括不限于 jpa mybatis springtemplate 都不咋地，在面相对象而且自省能力这么弱的语言里发明一种代替 sql 的 dsl 是不可能的。
我能想到的最好办法是整合项目设计的时候不考虑 sql，专门招聘一个实习生来写 dao 层。

Joker123456789

2020-11-12 11:15:00 +08:00

首先，你如果问的是把 sql 写在类里面这种方式可不可取，那我个人认为把 sql 写类里面，和写 xml 里没啥区别，现在都是 boot 打 jar 包，即使写在 xml 里也是无法在线上更改的。

无论写哪，只要牵扯到改，都要程序员改完提测然后重新打包发布，所以写类和写 xml 已经没有本质区别了。

所以，这种写法我个人认为完全没问题，不过肯定会有一些盲目遵守规范的人会说不行，这个就不做争论了。。

其次，如果你问的这你贴出来的这段代码有没有问题，那么其他人已经都告诉你了，你需要让代码分布的更合理一点，然后参数不要拼接，要用占位符。