引数の型を何でも List にしちゃう奴にそろそろ一言いっておくか - Qiita

この記事は C# その２ Advent Calendar 2018 の第一日の記事である。 はじめに この記事では、主にエンタープライズアプリケーション（SI、企業向けの業務システムやパッケージ製品）の開発に於いて、新規開発ではなく修正や拡張を行うようなシーンを想定して、無駄な工数をなるべく削減すべく自分なりに考えて実践しているベストプラクティスを書いている。 新規開発の場合でも、将来の拡張や修正が見込まれるはずなので、考慮すべき事は同じだ。 競技プログラミングや、組み込み開発の場合でも基本的な考え方は適用可能だが、メモリ効率やパフォーマンスを考慮する必要もあるので、あえて配列を使ったり、逸脱するようなケースもあるだろう。 対象とする読者層は、C#プログラミング歴1年以上、SIer やユーザー企業に所属（もしくは常駐）し、特に複数人チームでの開発に携わる若手プログラマ、初級から中級へのステ

[nishinonanase]

“Unnecessury”

[perl-o-pal]

そもそも副作用のあるコードは極力書かない方が良くて。

[SnowSwallow]

「省略(500行ぐらいある)」に強烈な怒りを感じさせる。こういうメソッド書く人はだいたいアンチパターンを多重掛けするという暗黙の怒りだ。そこにも一言いいたいんだろうけど、主題と外れるから言及しないんだよね…

[megumin1]

mutable,immutableがいい加減なC#で、userlandで「Listだけ」この辺りいくら頑張ったことろで焼け石に水なんですよね。所詮局所解でありファイクなimmutableでいくらでも抜け道がありますね。この点、Rust等は圧倒的に優れている。

[dexia2]

バックエンドでドメイン情報を扱う時は独自コレクションを作って、その中にロジックを書くこともそれなりにありますね。Listがあちらこちらにある場合はドメイン情報が散らばっているのではというのが気になります。

[amagitakayosi]

そんな〜　"ReadOnlyCollection<T> は IList<T> を実装しており、 Add や Remove といった要素の追加削除のメソッドが定義されているが、それらのメソッドを呼び出した時点で例外が発生する、という仕様"

[mattn]

SIer 関係ない

[kiyo_hiko]

“コンスタントラクタ” → 何かつよそう

[kote2kote2]

こいつと仕事したくないと思われたらどんなに美しいコードを書こうが負け

[gabill]

ゼパイルさん「人間の心理ってのは本当に面白い。Collectionの追加や削除ばかり警戒していると、IReadOnlyCollectionというだけで優秀な目利きがコロッと騙される」

[knstkny]

]

[zetamatta]

IReadOnlyCollection なんていうのがあったのか、知見。ただな、わし SortedDictionary(Of List(Of T)) なんてのを使うのが一番多くてな…

[overkitten]

500行とかは、lintでcomplexityの制約入れたらいいやん

[gnety]

そもそも引数で渡したオブジェクトの内容を変更してはならない

[aya_momo]

C#よく知らないけど、constが無い代わりにサブクラスが用意されているってこと？

[AmaiSaeta]

語弊を招きそうで且つ過激だが、そもそもJavaやC#みたいなオブジェクトが参照渡しなクソ言語を使うなと言うのが、俺の意見。

[indication]

MergeAndUniqueだったら、linqの出番だね(無関係)。paramsを使おうって話じゃなかった

[gfx]

タイトルから全然中身を予測できなかったけど、immutable intercaceを使おうって話だった。せやな。C# に限らないが。

[Ioannes]

なんか色々それ以前だろって話な気がする。

[knjname]

Clojure使えば全部immutableやで

[uxlayman]

わかりみ。この手の誤解をなくすために言語仕様を活用する話もっと盛り上がって欲しい

[deamu]

【「IReadonlyCollection って初めて聞いたな」って人もいるかもしれない。 (略) 要は、リードオンリーなコレクションってことだ】　←何にも情報が増えてないこの下りで爆笑してしまった

[yoiIT]

“メソッドの実装を読まないと分からないような書き方は悪だ。極悪だ。”

[degucho]

リストを直接書き換えるようなメソッドって書くかな？そう言う場合は戻りで新しいリストを返すよね

[mrpotas]

Unity用社内共通ライブラリのせいでC#のバージョン上げられない！、と非効率な戦いを強いられるクソ現場には早すぎる話だった

[hirose504]

“メソッドの実装を読まないと分からないような書き方は悪だ。極悪だ。我々の貴重な工数を浪費し、残業時間を増大させ、体力と精神力を消耗してしまって、その果てに幸福が待っているはずなど皆無なのである。”

[princo_matsuri]

"(3000行ぐらいある)"

[uunfo]

とりあえず見た瞬間typoを修正したくなった

[mohno]

「(500行ぐらいある)」はドッキリポイントではなかったか。「.NET 4.5 で IReadOnly 系が導入されたことを知らない」^_^;;;

[ene0kcal]

「というかそのような開発現場はもっと根本的なところに課題を抱えていそうだが。」コレアル！！メンバー間でラポールを築けているのは本当に強い。まぁ人間だもの。

[w1234567]

オブジェクト指向的にList<XXX>じゃなくてXXXsクラスを独自に作って扱えって話かと思ったら全然違った。てか普通にメソッドのコメント読めば分かるだろ、普通なら…