c# - ac# クラスはそれ自体のインスタンスを生成する必要がありますか?

Question

CallRate 型を定義するクラスがあります。ファイルからデータを読み取って、クラスの複数のインスタンスを作成する機能を追加する必要があります。

を返す静的メソッドをクラス CallRate に追加しましたList<CallRate>。クラスが独自のコンストラクターの 1 つを呼び出して、それ自体の新しいインスタンスを生成することは問題ありませんか? それは機能しますが、それが適切なことなのかどうか疑問に思っています。

List<CallRates> cr = CallRates.ProcessCallsFile(file);

score 20 · Accepted Answer

静的メソッドから独自のオブジェクトを取得することはまったく問題ありません。

例えば

ドットネットライブラリの 1 つは、あなたと同じことを行います。

XmlReadrer reader = XmlReader.Create(filepathString);

score 5 · Accepted Answer

確かにそれは問題ありません。場合によっては奨励されることさえあります。オブジェクトの作成を扱う設計パターンがいくつかあり、そのうちのいくつかはまさにあなたが説明していることを行います。

score 3 · Accepted Answer

パラメータの有効性を確認する必要がある場合、このパターンをよく使用します。コンストラクターから例外をスローしないことを強くお勧めします。ファクトリメソッドからそれほど悪くはないか、null を返すことを選択できます。

score 2 · Accepted Answer

私には問題ないようです。他の言語では、おそらく関数を記述しますが、C# のような言語では、静的メソッドがその役割を果たします。

score 1 · Accepted Answer

「独自のコンストラクターの1つを呼び出すことによってそれ自体の新しいインスタンスを生成する」ことを指摘するのと同じように

コンストラクターからではなく、静的メソッドからのものです。

score 1 · Accepted Answer

コンストラクターのオーバーロードの代わりに public static メソッドを使用することがあります。

特に、どのような種類のオブジェクト構築が意図されているかを示すためにパラメーターの型だけに依存するのは好ましくない状況では。

score 1 · Accepted Answer

上記で何度も示唆されているように、私は静的メソッドがインスタンスを返すことのファンです。

@Paul：上記のコメントにチェックを入れることを忘れないでください。これが最良の答えです。

score 1 · Accepted Answer

大丈夫です。作成したのは、単純なファクトリメソッドのようなものです。型の有効なインスタンスを作成する静的メソッドがあります。実際、メソッドは静的である必要さえなく、有効なコードを持っています。既存のオブジェクトから新しい有効なオブジェクトを作成するデザインパターン (Prototype) があります。http://www.dofactory.com/Patterns/PatternPrototype.aspxで詳細を参照してください。

score 1 · Accepted Answer

確かに、単純な解析 (または同様の) シナリオの場合、実際にはファクトリメソッドをクラスの一部にすることを好みます。はい - それはSRPを壊しますが、 KISSを満たします - だから私はそれを純勝利と呼んでいます。大規模なアプリ、またはより複雑な解析ルーチンの場合は、外部ファクトリクラスにする方が理にかなっています。

あなたの特定のケースでは、おそらくファイル名の代わりに IEnumerable<string> を取ったメソッドを好むでしょう-それでも解析ロジックを提供しますが、簡単な単体テストと「再利用」を可能にします。呼び出し元は、ファイルを IEnumerable に簡単にラップできます。

score 1 · Accepted Answer

多くの場合、ファクトリメソッドは適切な設計です。それらを C# で記述するときは、それらを「New」と呼ぶので、次のようになります。

new MyClass()

になる

MyClass.New()

自明なことに、次のように実装されています。

class MyClass
{
    public static MyClass New()
    {
        return new MyClass();
    }
}

ほとんどの場合、実際にクラスを作成するか、単に return を作成するnullか、または returnMyClassまたはそれから派生したものを作成するかについて、追加の条件がある場合にこれを行います。

score 1 · Accepted Answer

私は通常、クラスの即時実装が必要なときにこれを使用します。例えば

    public class Car
    {
        public static Car RedExpensiveCar = new Car("Red", 250000);

        public Car()
        {

        }

        public Car(string color, int price)
        {
            Color = color;
            Price = price;
        }

        public string Color { get; set; }
        public int Price { get; set; }
    }

これにより、コンストラクターのパラメーターを覚えたり、コードに記述したりする必要がなくなります。

Car car = Car.RedExpensiveCar;

score 0 · Accepted Answer

これを行うことは完全に受け入れられます。私がそうするとき、私は通常、クラスの実際のコンストラクターをプライベートにして、インスタンスを構築する唯一の方法が静的メソッドを使用することであることを明確にします。

これは、「構築」が常に新しいインスタンスを返すとは限らない場合に非常に役立ちます。たとえば、代わりに以前にキャッシュされたオブジェクトを返したい場合があります。

c# - ac# クラスはそれ自体のインスタンスを生成する必要がありますか?

12 に答える 12

Related

Reference