如何将字符串中的所有"pi"替换为"3.14"？现有C++代码失效

需求说明：将string类型字符串中的所有"pi"替换为"3.14"，示例：输入"xpix"，输出应为"x3.14x"（操作对象非字符数组）。以下是原代码的问题点及修正方案：

原代码的核心问题 #

• 值传递导致修改无效：replacePi函数的参数是string str，属于值传递，函数内对str的所有修改都只作用于原字符串的副本，不会影响主函数中的原变量s。必须改为引用传递（string &str）才能直接修改原字符串。
• 递归逻辑完全无效：代码中调用replacePi(str.substr(1))传递的是子串副本，递归修改的是这个副本，和原字符串没有任何关联。正确的递归应该直接操作原字符串的指定区间，而非传递子串副本。
• 字符串越界操作：当尝试替换"pi"时，直接执行str[i+2] = str[i]，但此时str的长度并未扩展，访问超出当前长度的索引会触发未定义行为，程序可能崩溃或输出错误结果。
• 手动移动字符的逻辑错误：即使扩展了字符串长度，原代码的字符移动方式也存在索引计算错误，无法正确完成字符后移以容纳"3.14"。

修正后的代码示例 #

#include<iostream>
using namespace std;

void replacePi(string &str, int index = 0) {
    // 递归终止：剩余字符不足2个，无法构成"pi"
    if (index >= str.size() - 1) {
        return;
    }

    // 先递归处理当前位置之后的子串
    replacePi(str, index + 1);

    // 检查当前位置是否为"pi"
    if (str[index] == 'p' && str[index + 1] == 'i') {
        // 使用string内置replace方法，直接替换2个字符为"3.14"，自动处理长度扩展
        str.replace(index, 2, "3.14");
    }
}

int main() {
    string s;
    cin >> s;
    replacePi(s);
    cout << s << endl;
    return 0;
}

修正说明 #

1. 引用传递参数：使用string &str确保函数对字符串的修改直接作用于原变量。
2. 索引式递归：通过index参数跟踪当前处理位置，避免传递子串副本，递归操作直接作用于原字符串。
3. 利用内置方法简化操作：使用string::replace方法直接替换"pi"为"3.14"，该方法会自动处理字符串长度的扩展和字符移动，避免手动操作的错误。
4. 合理的递归顺序：先处理后续子串，再处理当前位置，确保替换当前"pi"时不会干扰已经处理完成的后续部分。

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