c++ - 简单的 C++ 函数——这段代码是 "good"吗？

Question

以下代码是由为我的团队工作的顾问编写的。我不是 C++ 开发人员(虽然使用过多种语言)，但希望对以下代码有一些独立的意见。这是在 Visual Studio C++ 6.0 中。我有一个直觉 react (显然不是一个好 react )，但我想要一些来自经验丰富(甚至不是那么缺乏经验)的 C++ 开发人员的“直觉 react ”。提前致谢!

// Example call
strColHeader = insert_escape(strColHeader, ',', '\\'); //Get rid of the commas and make it an escape character

...剪断...

CString insert_escape ( CString originalString, char charFind, char charInsert ) {
    bool continueLoop = true;   
    int currentInd   = 0;

    do {
        int occurenceInd = originalString.Find(charFind, currentInd);

        if(occurenceInd>0) {
            originalString.Insert(occurenceInd, charInsert);
            currentInd = occurenceInd + 2; 
        }
        else {
            continueLoop = false;   
        }
    } while(continueLoop);
    return(originalString);
}

Answer 1

嗯。我觉得

CString strColHeader;
strColHeader.Replace(",", "\\,") 

也可以。

我不喜欢这段代码，我倾向于从 while 循环中中断，而不是使用不必要的 bool 'continue' 标志。当他可以使用 while (occurenceInd != 0) 作为他的循环控制变量而不是 bool 值时，这就加倍了。

增加计数器也依赖于“+2”，这似乎不是立即可以理解的(不是快速浏览)，最后(也是最重要的)他似乎没有发表评论。