if (StringUtils.isNotBlank(clientListForm.getOrderBy()) && StringUtils.isNotBlank(clientListForm.getIsAsc())) { StringBuffer orderSql = new StringBuffer(" order by "); List<String> orderByList = Arrays.asList(clientListForm.getOrderBy().split(",")); List<String> isAscList = Arrays.asList(clientListForm.getIsAsc().split(",")); if (orderByList.size() == isAscList.size()) { for (int i = 0; i < orderByList.size(); i++) { String orderBy = orderByList.get(i); String isAsc = isAscList.get(i); if (clientListOrderTypeMap.containsKey(orderBy)) { orderSql.append(clientListOrderTypeMap.get(orderBy)).append(" ") .append(clientListOrderWayMap.get(isAsc)).append(","); } } } orderSql.append(" b.client_id asc "); }
时间: 2024-02-24 14:54:38 浏览: 64
这段代码中有一些可以优化的地方:
1. 首先可以考虑使用 StringBuilder 替换 StringBuffer,因为 StringBuilder 的性能更好。
2. 在拼接 SQL 语句时可以使用 StringJoiner 类,简化代码逻辑,使代码更易读。
3. 在遍历 orderByList 和 isAscList 时可以使用 for-each 循环,使代码更简洁。
4. 可以将判断 orderByList 和 isAscList 长度相等的逻辑提前到 if 语句之外,这样可以避免在循环内部重复判断。
下面是优化后的代码:
```
if (StringUtils.isNotBlank(clientListForm.getOrderBy()) && StringUtils.isNotBlank(clientListForm.getIsAsc())) {
StringJoiner orderSql = new StringJoiner(", ", " order by ", " b.client_id asc");
List<String> orderByList = Arrays.asList(clientListForm.getOrderBy().split(","));
List<String> isAscList = Arrays.asList(clientListForm.getIsAsc().split(","));
if (orderByList.size() == isAscList.size()) {
for (String orderBy : orderByList) {
String isAsc = isAscList.get(orderByList.indexOf(orderBy));
if (clientListOrderTypeMap.containsKey(orderBy)) {
orderSql.add(clientListOrderTypeMap.get(orderBy) + " " + clientListOrderWayMap.get(isAsc));
}
}
}
// 使用 orderSql.toString() 获取最终生成的 SQL 语句
}
```
希望这些优化能对您有所帮助!如果有任何疑问,请随时向我提问。
阅读全文