C＃の引数を変更して返すのに最適ですか？

Question

すでにいくつかのプロパティ値が設定されているウィジェットを用意する必要があります。私はウィジェットの名前を変更する必要があります。私はオプション3に向けて描かれていますが、私はなぜその理由を明確に説明していません。

public void Do(Widget widget) { // 1 
    widget.Name = "new name"; 
} 

public void Do(ref Widget widget) { // 2 
    widget.Name = "new name"; 
} 

public Widget Do(Widget widget) { // 3 
    widget.Name = "new name"; 
    return widget; 
} 

私はいくつかの質問に悪魔の支持者を遊ぶのが好きと回答を集め、私はオプション3

オプション1に描かれていますなぜ私が説明するためにしたい：ちょうどなぜない渡されたウィジェットを変更しますか？あなたは1つのオブジェクトを "返す"だけです。渡されたオブジェクトを使用するだけではどうですか？

オプション2：なぜでないか返品無効ですか？なぜ、パラメータオブジェクト自体への実際のメモリポインタを使用しているというシグネチャで通信するだけではないのですか？

オプション3：あなたが渡しているのと同じオブジェクトを返すことはあなたにとって奇妙ではありませんか？

Answer 1

オプション1：これは最も一般的なアプローチである - あなたが参照自体を変更したくない場合は、参照を必要としません。渡されたオブジェクトが実際に変更されていることを期待して、メソッドの名前を適切に指定してください。

オプション2：これは、あなたが新しいWidgetインスタンスを作成したり、既存のウィジェットを指すように参照を設定つまり、渡されたリファレンス自体を変更したい場合は、あなたがしたい場合は（これが役に立つかもしれないだけに有用ですそれらがすべて同じプロパティを持つ場合は全体の数を低く保ちます。Flyweight patternを参照してください。返されたウィジェットは、この場合は不変でなければなりません。あなたの場合、これは適切ではないようです。

オプション3：これは流暢「ビルダー」のアプローチを可能に - また、その利点、一部の人々は、より表現力と自己文書化を検討ウィジェットへの変更のすなわちチェーンを持っています。また、Fluent interface

Answer 2

実際には3つのオプションの間に機能的な違いはありません。 （あなたの質問に関連しているの違い、ちょうどなしがあります。）それをシンプルに保つ - useオプション1.

Answer 3

オプション1：

ファイン。まさに私が使うもの。ウィジェット自体の内容は変更することはできません。

オプション2：

いいえいいえいいえ。あなたは、参照自体を変更しないので、あなたはrefを使用する必要はありません。あなたが参照自体を変更した場合（たとえばwidget = new Widget()、次いでout/refは正しい選択ですが、私の経験ではほとんど必要だと

オプション3：オプション1と同様に
しかし、流暢に連鎖させることができます個人的には私はこれが好きではありません。コピーを返してオリジナルのオブジェクトをそのままにしておくと、その署名だけが使用されます。

---

しかし、ここで最も重要なことは、どのようにあなた名前方法。その名前は、元のオブジェクトが変異していることを明確に暗示する必要があります。

多くの場合、オプション4を選択します。タイプを不変にしてコピーを返します。しかし、明らかに実体であり価値がないウィジェットでは、これは意味をなさない。

Answer 4

私はオプション1とオプション3の両方が実行可能なオプションだと思います。オプション3は、メソッドのウィジェットを変更していることを意味するという点で、自己文書化の利点があります。私は最悪のオプションはオプション2だと思う。refというキーワードは、あなたが最も確実にやっていないオブジェクトの参照を変更していることを意味する。