方法中返回空字符串仅为形式需求？能否优化？附字符串代码疑问

先聊聊你的疑惑：你最后返回空字符串绝对不是形式需求，这其实是原代码的一个bug——当输入字符串长度小于3的时候，你的循环根本不会执行，直接返回空，但正确逻辑应该是返回原字符串才对。另外你提到的void写法用System.out.println确实是不良实践，因为方法的职责应该是处理字符串并返回结果，而不是直接输出，这样方法的复用性会很差，所以用String返回类型是完全正确的选择，只是你的实现逻辑可以优化。

先看看你的原代码存在的问题：

public static String abc(String str) { 
    int i = 0; 
    while(i < str.length()-2) { 
        if (str.substring(i, i+3).equals("abc")) { 
            return str.substring(0, i) + str.substring(i+3); 
        } else { 
            if (i == str.length()-3) { 
                return str; 
            } 
        } 
        i++; 
    } 
    return ""; // 这里是bug，比如输入"ab"会返回空，而不是原字符串
}

问题分析 #

1. 边界情况处理错误：当字符串长度小于3时，str.length()-2会小于0，循环条件不成立，直接返回空字符串，但此时字符串里不可能有"abc"，应该返回原字符串。
2. 逻辑冗余：else里判断i == str.length()-3完全没必要，循环结束后直接返回原字符串即可，不需要在循环内额外判断。

优化方案 #

方案1：简化手动遍历逻辑

调整循环条件，同时修正边界返回值：

public static String removeFirstAbc(String str) {
    // 先处理字符串长度不足3或null的情况，直接返回原内容
    if (str == null || str.length() < 3) {
        return str;
    }
    
    int i = 0;
    // 循环到倒数第三个字符即可，因为要取3个字符的子串
    while (i <= str.length() - 3) {
        if (str.substring(i, i+3).equals("abc")) {
            // 找到第一个"abc"，拼接前后部分返回
            return str.substring(0, i) + str.substring(i+3);
        }
        i++;
    }
    // 循环结束没找到"abc"，返回原字符串
    return str;
}

方案2：利用Java内置方法（更简洁）

Java的String类提供了replaceFirst方法，专门用来替换第一个匹配的子串，直接用这个方法可以一行搞定，而且内置方法已经处理了所有边界情况：

public static String removeFirstAbc(String str) {
    // 正则表达式里"abc"是普通字符串，直接匹配即可，同时处理null避免空指针
    return str != null ? str.replaceFirst("abc", "") : null;
}

为什么你的void写法不好？

如果用void类型并直接输出，这个方法就只能用来打印结果，无法将处理后的字符串用于其他逻辑（比如继续拼接、存储到变量等），违背了单一职责原则——方法应该专注于数据处理，而不是输出操作。所以用String返回类型是正确的设计，只是你的实现需要修正边界问题。

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