C++クラスの配列を作成して印刷する

Question

クラスの配列を作成して印刷したいと思います。

今の私のアプローチは、次のとおりです。

1. その配列を印刷する機能を使用して、これらのクラスの1
2. の配列を作成する機能を使用
3. 使用集約
4. に2つの異なるクラスを作成します。
5. すべての関数は型voidでなければなりません。

#include <iostream> 
#include <string> 

using namespace std; 
const int invsize = 5; 

//first class: 
class Item { 
    private: 
     //i'm supposed to have all values set to private 
     string itname; 
     int itpower; 
     int itlasting; 
    public: 
     //this is so I can create an array of those classes later: 
     Item(string i, int j, int k) { itname = i; itpower = j; itlasting = k; } 
     //a method to print this class: 
     void itprint() {cout << itname << itpower << itlasting;}; 
     ~Item(){}; 
}; 

//second class: 
class Player { 
    private: 
     string plname; 
     int plrole; 
     int plbrain; 
     int plbrawn; 
     //aggregation: 
     Item * Inventory; 
    public: 
     void plprint(); 
     Player(); 
     ~Player(); 
}; 

//a Player constructor, in which I want to create the array of "Items" also: 
Player::Player() { 

    /*I'm adding info to the Player class first, it works fine and is irrelevant to my question so I didn't paste it here*/ 

    //now I'm creating an array of Item classes and giving them values: 
    Item Inventory[4] = { Item("generic name", 5, 5), Item("generic name", 5, 5), 
       Item("generic name", 5, 5), Item("generic name", 5, 5) };  
} 

//a method to print the Player class and the "Inventory" array: 
void Player::plprint() { 

    /*printing the info about the Player class here first, again, works fine and is irrelevant so I didn't copypaste it*/ 

    //printing the "Inventory" array: 
    cout << "Your inventory contains:" << endl; 
    for (int i=1;i<invsize;i++) { 
     Inventory[i].itprint(); 
    } 
} 

//and finally the main: 
int main() { 

    //creating a Player class - as I understand, the "Inventory" array should get created with it 
    Player * firstPlayer = new Player; 
    //printing the created object and its inventory 
    firstPlayer->plprint(); 

    system("pause"); 
    return 0; 
} 

プログラムをコンパイルし、私はそれを実行すると、正しくプレイヤークラスを作成しているようだ、それもうまくプレーヤーに関する情報を印刷し、その後、何かがうまくいかない：は、ここで私がこれまで持っているものですアレイとすべてのものがクラッシュする。

私はここで何かを理解できないと確信しています。私はどんな種類の助けにも非常に感謝しています。私はたくさんのグーグルで探せましたが、関連するチュートリアルや質問に答えられませんでした。前もって感謝します！また、私の悪い英語を言い訳、それは私の第二言語です。

Answer 1

それは正しく

プレイヤークラスを作成しているようだ、それはしませんでした。

Player::Player() { 

/*I'm adding info to the Player class first, it works fine and is irrelevant to my question so I didn't paste it here*/ 

//now I'm creating an array of Item classes and giving them values: 
Item Inventory[4] = { Item("generic name", 5, 5), Item("generic name", 5, 5), 
      Item("generic name", 5, 5), Item("generic name", 5, 5) };  

}

あなたはプレイヤーのコンストラクタでインベントリと呼ばれる項目の配列を作成している、それはあなたのメンバ変数のインベントリに言及していない、ローカル変数です。あなたのコンストラクタの最後、ローカル配列インベントリがところで...

を破壊されたときに

だから、なぜあなたは、インベントリの配列のためのポインタを使うのですか？配列そのものを使って簡単に行うことができます。

アレイに問題が発生し、すべてがクラッシュします。

void Player::plprint() { 

/*printing the info about the Player class here first, again, works fine and is irrelevant so I didn't copypaste it*/ 

//printing the "Inventory" array: 
cout << "Your inventory contains:" << endl; 
for (int i=1;i<invsize;i++) { 
    Inventory[i].itprint(); 
} 

}

あなたが1からあなたのアレイをカウントし、印刷しているので、それは、配列は0から開始する必要があります。だから、C++、またはほとんどすべてのプログラミング言語で、配列はあなたにエラーが発生しますので、もしあなたがアイテムの在庫を宣言しようとしているとき、あなたはクラスのアイテムのデフォルトコンストラクタが欠落している0

から開始し、for (int i = 0; i < invsize - 1; ++i) {/*...*/} 重要を使用する必要がありますまたはアイテムのインベントリ[4]

固定コード：

#include <iostream> 
#include <string> 

using namespace std; 
const int invsize = 5; 

//first class: 
class Item { 
private: 
    //i'm supposed to have all values set to private 
    string itname; 
    int itpower; 
    int itlasting; 
public: 
    //this is so I can create an array of those classes later: 
    Item() {}; 
    Item(string i, int j, int k) { itname = i; itpower = j; itlasting = k; } 
    //a method to print this class: 
    void itprint() { cout << itname << " " << itpower << " " << itlasting << std::endl; }; 
    ~Item() {}; 
}; 

//second class: 
class Player { 
private: 
    string plname; 
    int plrole; 
    int plbrain; 
    int plbrawn; 
    //aggregation: 
    Item Inventory [4]; 
public: 
    void plprint(); 
    Player(); 
    ~Player(); 
}; 

//a Player constructor, in which I want to create the array of "Items" also: 
Player::Player() { 

    /*I'm adding info to the Player class first, it works fine and is irrelevant to my question so I didn't paste it here*/ 

    //now I'm creating an array of Item classes and giving them values: 
    for (int i = 0; i < 4; ++i) 
    { 
     Inventory[i] = Item("generic name", 5, 5); 
    } 
}; 

//a method to print the Player class and the "Inventory" array: 
void Player::plprint() { 

    /*printing the info about the Player class here first, again, works fine and is irrelevant so I didn't copypaste it*/ 

    //printing the "Inventory" array: 
    cout << "Your inventory contains:" << endl; 
    for (int i = 0; i<invsize - 1; i++) { 
     Inventory[i].itprint(); 
    } 
} 

//and finally the main: 
int main() { 

    //creating a Player class - as I understand, the "Inventory" array should get created with it 
    Player * firstPlayer = new Player; 
    //printing the created object and its inventory 
    firstPlayer->plprint(); 

    system("pause"); 
    return 0; 
}