嵌套循环为何被视为低效与不良设计？替代方案及数字转单词Java代码的可扩展性优化咨询

问题1：当前代码的设计问题与替代方案 #

首先要纠正一点：你的代码里并没有传统意义上的嵌套循环，但那种在for循环里通过i+1、i+2向前检查字符并手动修改i的跳步逻辑，确实属于不良设计，主要原因有这几点：

• 可读性极差：其他开发者读代码时，需要反复梳理i的跳步规则和多个条件的优先级，很容易混淆逻辑。
• 边界错误风险高：比如你写的i < n.length()-2，如果后续要支持更多重复次数（比如4次），很容易写错边界条件，导致字符串越界或者漏处理末尾字符。
• 可维护性差：如果要修改重复规则（比如调整输出格式），必须在循环里修改多个条件和跳步值，很容易引入Bug。

更优的替代方案：先统计连续重复长度，再批量处理 #

我们可以用单次线性遍历+连续次数统计的方式，先从当前位置开始，数出连续相同数字的出现次数，再根据次数生成对应的文本。这种写法逻辑清晰，边界处理更安全，也更容易维护。

示例修改后的代码：

public class DigitToWord {
    public static final Map<String, String> DIGIT_WORDS = Map.of(
        "0", "zero", "1", "one", "2", "two", "3", "three", "4", "four",
        "5", "five", "6", "six", "7", "seven", "8", "eight", "9", "nine"
    );

    public static String convertNumberToWords(String n) {
        if (n == null || n.isEmpty()) {
            return "";
        }
        StringBuilder result = new StringBuilder();
        int index = 0;
        int length = n.length();

        while (index < length) {
            char currentDigit = n.charAt(index);
            String digitWord = DIGIT_WORDS.get(String.valueOf(currentDigit));
            // 统计当前数字连续出现的次数
            int repeatCount = 1;
            while (index + repeatCount < length && n.charAt(index + repeatCount) == currentDigit) {
                repeatCount++;
            }

            // 根据次数处理输出（对齐你描述的规则：首次读原词，后续重复用术语标注）
            result.append(digitWord).append(" ");
            if (repeatCount > 1) {
                // 先按你原代码逻辑处理，后续问题2会优化这部分
                if (repeatCount == 2) {
                    result.append("double ").append(digitWord).append(" ");
                } else if (repeatCount == 3) {
                    result.append("triple ").append(digitWord).append(" ");
                }
            }

            // 跳过已经处理的连续数字
            index += repeatCount;
        }

        // 移除末尾多余的空格
        return result.toString().trim();
    }
}

问题2：实现代码的可扩展性（支持更多重复次数） #

你提到的“为每种重复次数写单独函数”确实低效，更专业的做法是把重复次数和对应的术语解耦，用配置化的方式管理，具体步骤如下：

1. 定义重复术语映射表 #

用一个Map来存储“连续重复总次数”对应的英文术语，后续要加新的次数，只需要在这个映射里新增条目即可：

// 按照你描述的规则：3次连续→输出triple，所以key是总次数，value是对应术语
private static final Map<Integer, String> REPEAT_TERMS = Map.of(
    2, "double",
    3, "triple",
    4, "quadruple",
    5, "quintuple",
    6, "sextuplet"
    // 后续加更多次数只需要继续新增键值对
);

2. 基于映射表动态处理重复逻辑 #

在统计完连续重复次数后，直接从映射表中获取对应的术语，不需要写多个if-else分支：

public static String convertNumberToWords(String n) {
    if (n == null || n.isEmpty()) {
        return "";
    }
    StringBuilder result = new StringBuilder();
    int index = 0;
    int length = n.length();

    while (index < length) {
        char currentDigit = n.charAt(index);
        String digitWord = DIGIT_WORDS.get(String.valueOf(currentDigit));
        int repeatCount = 1;
        while (index + repeatCount < length && n.charAt(index + repeatCount) == currentDigit) {
            repeatCount++;
        }

        // 处理输出逻辑
        result.append(digitWord).append(" ");
        if (repeatCount > 1) {
            String term = REPEAT_TERMS.get(repeatCount);
            if (term != null) {
                // 如果有对应的术语，就添加术语+原词
                result.append(term).append(" ").append(digitWord).append(" ");
            } else {
                // 处理没有定义术语的情况（比如7次及以上），可以默认输出多次原词或自定义逻辑
                for (int i = 1; i < repeatCount; i++) {
                    result.append(digitWord).append(" ");
                }
            }
        }

        index += repeatCount;
    }

    return result.toString().trim();
}

额外的扩展性优化 #

• 如果后续需要支持更复杂的规则（比如不同数字有不同的重复处理逻辑），可以把DIGIT_WORDS和REPEAT_TERMS做成外部配置文件（比如JSON、Properties），不需要修改代码就能更新规则。
• 把重复次数的统计逻辑封装成单独的工具方法，比如getRepeatCount(String str, int startIndex)，让主逻辑更简洁。

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