循环中使用StringBuilder拼接字符串,是否存在更优实现方案?
现有实现的合理性分析
- 功能层面没问题:只要传入非空的字符串参数,能正确输出
"字符串1","字符串2"这类预期格式的结果,核心逻辑通顺。 - 存在明显隐患:如果调用时不传任何参数(比如
concatString()),newValue.deleteCharAt(newValue.length()-1)会直接抛出IndexOutOfBoundsException——因为此时StringBuilder是空的,没有可删除的字符。
更优的实现方式
1. 使用StringJoiner(Java 7及以上)
这是Java专门为分隔字符串场景设计的工具类,自带首尾分隔符处理逻辑,代码简洁且安全:
public void concatString(String... values) { // 分隔符用"\",\"",前缀后缀都是"\",刚好拼接出目标格式 StringJoiner joiner = new StringJoiner("\",\"", "\"", "\""); for (String str : values) { joiner.add(str); } System.out.println(joiner.toString()); }
空参数调用时会输出空字符串,不会报错,完全适配需求。
2. 使用Stream API(Java 8及以上)
借助Stream的映射和收集操作,代码更优雅:
import java.util.Arrays; import java.util.stream.Collectors; public void concatString(String... values) { String result = Arrays.stream(values) .map(str -> "\"" + str + "\"") .collect(Collectors.joining(",")); System.out.println(result); }
先给每个字符串套上双引号,再用逗号连接,空输入时输出空字符串,逻辑清晰且代码量少。
3. 优化原有StringBuilder实现(兼容低版本Java)
如果要保留StringBuilder的写法,补上空参数判断,同时避免末尾删字符的操作:
public void concatString(String... values) { if (values == null || values.length == 0) { System.out.println(""); // 可根据需求自定义空输入的输出内容 return; } StringBuilder newValue = new StringBuilder(); for (int i = 0; i < values.length; i++) { newValue.append("\"").append(values[i]).append("\""); // 仅当不是最后一个元素时添加逗号 if (i != values.length - 1) { newValue.append(","); } } System.out.println(newValue); }
修改后空输入不会报错,也无需额外删除字符,逻辑更严谨。
内容的提问来源于stack exchange,提问作者user18542516
相关产品推荐
相关产品推荐

