1. 问题背景

在实际Java开发中，我们经常需要构建包含动态内容的字符串。一个常见的场景是在错误处理中，需要将固定错误信息与动态错误详情结合起来。然而，如果不注意运算符优先级，很容易写出看似正确但实际上有逻辑错误的代码。

2. 问题案例

原始问题代码（简化示例）：

// 错误示例
return "错误前缀: " + errorDetail == null ? "默认值" : errorDetail;

开发者的意图是：如果errorDetail为null，返回带默认值的完整字符串；否则返回带有实际错误详情的完整字符串。

但实际上，当errorDetail不为null时，返回的结果只有错误详情，缺失了错误前缀。

3. 根本原因分析

3.1. Java运算符优先级

问题根源在于对Java运算符优先级理解不足。以下是相关运算符的优先级顺序（从高到低）：

1. () - 括号
2. + - 字符串拼接
3. ==, != - 相等性比较
4. ? : - 三元条件运算符

3.2. 代码实际执行逻辑

原始代码被编译器解释为：

// 实际执行顺序
return ("错误前缀: " + errorDetail) == null ? "默认值" : errorDetail;

执行过程：

1. 首先执行字符串拼接："错误前缀: " + errorDetail
2. 然后判断拼接结果是否等于null
3. 由于拼接后的字符串永远不可能为null（至少包含"错误前缀: "），所以条件始终为false
4. 最终执行三元表达式的false分支，返回errorDetail本身

4. 解决方案对比

4.1. 方案一：使用括号（最直接）

// 正确写法1
return "错误前缀: " + (errorDetail == null ? "默认值" : errorDetail);

优点：代码简洁，一目了然 缺点：重复引用了errorDetail变量

4.2. 方案二：使用临时变量（最清晰）

// 正确写法2
String detail = errorDetail == null ? "默认值" : errorDetail;
return "错误前缀: " + detail;

优点：

• 逻辑分离，易于理解和维护
• 避免重复调用可能的方法
• 便于调试

4.3. 方案三：使用Objects.toString（最安全）

import java.util.Objects;

// 正确写法3
return "错误前缀: " + Objects.toString(errorDetail, "默认值");

优点：

• 标准库方法，语义清晰
• 自带null安全处理

4.4. 方案四：使用String.format（需要格式化时）

// 正确写法4
String detail = errorDetail == null ? "默认值" : errorDetail;
return String.format("错误前缀: %s", detail);

优点：适合复杂的字符串格式化

5. 最佳实践建议

1. 明确优先级：当表达式包含多个运算符时，主动使用括号明确意图
2. 分步处理：复杂的字符串构建应该分步进行，使用临时变量
3. 利用工具方法：善用Objects.toString()等工具方法处理null值
4. 编写单元测试：为字符串构建逻辑编写测试，覆盖null和非null情况

// 最佳实践示例
public String buildErrorMessage(String errorDetail) {
    // 1. 处理可能的null值
    String safeDetail = Objects.toString(errorDetail, "未知错误");
    
    // 2. 构建完整消息
    return String.format("操作失败: %s (时间: %s)", 
        safeDetail, 
        LocalDateTime.now().format(DateTimeFormatter.ISO_LOCAL_TIME));
}

注意

记住：清晰的代码胜过聪明的代码。宁可多写几个括号或临时变量，也不要依赖对运算符优先级的精确记忆。