web-dev-qa-db-ja.com

複数のコンストラクターを使用して、読みやすさを維持しながら重複したコードを削除するにはどうすればよいですか？

int a, b, c;

Constructor()
{
    a = 5;
    b = 10;
    c = 15;
    //do stuff
}
Constructor(int x, int y)
{
    a = x;
    b = y;
    c = 15;
    //do stuff
}
Constructor(int x, int y, int z)
{
    a = x;
    b = y;
    c = z;
    //do stuff
}

「もの」といくつかの割り当ての重複を防ぐために、次のようなことを試しました。

int a, b, c;

Constructor(): this(5, 10, 15)
{
}
Constructor(int x, int y): this(x, y, 15)
{
}
Constructor(int x, int y, int z)
{
    a = x;
    b = y;
    c = z;
    //do stuff
}

これは私がやりたいことのために機能しますが、新しいオブジェクトを作成したり、いくつかの計算を行うために、長いコードを使用する必要がある場合があります。

int a, b, c;

Constructor(): this(new Something(new AnotherThing(param1, param2, param3),
    10, 15).CollectionOfStuff.Count, new SomethingElse("some string", "another
    string").GetValue(), (int)Math.Floor(533 / 39.384))
{
}
Constructor(int x, int y): this(x, y, (int)Math.Floor(533 / 39.384))
{
}
Constructor(int x, int y, int z)
{
    a = x;
    b = y;
    c = z;
    //do stuff
}

このコードは以前とほとんど同じですが、渡されているパラメーターのみが読みやすくありません。私は次のようなことをすることを好むでしょう：

int a, b, c;

Constructor(): this(x, y, z) //compile error, variables do not exist in context
{
    AnotherThing at = new AnotherThing(param1, param2, param3);
    Something st = new Something(aThing, 10, 15)
    SomethingElse ste = new SomethingElse("some string", "another string");

    int x = thing.CollectionOfStuff.Count;
    int y = ste.GetValue();
    int z = (int)Math.Floor(533 / 39.384);

    //In Java, I think you can call this(x, y, z) at this point.
    this(x, y, z); //compile error, method name expected
}
Constructor(int x, int y): this(x, y, z) //compile error
{
    int z = (int)Math.Floor(533 / 39.384);
}
Constructor(int x, int y, int z)
{
    a = x;
    b = y;
    c = z;
    //do stuff
}

基本的に、コンストラクター本体内でパラメーターを作成しています。次に、これらのビルドされたパラメーターを別のコンストラクターに渡そうとしています。 Javaでコーディングするときに、「this」および「super」キーワードを使用して、別のコンストラクターの本体内でコンストラクターを呼び出すことができたことを覚えていると思います。 C＃では不可能のようです。

これを簡単に行う方法はありますか？何か間違ったことをしましたか？これが不可能な場合は、読み取り不能なコードをそのまま使用する必要がありますか？

複製されたコードを、コンストラクターの完全に別のメソッドに常にカットできると思います。次に、各コンストラクターは独自の処理を行い、他のコンストラクターによって共有されるコードを呼び出します。

c#parametersconstructorthiscode-duplication
40
Cheese

すべてのコンストラクター（readonlyフィールドを使用できないようにする）またはファクトリーメソッド（派生クラスを使用すると複雑さが増す）から初期化メソッドを呼び出す代わりに、 パラメーターオブジェクト ：

int a, b, c;

public Constructor()
    : this(new ConstructorParameters())
{
}

public Constructor(int x, int y)
    : this(new ConstructorParameters(x, y))
{
}

public Constructor(int x, int y, int z)
{
    a = x;
    b = y;
    c = z;
    //do stuff 
} 

private Constructor(ConstructorParameters parameters)
    : this(parameters.X, parameters.Y, parameters.Z)
{
}

private class ConstructorParameters
{
    public int X;
    public int Y;
    public int Z;

    public ConstructorParameters()
    {
        AnotherThing at = new AnotherThing(param1, param2, param3); 
        Something st = new Something(at, 10, 15) 
        SomethingElse ste = new SomethingElse("some string", "another string"); 

        X = st.CollectionOfStuff.Count; 
        Y = ste.GetValue(); 
        Z = (int)Math.Floor(533 / 39.384); 
    }

    public ConstructorParameters(int x, int y)
    {
        X = x;
        Y = y;
        Z = (int)Math.Floor(533 / 39.384);
    }
}
39
Michael Liu

できるよ

Constructor() : this(5, 10, 15)
{
}
Constructor(int x, int y) : this(x, y, 15)
{
}
Constructor(int x, int y, int z)
{
  int a = x;
  int b = y;
  int c = z;
  //do stuff
}

ただし、パラメーターに応じて派手なロジックを実行する必要がある場合は、 factory pattern を使用します。

public class myclass
{
  private myclass(int x, int y, int z)
  {
    int a = x;
    int b = y;
    int c = z;
    //do stuff
  }
  public static myclass Create()
  {
    AnotherThing at = new AnotherThing(param1, param2, param3);
    Something st = new Something(aThing, 10, 15)
    SomethingElse ste = new SomethingElse("some string", "another string");

    int x = thing.CollectionOfStuff.Count;
    int y = ste.GetValue();
    int z = (int)Math.Floor(533 / 39.384);

    return new myclass(x, y ,z);
  }
  public static myclass Create(int x, int y)
  {
    if (x = 1)
      return new myclass(x, y, 2)
    else 
      return new myclass(x, y, 15);
  }
  public static myclass Create(int x, int y, int z)
  {
    //so on and so forth
    return new myclass(x, y, z);
  }
}

ファクトリー・パターンは必要ありませんが、コンストラクター・ロジックを読みやすくします。

19
Erik Philips