2

サイトを検索しましたが、問題の解決策が見つかりません。小さな変更を加えてみましたが、何も解決しませんでした。「string subscript out of range」エラーが発生し続けます。何故かはわからない。多分私は盲目で、どこかで小さなエラーを見落としています。今、私はここに助けを求めています。

プログラムに関する情報: このプログラムは、姓名を入力して検証し、大文字と小文字の変換を適用します。ユーザーが姓名を入力すると、画面がクリアされ、ユーザーが入力した名前が表示されます。次に、製品の評価を入力して検証し、大文字と小文字の変換を適用します。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 << "*";
    }
}
4

2 に答える 2

1

45行目

Rating (ratingString, ratingInt, MAX_PRODUCTS);

ratingStringは空です。Line76まで走ったとき

    cin  >> ratingString[count];

境界外のインデックスを参照しています。

この編集はどうですか：

    char cc;
    cin  >> cc;
    ratingString.push_back(cc);
于 2012-11-27T10:04:14.053 に答える
0

私は以下のループを信じています。MAX

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

次に、以下のループで、使用していて、文字列の長さcount++を超えています。ratingString

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

この問題を解決するには、正しいインデックスとインクリメントを使用するか、文字列の長さもチェックしてください。

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

文字列インデックスの文字が使用されるすべてのループに文字列長チェックを配置することをお勧めします。

于 2012-11-27T05:21:41.307 に答える