0

すべて問題ないように見えますが、入力Iすると次のように表示されます

Unhandled exception at 0x64A1EB90 (msvcr110d.dll) in ConsoleGame1.exe: 0xC0000005: Access violation writing location 0xCCCCCCCC.

First-chance exception at 0x64A1EB90 (msvcr110d.dll) in ConsoleGame1.exe: 0xC0000005:     Access violation writing location 0xCCCCCCCC.
Unhandled exception at 0x64A1EB90 (msvcr110d.dll) in ConsoleGame1.exe: 0xC0000005: Access violation writing location 0xCCCCCCCC.
The program '[5088] ConsoleGame1.exe' has exited with code 0 (0x0).

コード：

void Inventory();

struct Item
{
    string itemName;
    string itemDescription;
    int itemNumber;
    bool haveItem;

    void DisplayItem();
};

int main()
{
    char inv;
hint:
    cout << "HINT: To open your inventory  press 'I'.\n";
    cin >> inv;
    if (inv=='I') Inventory();
    else goto hint;
    system("pause");
    return 0;
}

void Inventory()
{
    Item Letter =
    {
        Letter.itemName = "Letter",
        Letter.itemDescription = "...",
        Letter.itemNumber = 1,
        Letter.haveItem = true
    };
    Item Wood =
    {
        Wood.itemName = "Wood",
        Wood.itemDescription = "Birch wood.",
        Wood.itemNumber = 2,
        Wood.haveItem = false
    };
    Letter.DisplayItem();
    Wood.DisplayItem();
}
4

1 に答える 1

1

当面の問題に対処するために、まだ構築されていないオブジェクトに割り当てています。

Item Letter =
{
    Letter.itemName = "Letter",
    Letter.itemDescription = "...",
    Letter.itemNumber = 1,
    Letter.haveItem = true
};

Letterinitialisng の引数を指定するときに、のメンバーに割り当てていますLetter。それはしません。あなたが求めていたのは：

Item Letter =
{
    "Letter",
    "...",
    1,
    true
};

ただし、コードは一般的に、コメントで述べたように、ガイドとして優れた本を使用して基本から始める方がよいことを示しています。たとえば、ループの代わりに使用したくないことは間違いありません。gotoまた、Itemクラスはコンストラクターを使用できます。

于 2013-07-30T14:29:55.567 に答える