C++モンスターのベクトル

Question

タイプが私のオーバーロードコンストラクターであるか、正確にはモンスターのベクトルを作りたいと思っていますが、私はそれを渡すと思われます。私は何を間違えたのですか？

// Monster.cpp 

#include <iostream> 
#include <string> 
#include "Monster.h" 
#include "Player.h" 
#include "Random.h" 
#include "Weapon.h" 
using namespace std; 

Monster::Monster(const std::string& name, int hp, int acc, 
    int xpReward, int armor, const std::string& weaponName, 
    int lowDamage, int highDamage, int monstergold) 
{ 
    mName = name; 
    mHitPoints = hp; 
    mAccuracy = acc; 
    mExpReward = xpReward; 
    mArmor = armor; 
    mWeapon.mName = weaponName; 
    mWeapon.mDamageRange.mLow = lowDamage; 
    mWeapon.mDamageRange.mHigh = highDamage; 
    mGold = monstergold; 
} 

---

これは、ロールが20よりも高かった場合、それはエラーにモンスター

else if (roll > 20) 
{ 
    vector <Monster(const std::string& name, int hp, int acc,int xpReward, int armor, const std::string& weaponName, int lowDamage, int highDamage, int monstergold)> MonsterArray; 
    MonsterArray.push_back("Orc Lord", 25, 15, 2000, 5,"Two Handed Sword", 5, 20, 100); 


    cout << "You encountered an multiple monsters!!!" << endl; 
    cout << "Prepare for battle!" << endl; 
    cout << endl; 
} 

---

のグループをして遭遇する必要があり、それはありませんオーバーロードされた関数と言う、マップであり、 。私はこれが間違っていることを知っているが、私は本当にそれをやる方法が全くわからない。何かアドバイス？

Answer 1

あなたはテンプレートではなく、コンストラクタに呼び出そうと型名を指定する必要があります。

// Here you define that vector will contain instances of Monster class 
vector<Monster> MonsterArray; 

// Add new monster by explicitly calling 
// to the constructor and pushing into container 
MonsterArray.push_back(Monster("Orc Lord", 25, 15, 2000, 5,"Two Handed Sword", 5, 20, 100)); 

私はThe C++ Programming Language本を読むことをお勧めしたいものの。

---

また、あなたが例えば、ベクトルコンテナの含ま不足しているようだ。：

#include <vector> 

Answer 2

あなたのベクトルを宣言しながら、あなたのモンスタークラスのすべてのパラメータを指定する必要はありません。 は、単にそのような何か：（[良い初心者の書籍のカップルを読む]私はあなたがする必要があると思う

std::vector<Monster> monsters; 
monsters.push_back(new Monster("Orc Lord", 25, 15, 2000, 5,"Two Handed Sword", 5, 20, 100));