クラスへのポインタ*

Question

私はクラス のrpgゲームで作業しています。インスタンスを格納するActionList *を持つ構造体呼び出しCharacterを作成しました。 GeneralPlayerは、他のプレーヤークラスが継承されているクラスです。 これは私のヘッダファイルです：私はvoid *にGeneralPlayer *を変換しようとしていた

class Battle 
{ 
public: 
     struct Character 
     { 
     char type;//monster or player? 
     bool alive; 
     void*instance;//pointer to instance 
     }; 
     Battle(GeneralPlayer*,AbstractMonster*,int,int,int);  
     Battle(GeneralPlayer*, AbstractMonster*, int, int); 
private: 
     Character *ActionList; 
}; 

。しかし、コードは私が思ったように動作しないようです。 PおよびMは、それらのプレーヤクラスのポインタの配列である。

Battle::Battle(GeneralPlayer*P, AbstractMonster*M, int a, int b, int c) 
{ 
    a = numP; 
    b = numM; 
    c = turn_limit; 
    ActionList = new Character[numP + numM]; 
    P = new GeneralPlayer[numP]; 
    for (int i = 0; i < numP; i++) 
    { 
     ActionList[i] = static_cast<void*>(P[i]); 
     ActionList[i].type = 'p'; 
    } 
    for (int i = numP; i < numP+numM; i++) 
    { 
     ActionList[i] = static_cast<void*>(M[i]); 
     ActionList[i].type = 'm'; 
    } 
} 

エラーC2440が表示され続けます。誰もが私の問題を解決することができれば幸いです。

Answer 1

一つCharacter構造がGenericPlayerまたはAbstractMonsterどちらかの親ではないということです。あなたのコードは、これは、その後、選手のリストがすでにBattleコンストラクタの呼び出し元によって初期化されていることを想定している

ActionList[i].type = 'p'; 
ActionList[i].alive = true; 
ActionList[i].instance = &P[i]; 

のようなものであるべきことを意味する、Character::instanceメンバーはプレイヤーやモンスターを指すべきであると思われます新しいプレイヤーの配列を割り当てるべきではないので、P = new GenericPlayer[numP];ステートメントを削除する必要があります。

"generic pointer"（what void *）を実際に指していると言ったメンバーは、悪いデザインとみなされます。代わりに、モンスターとプレイヤーの両方に共通の基本クラスを用意し、それを指すポインターを使用します。次に、多型とvirtualメンバー関数を正しく使用すると、typeフィールドは必要ありません。そして、プレイヤーやモンスターが生きているかどうかを伝える他の手段を使用するコードをリファクタリングするのは簡単ですし、Battle::Characterクラスはまったく必要なく、共通基底クラスへのポインタの配列を使うことができます代わりにコードを少し簡略化して、保守性に非常に優れています。

---

コードを表示する際に問題がいくつかあります。実行時に問題が発生することがあります。

一つの問題は、モンスターをループの反復では、あなたがnumP + numMまでnumPとループにoを初期化するが、配列MがnumP + numMの要素が含まれていない場合は、境界の外に行こうということです。

代わりに、私はあなたに教えてください。

for (int i = 0; i < numM; i++) 
{ 
    ActionList[i + numP].type = 'm'; 
    ActionList[i + numP].alive = true; 
    ActionList[i + numP].instance = &M[i]; 
} 

Answer 2

オブジェクトをポインタに変換しようとしていますが、&演算子を使用して問題のポインタを取得してください。ここでの問題の

ActionList[i] = (void*)&P[i]; 
ActionList[i] = (void*)&M[i];