简单的C函数 – 这段代码“好”吗？

以下代码由为我的小组工作的顾问制作.我不是C开发人员(虽然使用多种语言工作),但想对以下代码提出一些独立意见.这是在Visual Studio C 6.0中.我有一种直觉反应(显然不是一个很好的反应),但我想要那些经验丰富(甚至不是那么没有经历过)C开发者的“直觉反应”.提前致谢！

// 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);
}

解决方法

哼.我认为

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

也会这样做.

我不喜欢代码,我倾向于从while循环中断,而不是有一个不必要的bool’continin’标志.当他可以使用while(occurenceInd！= 0)作为他的循环控制变量而不是布尔值时,这会加倍.

增加计数器还依赖于“2”,这似乎不能立即理解(不是一瞥),最后(最重要的)他似乎没有做评论.