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

时间:2020-08-17 22:09:29

标签: c++ string while-loop

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

#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;
}

1 个答案:

答案 0 :(得分:1)

此声明

input.substr(i);

不会更改对象input本身。

因此,如果某个索引i input.at(i)不等于input.at(i+1),您将有一个无限循环,否则您可能会遇到异常,因为{{1} }可以等于i + 1

根据成员函数input.size()

的描述

抛出:如果pos> = size(),则超出范围。

该程序可以以不同的方式实现。例如以下方式

at

程序输出为

#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;
}