C＃で何も返さずに参照変数を変更するのは悪い習慣と考えられますか？

Question

メソッド内の参照変数を返さずに変更すると、悪い習慣とみなされますか？例えば

：

public Widget GetWidget() 
{ 
    Widget widget = FetchWidget(); 
    ApplyPartNamePrefixes(widget); 
    return widget; 
} 

public void ApplyPartNamePrefixes(Widget widget) 
{ 
    widget.PartName1 = String.Format("{0}{1}", "*", widget.PartName1); 
    widget.PartName2 = String.Format("{0}{1}", "**", widget.PartName2); 
} 

Answer 1

いいえ、それは大丈夫です。これが基準変数のポイントです。 :)

Answer 2

オブジェクト内の値が変更されることがドキュメントを呼び出すメソッドであれば問題ありません。これは、XMLのドキュメントまたは適切な命名規則によるものです。あなたはApplyPartNamePrefixes(Widget widget)を持っていますが、これは幾分暗黙のうちにウィジェットの状態を変更することを暗示していますが、DoStuffがあいまいである場合は、GetDataのようになります。