删除[]搞砸了我的程序。发生了什么事情？

我一眼就看不出问题，但有很多评论：

• 切勿写入裸new或delete。在非常的情况下，使用std::make_unique或类似方法。然而，对于该应用，std::string和std::vector将更好
• 如注释中所述，您没有将CountOfLetters的内容初始化为零（new不会为您执行此操作）
• checkLetters中的第一个循环（初始化Letters的地方）需要是一个双循环。（你并没有检查这个字母以前从未见过——只是它不是一个未初始化的值！）
• 您应该更喜欢将字符串作为常量引用传递

最后，checkLetters使用std::map:会更好

    bool checkLetters(const std::string& input) {
        if (input.length() == 0)
           return true;  // Just in case no characters at all.
        std::map<char,int> counts;
        for (const char c : input) {
            // No point in running over the string twice.  Do it in one go.
            if (!std::upper(c)) return false;
            // If there is an entry in the map, increment it.  If not,
            // create it (with value zero), and then increment.
            counts[c]++;
        }
        // input[0] would blow up if the string is length zero and we hadn't checked above
        const int target = counts[input[0]];
        for (const auto& keyvalue : counts)
        {
            const int value = keyvalue.second;
            if (value != target)
                return false;
        }
        return true;
   }

EDIT：我把问题误读为"输入中的每个字母都必须只有一个计数"。这是我的新答案：

我浏览了一下你的代码，纠正了你的错误。我基本上重写了您原始代码的部分内容，并制作了一个简单的checkLetters函数。根本不需要使用动态分配，除非特定情况需要。

您所能做的就是维护一个map<char,int>，它存储输入中每个字母的计数。在对输入进行交互之后，我们保留一个变量mustHaveCnt，它是每个字母必须具有的计数。现在，遍历映射中的所有字母，如果计数超过mustHaveCnt变量，则返回false。否则，答案是正确的

此外，我已经重写了您的输入条件。当输入为"完成"时，它会正确终止

#include <iostream>
#include <string>
#include <ctype.h>
#include <map>
using namespace std;
//check if everything is uppercase;
bool checkUpper(string input){
    for(int i=0;i<input.length();i++)
        if(!isupper(input[i])) return false;
    return true;
}
//Checks if the input has the same number of runners
//use must have the header file 'map' included, e.i write "#include <map>" at the top
bool checkLetters(string input){
    map<char,int> storeCnt;
    for(int i=0;i<input.size();i++)
        storeCnt[input[i]]++; //Increment the count of each letter found
    int mustHaveCnt=storeCnt.begin()->second; //The second entry stores the count
    for(map<char,int>::iterator it= ++storeCnt.begin(); it!=storeCnt.end(); it++) //Iterate and check cnt
        if(it->second != mustHaveCnt)
            return false; //terminate here since the count does not match
    return true;
}
int main(){
    cout<<"Enter a string of uppercase characters to indicate places(enter done to terminate):n";
    string input; //Let's avoid global declaration, unless needed
    cin>>input;
    while(input.compare("done")!=0)
    {
        int flag=1;
        if(!checkUpper(input)){
            cout<<"Uppercase letters ONLY! try again.n";
            flag=0;
        }
        if(flag && !checkLetters(input)){
            cout<<"Each team must have the same number of runners! try again.n";
            flag=0;
        }
        if(flag)
            cout<<"Valid String!n";
        cin>>input;
    }
    return 0;
}