使用 string.substr(post,len) 的无限循环

Question

你能弄清楚为什么它一直在控制台中无限循环吗？程序员应该列出用户插入字符串的每个字符，并在每个唯一字符旁边的括号中显示该字符在字符串中出现的次数......不知道为什么。

#include <iostream>
#include <string>
#include <algorithm>
using namespace std;

int main () {
  string input;
  cout << "input string: " , cin >> input;
  sort (input.begin() , input.end());
  while (!input.empty()) {
    int j{1}, i{0};
    while (input.at(i) == input.at(i+1)) {
      j++;
      i++;
    }
    cout << input.at(i) << " (" << j << "), ";
    input.substr(i);
  }
  return 0;
}

Answer 1

此声明

input.substr(i);

不会更改对象input 本身。

因此，如果某些索引 i input.at(i) 不等于 input.at(i+1)，您将有一个无限循环，或者您可能有一个超出范围的异常，因为 i + 1 可以等于 input.size() .

来自成员函数at的描述

如果 pos >= size()，则抛出：out_of_range。

该程序可以以不同的方式实施。比如下面这种方式

#include <iostream>
#include <string>
#include <algorithm>

int main() 
{
    std::cout << "input string: ";
    std::string input;
      
    std::cin >> input;
      
    std::sort( input.begin() , input.end() );
    
    for ( size_t i = 0; i < input.size(); )
    {
        size_t j = input.find_first_not_of( input[i], i );
        
        if ( j == std::string::npos ) j = i + 1;
        
        if ( i != 0 ) std::cout << ", ";
        std::cout << input[i] << " (" << j - i << ")";

        i = j;
    }
    
    std::cout << '\n';
    
    return 0;
}

程序输出是

input string: Hello
H (1), e (1), l (2), o (1) 

或者你可以使用标准容器std::map或std::unordered_map作为例子

#include <iostream>
#include <string>
#include <map>

int main() 
{
    std::cout << "input string: ";
    std::string input;
      
    std::cin >> input;
      
    std::map<char, size_t> m;
    
    for ( const auto &c : input )
    {
        ++m[c];
    }

    bool first = true;
    for ( const auto &p : m )
    {
        if ( !first  ) std::cout << ", ";
        std::cout << p.first << " (" << p.second << ")";
        first = false;
    }
    
    std::cout << '\n';
    
    return 0;
}

如果您希望输入字符串的字符按照它们在字符串中出现的顺序输出，那么程序可以看起来像

#include <iostream>
#include <string>
#include <map>

int main() 
{
    std::cout << "input string: ";
    std::string input;
      
    std::cin >> input;
      
    auto less = [&input]( const auto &c1, const auto &c2 )
    {
        return input.find( c1 ) < input.find( c2 );
    };
    
    std::map<char, size_t, decltype( less )> m( less );
    
    for ( const auto &c : input )
    {
        ++m[c];
    }
    
    bool first = true;
    
    for ( const auto &p : m  )
    {
        if ( !first  ) std::cout << ", ";
        std::cout << p.first << " (" << p.second << ")";
        first = false;
    }
    
    std::cout << '\n';
    
    return 0;
}

或者在不改变原始字符串且不使用额外容器的情况下，程序可以如下所示。

#include <iostream>
#include <string>

int main() 
{
    std::cout << "input string: ";
    std::string input;
      
    std::cin >> input;
      
    for ( size_t i = 0; i < input.size(); i++ )
    {
        size_t j = 0;
        
        while ( j != i && input[j] != input[i] ) j++;
        
        if ( j == i )
        {
            size_t count = 1;
            while ( ++j < input.size() )
            {
                if ( input[j] == input[i]  ) ++count;
            }
            if ( i != 0  ) std::cout << ", ";
            std::cout << input[i] << " (" << count << ")";
        }
    }
    
    std::cout << '\n';
    
    return 0;
}

程序输出可能看起来像

input string: elephant
e (2), l (1), p (1), h (1), a (1), n (1), t (1)