メンテ不能の強者、引数リモコンパターン - jfluteの日記

Let's read! さあ、いきなり、コードを見てみてください。 public void index() { // what do you select? ... = logic.selectNandemoMember(null, "S", null , false, true, false, CDef.MemberStatus.Formalized , true, false, true, true); } Action, Controller, Service相当のクラスが、そのLogicのメソッドを呼んでいると思ってください。 こういうメソッド見たことありませんか？読みやすいですか？ さて、Logic側はこうなっています。(省略してますけど、もっとデカくなります) public List<Member> selectNandemoMember(Integer memberId ,

[Haaaa_N]

わかる,独自のO/Rマッパーは本当に厄介,chromeはてなブックマーク拡張とか手に負えなくてforkを諦めた

[akabekobeko]

状態依存を避けるためフィールドより引数へ依存するように修正してゆくときメソッドの責任分担が不十分だとなりがち。けっこうやりがち。Linter で引数の最大を 4 ぐらいに制限しておくと危険な兆候に気づけてよい。

[rryu]

処理があみだくじみたいになってて、分岐を制御する値が引数として来るやつあるある。引数のパターンが分らないとあみだくじの経路が決まらないので、関数だけを見てもどうにもならないつらさ。

[Yagokoro]

たまによく見るアレだな。

[wwolf]

芸術的なコードと呼んでる奴だ

[kamei_rio]

フィールドより引数、わかるわ……orz

[umai_bow]

うんざりするけど、ちゃんとエラー吐くし、ドキュメントがしっかりしてれば実はそこまで嫌いじゃない。関数シグネチャって強力な機能だよ。

[katsush]

sql

[n314]

管理画面からの場合はisAdminを付けて特別な処理をしよう => 管理画面専用のフラグが増えていく。とかやりがち。

[mactkg]

ありがち

[tyru]

うわあ、あるある、こういうコード。なぜこういうコードが生まれてしまうかについても書かれてある。