wiki:ebiharaHackOpenPNEFrameworkRefactoring
 ここの情報は古いです。ご理解頂いた上でお取り扱いください。

OpenPNE フレームワークリファクタリング作業用Wikiページ (By ebihara <ebihara@…>)

wiki:tejima-communityhack のページ名だけ見て、ああこりゃいいやと思い作った。

OpenPNE フレームワークリファクタリング作業とは

マーチン・ファウラー著『リファクタリング』を最近読みはじめた海老原が、実際に同書に書かれたリファクタリングを OpenPNE で次々試していく作業です。

改善作業手順

以下の 1. 〜 4. の手順を繰り返します。

1. 改善対象のコードを決定する

  • 第三章「コードの不吉な匂い」を参考にしながら、改善するべきコードを決定する

2. 改善テーマを決定

  • どういった方針でリファクタリングするか(どのような変更を施すか)を決定する

3. テストを構築する

  • コードを書き換える前にテストコードを書く

4. 改善テーマに則り、リファクタリング

  • リファクタリング・カタログの修正項目を適用していく
  • 各項目を適用する毎にテストし、コミットする

改善作業ログ

20080325.001 長過ぎる openpne_forward() をシンプルにする

controller.php で定義されている openpne_forward() は非常に長く、関数内で多数の処理をおこなっているため、コードの理解が困難であるので修正する。

以下の対処を施す

  • openpne_forward() 内でひとまとめにできるコードの断片を見つけ、別の関数として定義する(メソッドの抽出(110))
Last modified 9 years ago Last modified on Mar 25, 2008, 5:38:31 AM