次のように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);
}
別のコンストラクターからコードをコピーしないようにする必要があります...
両方のコンストラクターから呼び出される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;
}
public Point2D(Point2D point) : this(point.X, point.Y) { }
たぶんあなたのクラスは完全ではありません。個人的には、オーバーロードされたすべてのコンストラクターでプライベート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;
}
}