連番付きフィールドこの広告は、90日以上更新していないブログに表示しています。リファクタリングあまりの糞コードに遭遇し、怒りのリファクタリングを行った。久しぶりにブログを書き始めた。今日したのは、言語はC#で、フィールドに連番の付いたフィールドがあり、そのどれかに値が入っていることを調べる処理のリファクタリング。モデルのフィールドの構造は諸事情により、変更不可とする。メソッドなどは追加してOK。}}このモデルに対し、別の処理でこのどれかに値が入ってくる。どのValueNに値が入っているか、Nの数値を取得したい。}}あぁ!もう!!!このIF文乱舞で嫌気が差さない場合は自分を疑ったほうが良い。私が修正した結果がこちらAfterまず、ModelにValue達をDictionaryで返すメソッドを追加した。モデルの構造は諸事情により、変更不可とする。}}このメソッドを利用して、BudinessLogicもこう修正した。}}だいぶマシになった。ついでにこの後、メソッド名もGetValueIndexでは意味が違っているので、GetMaxValueIndexに変更した。リファクタリングはテストコードを書いて、現状のメソッド仕様を把握・テスト出来る状態にしてから取り掛かったほうが良いね。今日じゃないけど、最近やったリファクタリング。PrimeFacesを研究しよう!...2012-11-07List.jsの使い方List.jsなるJavascriptライブラリがスゴイ!みたいな記事を見て...2011-08-31KXml2を使うXStreamは内部でKXml2を利用していることは既に書いた。7参加グループプログラミングはてなブログをはじめよう!clash_m45さんは、はてなブログを使っています。ブログを報告する引用をストックしましたストック一覧を見る閉じる引用するにはまずログインしてくださいログイン閉じる引用をストックできませんでした。再度お試しください閉じる限定公開記事のため引用できません。