web-dev-qa-db-ja.com

C#コンストラクターのオーバーロード

次のようにC#でコンストラクタを使用する方法:

public Point2D(double x, double y)
{
    // ... Contracts ...

    X = x;
    Y = y;
}

public Point2D(Point2D point)
{
    if (point == null)
        ArgumentNullException("point");
    Contract.EndContractsBlock();

    this(point.X, point.Y);
}

別のコンストラクターからコードをコピーしないようにする必要があります...

61

両方のコンストラクターから呼び出されるInitializeなど、プライベートメソッドに共通のロジックを含めることができます。

引数の検証を実行したいという事実のため、コンストラクターチェーンに頼ることはできません。

例:

public Point2D(double x, double y)
{
    // Contracts

    Initialize(x, y);
}

public Point2D(Point2D point)
{
    if (point == null)
        throw new ArgumentNullException("point");

    // Contracts

    Initialize(point.X, point.Y);
}

private void Initialize(double x, double y)
{
    X = x;
    Y = y;
}
61
João Angelo
public Point2D(Point2D point) : this(point.X, point.Y) { }
177
Mark Cidade

たぶんあなたのクラスは完全ではありません。個人的には、オーバーロードされたすべてのコンストラクターでプライベートinit()関数を使用しています。

class Point2D {

  double X, Y;

  public Point2D(double x, double y) {
    init(x, y);
  }

  public Point2D(Point2D point) {
    if (point == null)
      throw new ArgumentNullException("point");
    init(point.X, point.Y);
  }

  void init(double x, double y) {
    // ... Contracts ...
    X = x;
    Y = y;
  }
}
5
jp2code