C++中不可预测的无限For循环

2022-04-20 00:00:00 string reference for-loop infinite-loop c++

我正在编写一个程序来返回该字符的第一次出现以及该字符在字符串中的频率。 函数中的For循环正在执行无限次,而If条件和块甚至没有执行一次。

有什么问题?

string::size_type find_ch(string &str,char ch,int& i_r)
{
    string::size_type first=0;
    for(auto i=str.size()-1;i>=0;i--)
    {
        cout<<"
Inside a for loop."<<endl;
        if(str[i]==ch)
        {
            cout<<"Inside if."<<endl;
            first=i+1;
            i_r++;
        }
    }
    return first;
}

解决方案

此循环:

for(auto i = str.size() - 1; i>=0; i--)

仅当i小于0时才会退出。但这不是unsigned int的有效值。该值将换行到最大值unsigned int,您将得到一个无限循环。

请注意,std::string上的.size()返回size_t,这基本上是unsigned int类型。

解决此问题的一种方法是将.size()的返回类型转换为int,如下所示:

for(auto i = static_cast<int>(str.size()) - 1; i>=0; i--)

请注意,在减去1之前进行强制转换很重要,否则在str为空时会得到错误的答案。

在C++20中,您可以通过调用std::ssize()自由函数完全避免此问题,该函数返回大小的签名版本。

相关文章