[C++] Error:no matching function for call to "Card::Card()"

Question

クラスの配列を宣言したところでエラーがでました。

Card c[52];

no matching function for call to "Card::Card()"

Cardクラスのコンストラクタは以下のようになります。

class Card{
public:
	int suit;
	int rank;
	Card(int s, int r){ suit = s; rank = r;}
}

Cardクラスを宣言する書き方が間違っているのでしょうか？
教えていただきたいです。

Answer 1

Card c[52];

ここで Card::Card() が(52回)呼び出されますが、Card::Card()が定義されていないのが原因。

class Card{
public:
    Card() { 適切に初期化 }
};

を追加するか、あるいは

class Card{
public:
 /*          ↓        ↓  適切なデフォルト値 */
 Card(int s=-1,int r=-1) { suit = s; rank = r;}
};

Comments

[amber_snob]

ありがとうございます。解決しました！

[SaitoAtsushi]

更に言うなら要素の初期化は初期化リストを使った方がよいですね。 コンストラクタは以下のような要領で書くのが好ましいです。

Card(int s=-1,int r=-1) : suit(s), rank(r) {}

int を含む基本的な型は明示的に初期化しない限り初期化されないルールなのでこの場合はどちらでもあまり差はないんですが、普通のクラス (のオブジェクト) の初期化で初期化リストを使わなかったらデフォルトコンストラクタで初期化してから代入するという形になってしまって無駄なので。