0
struct point {
    int x; int y;
    point(int a, int b) {
        x = a;
        y = b;
    }
};
class circle {
public:
    point p;
    double r;
    circle(point a, double b) {
        p = a;
        r = b;
    }
    //circle(point a, double b) : p(a), r(b) {}; //no error
};
int main() {
    point p1(2,7);
    circle c(p1, 0.3);
    cout << c.p.x << c.p.y << endl;
    return 0;
}

質問: circle のコンストラクタの部分で”クラスpointの規定のコンストラクタは存在しません”というエラーが出ます。pointの引数2個のコンストラクタを作成したはずなのになぜでしょうか。

2
  • 使っている言語はC++でしょうか。
    – nekketsuuu
    Commented 2017年2月19日 1:38
  • 1
    candidate: point::point(int, int) expects 2 arguments, 0 provided というエラーメッセージも表示されていませんか? 2個の引数を取るコンストラクタを定義してしまっているので、コンパイラはデフォルトコンストラクタを自動生成してくれません。そのため、デフォルトコンストラクタ(point() {...})を明示的に定義する必要があります。
    – user39889
    Commented 2017年2月19日 2:26

2 件の回答 2

3

コンストラクタでは{...}内の処理が始まる前にクラスメンバをすべて初期化します。
このcircleクラスのコンストラクタではメンバの初期化方法が記述されていない(:以降がない)
ため、各メンバの「デフォルト」の初期化方法で初期化します。
クラス、構造体のデフォルトの初期化方法は引数なしコンストラクタ呼び出しです。
ですが、他の回答にあるようにcircleは引数2つのコンストラクタを定義しているので、
引数なしのコンストラクタは暗黙には定義されません。

という理由で、ご質問のコンパイルエラーになっていると思います。
エラーを回避するには以下のどちらかの方法をとるとよいと思います。

  1. 質問のコメントアウト箇所のようにpを初期化する
  2. 構造体pointに引数なしのコンストラクタを定義する

なお、もとのコードの「p = a」の箇所は代入演算子(operator=())が実行されている
はずです。暗黙に生成されるそれは「shallow copy」ですのでポインタが指している内容を
コピーしたい場合、自前で「deep copy」する代入演算子を定義する必要があります。

2
  • 2
    2点目の指摘は誤りです。ユーザ定義コンストラクタを宣言しても、暗黙のコピーコンストラクタは生成されます。
    – yohjp
    Commented 2017年2月19日 4:49
  • 2
    確認しましたが、指摘どおり暗黙のコピーコンストラクタは生成されますね。回答修正します。
    – user20098
    Commented 2017年2月20日 14:17
2

この形のコンストラクタ

circle(point a, double b) {
    p = a;
    r = b;
}

は各親クラス及びフィールドの引数なしのコンストラクタ(=デフォルトコンストラクタ)を呼び出すことになります。

クラスにコンストラクタが定義されていない場合、引数なしのコンストラクタは自動的に作成されます。pointクラスには、ほかのコンストラクタ(point(int, int))が定義されていたから、引数なしのコンストラクタが自動的に作成されません。

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。