使用for循环操作StringBuilder的重复内容与格式问题咨询

嘿，我一眼就看出你代码里的问题啦，咱们一步步来搞定这两个头疼的问题：

问题根源分析 #

1. 格式错误：你在第一次循环时就把完整句子Your number X is not the highest one.塞进了StringBuilder，后面的循环只追加数字，结果所有数字都堆在了句子末尾，完全不符合预期格式。
2. 重复内容：如果你的实际代码里出现多次完整字符串重复，大概率是循环里的判断逻辑出了问题（比如每次循环都误触发了else分支）；另外你提到的数字/普通字符串重复，用常规List存储肯定会留下重复元素。

针对性解决方案 #

要同时搞定格式和去重，咱们换个思路：先收集所有需要的元素，最后再一次性拼接成正确格式的字符串，去重用Set就能轻松解决（LinkedHashSet还能保留元素的插入顺序）。

修改后的代码示例 #

import java.util.LinkedHashSet;
import java.util.Random;
import java.util.Set;

public static String process() {
    // 用LinkedHashSet：自动去重 + 保留插入顺序，不需要去重就换成ArrayList
    Set<Integer> numberSet = new LinkedHashSet<>();
    Random rm = new Random();
    
    // 先收集所有数字（自动去重）
    for (int i = 0; i < 10; i++) {
        numberSet.add(rm.nextInt(30));
    }
    
    // 把数字集合转成逗号分隔的字符串
    String numberList = String.join(", ", numberSet.stream()
                                                  .map(String::valueOf)
                                                  .toArray(String[]::new));
    
    // 最后拼接成符合预期的完整句子
    return String.format("Your number %s is not the highest one.", numberList);
}

针对普通字符串去重的场景 #

如果是普通字符串重复的情况，逻辑完全一样，把Set<Integer>换成Set<String>就行：

Set<String> contentSet = new LinkedHashSet<>();
// 循环添加字符串到contentSet...
String contentStr = String.join(", ", contentSet);
// 再拼到你的目标句子里

关键技巧总结 #

• 格式问题：永远不要在循环里拼接完整句子，先把所有需要展示的元素收集好，最后用String.join和String.format一次性构建正确格式，既简洁又不容易出错。
• 去重问题：用Set（推荐LinkedHashSet）代替List存储元素，自动帮你过滤重复内容，比手动用Map判断重复要高效得多。

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