1. 从Optional到List的转换：空值判断的差异
当我们将代码中原本使用optional
错误示例：
public Listdelete(String department_ID) { // ... List existing = get(department_ID); if (existing.isPresent()) { // 错误：List类型没有isPresent()方法 // ... } // ... }
原因分析：isPresent()方法是java.util.Optional类特有的，用于指示该Optional实例是否包含一个非空的值。而java.util.List接口及其实现类（如LinkedList）并没有提供isPresent()方法。因此，尝试在List对象上调用此方法会导致编译错误：“The method isPresent() is undefined for the type List
正确实践： 要判断一个List是否包含元素，应该使用isEmpty()方法。如果列表不为空（即包含一个或多个元素），则!list.isEmpty()为真。
public Listdelete(String department_ID) { if ((department_ID == null) || (department_ID.isEmpty())) { return new LinkedList<>(); } List existing = get(department_ID); if (!existing.isEmpty()) { // 正确：检查List是否为空 String sql = "DELETE employee.*, department.* " + "FROM employee, department " + "WHERE employee.department_ID = :department_ID AND department.department_ID = :department_ID;"; // ... (省略部分代码) int rows = jdbcTemplate.update(sql, parameters); if (rows > 0) { return existing; } } return new LinkedList<>(); }
注意事项： 在某些情况下，你可能需要检查List对象本身是否为null。虽然良好的编程习惯通常会避免返回null的List，而是返回一个空列表，但如果存在返回null的可能性，可以使用Objects.nonNull(list)进行防御性检查。然而，如果你的方法始终保证返回一个非null的List（即使是空列表），则此检查是多余的。
2. 返回空列表的最佳实践
在从Optional.empty()转换为返回空列表时，使用new LinkedList()是有效的，因为它确实创建并返回了一个空的LinkedList实例。
// 原始代码： // return Optional.empty(); // 转换后： return new LinkedList<>(); // 有效，但有更好的选择
然而，在Java中，更推荐的做法是返回一个不可变的空列表。这有几个优点：
立即学习“Java免费学习笔记（深入）”；
- 安全性： 调用者无法修改返回的列表，避免了意外的副作用。
- 性能： 对于空列表，通常可以重用同一个不可变实例，减少内存开销。
- 清晰性： 明确表示该列表不应被修改。
推荐实践：使用 List.of() 返回不可变空列表
从Java 9开始，可以使用List.of()方法创建不可变列表。要返回一个空的不可变列表，只需调用List.of()而不传入任何参数。
public Listdelete(String department_ID) { if ((department_ID == null) || (department_ID.isEmpty())) { return List.of(); // 推荐：返回不可变的空列表 } List existing = get(department_ID); if (!existing.isEmpty()) { // ... (数据库操作) int rows = jdbcTemplate.update(sql, parameters); if (rows > 0) { return existing; } } return List.of(); // 推荐：返回不可变的空列表 }
如果调用方需要一个可修改的列表，他们可以根据返回的不可变列表轻松创建一个新的可修改列表，例如：new ArrayList(returnedList)。
3. 其他专业实践建议
除了上述的类型转换和空值处理，代码中还存在一些可以优化的点，以提升代码质量和可维护性。
3.1 命名规范
Java社区普遍遵循驼峰命名法（camelCase），并且避免在变量名和方法名中使用下划线。同时，通常不建议使用全大写来命名普通变量（全大写通常用于常量）。
建议： 将department_ID改为departmentId。
3.2 方法返回类型设计
在delete方法中返回List
- 操作是否成功： 可以返回boolean。
- 受影响的行数： 可以返回int（如jdbcTemplate.update()的返回值）。
- 无返回值： 如果删除操作是幂等的且不需要反馈具体删除内容，可以返回void。
返回一个List
建议： 根据delete方法的实际业务需求，重新评估其返回类型。例如，如果目的是告知调用者有多少条记录被删除，可以返回int。
// 示例：返回受影响的行数
public int delete(String departmentId) {
if ((departmentId == null) || (departmentId.isEmpty())) {
return 0; // 无效输入，返回0
}
// ... (查询现有部门，判断是否执行删除)
List existing = get(departmentId);
if (!existing.isEmpty()) {
String sql = "DELETE employee.*, department.* " + "FROM employee, department "
+ "WHERE employee.department_ID = :departmentId AND department.department_ID = :departmentId;";
MapSqlParameterSource parameters = new MapSqlParameterSource();
parameters.addValue("departmentId", departmentId);
return jdbcTemplate.update(sql, parameters); // 返回受影响的行数
}
return 0; // 未执行删除，返回0
} 3.3 使用文本块（Text Blocks）优化SQL字符串
对于多行的SQL语句，使用字符串拼接或转义字符会使代码难以阅读和维护。Java 15引入的文本块（Text Blocks）可以显著改善这种情况。
示例：
// 原始SQL字符串：
String sql = "DELETE employee.*, department.* " + "FROM employee, department "
+ "WHERE employee.department_ID = :department_ID AND department.department_ID = :department_ID;";
// 使用文本块优化：
String sql = """
DELETE employee.*, department.*
FROM employee, department
WHERE employee.department_ID = :departmentId AND department.department_ID = :departmentId;
""";文本块以三个双引号"""开始和结束，允许在其中直接书写多行文本，无需手动处理换行符或引号转义，极大地提高了SQL语句的可读性。
总结
从Optional到List的类型转换需要我们重新审视对空值的判断逻辑，将isPresent()替换为!isEmpty()。同时，在返回空列表时，优先选择List.of()来提供不可变且高效的空列表实例。此外，遵循Java的命名规范、合理设计方法返回类型以及利用文本块等现代Java特性，能够进一步提升代码的专业性和可维护性。通过采纳这些建议，开发者可以编写出更健壮、更易读、更符合最佳实践的Java代码。
