参照型パラメータを変更するメソッドが不正ですか？私はこのような方法を見てきました

Question

：

public void Foo(List<string> list) 
{ 
    list.Add("Bar"); 
} 

は、メソッドのパラメータを変更するには、このことをお勧めますか？

これは良くありませんか？

public List<string> Foo(List<string> list) 
{ 
    // Edit 
    List<string> newlist = new List<string>(list); 
    newlist.Add("Bar"); 
    return newlist; 
} 

予期しない副作用が最初の例のように感じられます。

Answer 1

あなたが指定した例では、最初のものは2番目のものより私にはかなり良いと思われます。リストを受け入れてリストを返すメソッドを見た場合、私の最初の前提は新しいリストを返していて、それが与えられたリストに触れていないということです。したがって、第2の方法は予期しない副作用を伴う方法である。

メソッドの名前が適切であれば、パラメータを変更する危険性はほとんどありません。これを考慮してください：

public void Fill<T>(IList<T> list) 
{ 
    // add a bunch of items to list 
} 

"Fill"のような名前では、メソッドがリストを変更することはかなり確信で​​きます。

Answer 2

両方のメソッドで全く同じことをしていますが、どちらか一方だけが同じリストを返しています。

私の意見では、実際にあなたがやっていることに依存します。何が起きているのかについてのドキュメントが明確であることを確認してください。あなたがそのようなことをするなら、事前条件と事後条件を書いてください。

Answer 3

この場合、どちらの方法も多かれ少なかれ同じことをします。どちらが渡されたListを変更します。

目的は、このような方法で不変のリストを持っているのであれば、第二の例は、送られてきたListのコピーを作成し、その上Add操作を実行する必要があります新しいListとそれを返します。

私は私の推測では、の線に沿って何かになり、C＃のも.NETに精通していないよ。この方法で

public List<string> Foo(List<string> list) 
{ 
    List<string> newList = (List<string>)list.Clone(); 
    newList.Add("Bar"); 
    return newList; 
} 

、Fooメソッドを呼び出す方法は、新しく作成されたListが返されます渡された元のListは触れられませんでした。

これは実際には仕様またはAPIの「契約」に従いますので、Listを変更することができる場合は、最初のアプローチには問題はありません。

Answer 4

拡張メソッドの出現により、副作用をもたらすメソッドを扱うのが少し簡単になりました。たとえば、あなたの例では、それは

public static class Extensions 
{ 
    public static void AddBar(this List<string> list) 
    { 
    list.Add("Bar"); 
    } 
} 

を言うと、何かがリストに起こっていること、それが明確になり

mylist.AddBar(); 

でそれを呼び出すために、はるかに直感的になります。

コメントに記載されているように、これはリストの変更がより混乱する傾向がありますので、リストで最も役立ちます。単純なオブジェクトでは、オブジェクトを適切に修正する傾向があります。

Answer 5

実際には、パラメータとしてリストを受け取るメソッドがリストを変更するということは予期せぬことではありません。あなたはそれだけでも、リストを変更する方法を禁止するものではありません。このようなインターフェイスを使用することにより

public int GetLongest(IEnumerable<string> list) { 
    int len = 0; 
    foreach (string s in list) { 
     len = Math.Max(len, s.Length); 
    } 
    return len; 
} 

：あなただけのリストから読み込む方法をしたい場合は、あなただけの読み取りを可能にするインターフェースを使用しますたとえば、文字列配列のように、インターフェイスを実装するコレクションを使用できるので、より柔軟になります。

一部の言語では、メソッドに変更を加えることを禁止するパラメータに適用できるconstキーワードがあります。 .NETにはこのために使用できるインターフェイスと不変の文字列があるので、実際にはconstパラメータは必要ありません。