基于Eclipse的Java木材尺寸超差判断代码优化咨询

你的代码功能正常，但确实存在冗余问题——硬编码所有布尔组合会导致扩展性差（比如后续新增维度时修改成本极高）。可以通过收集不合格维度、动态生成报告的方式简化代码，同时让逻辑更清晰、可维护。

优化思路 #

• 把公差值定义为常量，避免硬写死在字符串里，方便后续统一修改。
• 用列表收集所有超差的维度名称，不用枚举所有布尔组合。
• 根据不合格维度的数量，动态拼接对应的提示文本，彻底消除冗余的else if分支。

优化后的代码示例 #

// 定义公差常量，统一管理
private static final double TOLERANCE = 0.03125;

public String generateLumberReport(boolean thicknessIsGood, boolean widthIsGood, boolean lengthIsGood) {
    List<String> badDimensions = new ArrayList<>();
    
    // 收集所有超差的维度
    if (!thicknessIsGood) {
        badDimensions.add("Thickness");
    }
    if (!widthIsGood) {
        badDimensions.add("Width");
    }
    if (!lengthIsGood) {
        badDimensions.add("Length");
    }
    
    // 根据超差数量生成对应报告
    if (badDimensions.isEmpty()) {
        return String.format("Lumber dimensions meet tolerance standards of %.5f inches", TOLERANCE);
    } else {
        StringBuilder reportBuilder = new StringBuilder("Lumber dimensions are not within tolerance. ");
        int badCount = badDimensions.size();
        if (badCount == 3) {
            reportBuilder.append("All dimensions exceed tolerance of ").append(TOLERANCE).append(" inches.");
        } else if (badCount == 1) {
            reportBuilder.append(badDimensions.get(0)).append(" exceeds tolerance of ").append(TOLERANCE).append(" inches.");
        } else {
            // 精准列出两个超差的维度，替代原代码的模糊描述
            reportBuilder.append(String.join(" and ", badDimensions))
                        .append(" exceed tolerance of ").append(TOLERANCE).append(" inches.");
        }
        return reportBuilder.toString();
    }
}

优化点说明 #

• 消除冗余分支：不再需要枚举8种布尔组合，通过主动收集问题维度简化逻辑，可读性大幅提升。
• 扩展性强：后续新增维度（比如高度）时，只需添加一行if (!heightIsGood)的判断，无需新增大量分支。
• 报告更精准：原代码的else分支只能笼统说明“两个维度超差”，优化后可以明确指出具体是哪两个维度，提升报告实用性。
• 常量复用：公差值统一定义为常量，修改时只需改动一处，避免遗漏字符串中的硬编码值。

进阶优化建议 #

如果thicknessIsGood这类布尔值是通过实际值与预期值计算得出的，还可以把“判断是否超差”的逻辑封装成工具方法，进一步拆分职责：

private static boolean isWithinTolerance(double actual, double expected) {
    return Math.abs(actual - expected) <= TOLERANCE;
}

调用时直接用isWithinTolerance(actualThickness, expectedThickness)来获取布尔值，让代码职责更清晰。

内容的提问来源于stack exchange，提问作者fourth