Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- // メソッドの戻り値についても同じ悩みがあるにはある。
- // ただメソッドの戻り値の場合は、そもそも使い回すことが少ないし、使い回す場合でも
- // クライアントが十分コントロール可能(適切なタイミングで ToArray() 呼ぶとか)
- // なことが多いので、いつでも IEnumerable<T> でいいんじゃね感が割とある。
- public class Parent1
- {
- // クライアントから見て、列挙が何度も走ってしまう懸念がないのが良い。
- // また、コレクションが変更されるのを想定してない感が割と出ている。
- public Child[] Children { get; set; }
- }
- public class Parent2
- {
- // クライアントからすると、列挙が何度も走ってしまう懸念があり、良くない。
- // それを避けたいクライアントは ToArray() などしてから使うかもしれないが、
- // Parent1 のインスタンスを複数箇所で使い回す場合には避けようがない。
- // コレクションが変更されるのを想定してない感は割と出ている。
- public IEnumerable<Child> Children { get; set; }
- }
- public class Parent3
- {
- private readonly IList<Child> _children = new List<Child>();
- // クライアントからすると、null チェック不要なのが嬉しい。
- // しかしコレクションが変更されるのを想定している感が満載なので、
- // 例えば何かの検索メソッドの戻り値でには不適切に思える。
- // 逆に変更されるのを想定しているところではこれが良さ気。
- public IList<Child> Children
- {
- get { return _children; }
- }
- }
- public class Parent4
- {
- // コレクションが変更を想定してないことを最も主張するのはこれ。
- // しかし ReadOnlyCollection<T> なんて、BCL でほとんど見かけないけど使って良いのか。
- // Immutable な型を実装するためにはかなり使える型だが、普段の Entity クラスの
- // 開発なんかではまず不要。
- public ReadOnlyCollection<Child> Children { get; set; }
- }
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement