私は現在、カードゲームに取り組んでいます

Question

オブジェクトの配列を初期化し、私はいくつかの初期化コードとのトラブルを抱えている：

// in my class... 
Card cards[20]; 
// in method... 
for(int i = 0; i <= 20;i++) 
    cards++ = new Card(i, /*i as char +*/ "_Card.bmp"); 

トラブルが私のコンパイラのがcards++はリットルではないことを私に告げることです-値。ポインタ配列の等価物全体を読み上げましたが、私はそれを理解していると思っていましたが、うーん、それを動作させることはできません。 cardsがポインタに劣化し、new演算子が私の新しいカードインスタンスの場所へのポインタを与えるので、私の理解は、上記のコードはコンパイルする必要があります。右？

添え字も試してみましたが、cards+i、cards++、cards[i]という同じことを言っているだけではありませんか？私はそれらのそれぞれがl値であり、ポインタとして扱われると考えました。

Answer 1

Card cards[20]; 

cardsはすでにオブジェクトの配列です。それらはデフォルトコンストラクタ（引数のないコンストラクタ）で構築されます。もう一度newする必要はありません。おそらく、コンストラクタ引数に相当するメンバ関数が必要であり、それを介して代入する必要があります。

for (int i=0; i<20; ++i) // array index shouldn't include 20 
    cards[i].memberFunction(/*....*/); 

さらに簡単にstd::vector

std::vector<Card> cards; 
for(int i=0; i<20; ++i) 
    cards.push_back(Card(i, /*i as char +*/ "_Card.bmp");) 

Answer 2

コードCard cards[20];既に20個のCardオブジェクトの配列を作成し、デフォルトコンストラクタでそれらを作成し使用することです。これはあなたのコードを与えたいとは思っていないかもしれません。

代わりにvectorを使用することをおすすめします。あなたのforループは配列の最後を過ぎ20から0ので、1から行く

std::vector<Card> cards; 

for(int i = 0; i < 20;i++) 
{ 
    cards.push_back(Card(i, /*i as char +*/ "_Card.bmp")); 
} 

注意。

Answer 3

との不必要なサイズ変更を避けたい場合、C++は通常、各オブジェクトを割り当てられた状態で1つずつ初期化するため、より複雑です。回避策の一つは、それをJavaの道を行うことです - ので、同じように、ループやポインタの配列を使用します。

Card *cards[20]; 
for (int i=0; i<20; i++) { 
    cards[i] = new Card(i); 
} 

別のオプションが明示的に初期化されていないメモリを取得するためのmallocを使用することです：

Card *cards = malloc(20 * sizeof(Card)); 
for (int i=0; i<20; i++) { 
    new (&(cards[i])) Card(i); 
}