我收到“字符串下标超出范围错误”。我不知道为啥

Posted 2023-02-22

【中文标题】我收到“字符串下标超出范围错误”。我不知道为啥

【英文标题】：I'm getting a "string subscript out of range error". I can not figure out why我收到“字符串下标超出范围错误”。我不知道为什么

【发布时间】：2012-11-27 05:18:00

【问题描述】：

我搜索了该网站，但找不到解决问题的方法。我尝试进行细微的更改，但没有任何解决方法。我不断收到“字符串下标超出范围”错误。我不知道为什么。也许我是盲人，我在某处遗漏了一个小错误。现在我在这里请求帮助。

关于程序的信息：该程序将输入名字和姓氏，对其进行验证并应用大小写转换。用户输入名字和姓氏后，它将清除屏幕并显示用户输入的姓名。然后它将输入产品评级，对其进行验证并应用案例转换。显示一个标题，后跟一个对应于 5 个产品值的条形图。

编辑：我想对帮助过我的人说声谢谢。我终于解决了这个问题，感谢你们。我不得不说这里有一个很棒的社区，反应非常好。我现在要去上课，但我会为将来可能遇到同样问题的人发布我更新的代码。再次非常感谢你们。

#include <iostream>
#include <string>

using namespace std;

void Name (string, string&, const int);
void Rating (string&, int[], const int);

void main()

    const int MAX_FIRST_NAME = 20;
    const int MAX_LAST_NAME  = 25;
    const int MAX_PRODUCTS   =  5;

    string  firstNameQuestion = "First Name";
    string  lastNameQuestion  = "Last  Name";

    string    firstName;
    string     lastName;
    string ratingString;

    int ratingInt [MAX_PRODUCTS];


    while (true)
    
        Name (firstNameQuestion, firstName, MAX_FIRST_NAME);

        if (firstName == "Quit")
            break;

        Name (lastNameQuestion, lastName, MAX_LAST_NAME);

        if (lastName == "Quit")
            break;

        system ("cls");

        cout << "First Name: "  << firstName;
        cout << endl;
        cout << "Last  Name: "  << lastName;
        cout << endl;
        cout << endl;

        Rating (ratingString, ratingInt, MAX_PRODUCTS);
       


void Name (string question, string& answer, const int MAX)

    int count;

    do
    
        cout << question << " (" << MAX << " chars max. type \"quit\" to stop):     ";
        getline (cin, answer);
    
    while (answer.empty() || answer.length() > MAX);

    answer[0] = toupper (answer[0]);

    for (count = 1; count < answer.length(); count++)
        answer[count] = tolower ( answer[count] );


void Rating (string& ratingString, int ratingInt[], const int MAX)

    int  count;
    int    who;

    for (count = 0; count < MAX; count++)
    
        do
        
            cout << "Rating for product no." << count + 1 << " (A to E): ";
            cin  >> ratingString[count];
            ratingString[count] = toupper (ratingString[count]);
           
        while (ratingString.empty() || ratingString.length() > 1 ||     ratingString[count] > 'E');
    

    for (who = 0; who < MAX; who++)
    
        if (ratingString[who] == 'A')
            ratingInt[who] = 10;

        if (ratingString[who] == 'B')
            ratingInt[who] = 8;

        if (ratingString[who] == 'C')
            ratingInt[who] = 6;

        if (ratingString[who] == 'D')
            ratingInt[who] = 4;

        else 
            ratingInt[who] = 2;
    

    cout << endl;
    cout << endl;
    cout << "Consumer satisfaction bar chart: ";
    cout << endl;

    for (count = 0; count > MAX; count++)
    
        cout << endl;
        cout << "Product #" << count + 1 << "      ";

        for (who = 0; who > ratingInt[count]; who++)

            cout << "*";
    

【问题讨论】：

我从你的标签中删除了 C#，因为这显然是 C++。

@SimonWhitehead 不是 OP，是编辑

【参考方案1】：

第 45 行

Rating (ratingString, ratingInt, MAX_PRODUCTS);

ratingString 为空。 当它运行到 Line76

    cin  >> ratingString[count];

您正在引用超出边界的索引。

这个编辑怎么样：

    char cc;
    cin  >> cc;
    ratingString.push_back(cc);

【讨论】：

非常感谢，我昨晚解决了这个问题，没有从 void main 中获取变量，而是在 Input Rating 中创建了“string ratingString”。下课回来后，我会发布我的新代码，以便人们看到我的更改。

【参考方案2】：

我相信下面的循环，计数达到MAX

    for (count = 0; count < MAX; count++)

然后在下面的循环中再次使用count++，它超出了字符串ratingString 的长度。

   for (who = 0; who < MAX; count++)

要解决此问题，请使用更正索引+增量或同时检查字符串长度。

   for (who = 0; who < MAX && who < ratingString.length(); who++)

最好将字符串长度检查放在使用字符串索引处的字符的所有循环中。

【讨论】：

即使这是我错过的错误。它没有解决错误。很好的发现，不敢相信我错过了。

@user1855446 我仍然认为有问题。如果名称中没有像MAX 那么多的字符怎么办。您需要检查字符串长度以及我在更新的答案中提到的。

如果我按照你说的做，程序会跳过 for 循环。

@user1855446 哪个for 循环被跳过了？确保您的字符串已填充。 如果 string unavailability(length=0) 导致 for 循环被跳过，那么这也是下标错误的原因。

非常感谢，这就是问题所在。我昨晚在你的帮助下解决了这个问题。您对该主题的反应时间很棒，非常感谢。