我的 C++ 代码中有一个棘手的错误；如何解决？

Question

我在下面为一个小程序编写了这段代码，该程序获取一个字符串（并将其放入数组中）和一个数字，然后它在数组中搜索以查找它是否有重复的字符（并计算重复的次数） ， 如果是这样的话;程序将比较特定字符重复的次数以及重复次数是否等于用户给出的次数；程序将用字符“A”替换它们。例如;如果我们给程序“BBCC”和数字 2，它应该给我们结果“AAAA”。问题是我的代码适用于某些示例，不适用于上面的示例。代码如下：

#include <iostream>
#include <stdio.h>

using namespace std;

int main()
{
char str[30];
int i, j, k, l, number, counter = 0;
char ch;

cout << "Enter a string: ";
gets (str);

cout << "\nEnter a number: ";
cin >> number;

for (i =0; str[i]; i++){
    ch = str[i];
    for (j = 0; str[j]; j++){
        if (str[j] == ch){
            counter++;
        }
    }
    if (counter == number){
        for (k = 0; str[k]; k++){
            if (str[k] == ch){
                str[k] = 'A';
            }
        }
    }
}

for (l = 0; str[l]; l++){
    cout << str[l];
}
getchar ();
return 0;
}

另一个错误是，如果我们给它字符串“ABC”和数字 2；结果将是“AAC”！ 我知道我的代码有点乱，因为我是初学者，所以请道歉并帮助我解决我的问题。 谢谢。

Answer 1

您需要在循环中重置 counter 变量。

对于ABC 示例：

1. 循环将首先计算As。有 1 个 A，所以 counter = 1。
2. 下一次迭代将计数Bs。有1个B，所以计数器增加到counter = 2！
3. 这是输入的数字，所以它会将所有Bs 替换为A => AAC

Answer 2

for (i = 0 ; str[ i ] ; i ++ )
{
    if( str[ i ] != 'A' ) // because replacing A with A is pointless
    {
        counter = 0 ;
        ch = str[ i ] ;

        for ( j = 0 ; str[ j ] ; j ++ )
        {
            if ( str[ j ] == ch )
            {
                    counter ++ ;
            }
        }

        if (counter == number)
        {
            for ( j = 0 ; str[ j ] ; j ++ )
            {
                if ( str[ j ] == ch )
                {
                    str[ j ] = 'A' ;
                }
            }
        }
    }
}

我试着把所有的cmets都放在一起修正+一些优化，如果还有什么问题，请告诉我

编辑：

C++11 很漂亮，因为他的简化为 ;-)

map< char , vector< int > > counters ;

for (i = 0 ; str[ i ] ; i ++ )
{
    counters[ str[ i ] ].push_back( i ) ;
}

for( pair< char , vector< int > > liste : counters )
{
    vector< int > vec = liste.second ;

    if( vec.size( ) == number )
        for( int elt : vec )
            str[ elt ] = 'A' ;
}

说明：我遍历数组，将每个索引存储在与 char 关联的索引列表中（地图的用途）

最后，我只查看每个字符的向量的大小，如果大小等于number，我会通过这个列表将每个元素“指向”到“A”的索引