「PHPUnit」に関する日記(2)

またPHPUnitの話題だけれども、CRAPインデックス。
コードの複雑性を評価したもので、でかいほどヤバい(悪い意味で)。
で、これの大きいメソッドをコードカバレッジのダッシュボードから見ることができるんだけれども、実際見てみたら、触れたくないカオスなコードが…。
気合いのあるときになんとかしよう。

今更ながら、カバレッジ出すようにしてみました。
決して良くはない数字だし、PHPUnitの対象にしていないコードは根本的にカバレッジの対象にもならないけど、まあ、最初はこんなもんかな。
ほとんどロジックがなくてテストするまでもないテンプレート的なPHPファイルも多いので、うまい事分けてカバレッジ上げていきたいものです。

内部構造リニューアル終わりました！！！！
やったこととしては、以下の通り。

• メイン情報とサブ情報のリレーションの向きを逆転させる(根底をひっくり返す)
• サブ情報に向いていたリレーションを全部メイン情報に向ける(根底がひっくり返った余波)
• 使わなくなった項目の削除(DROP COLUMNのないSQLiteなので、テーブル丸ごと作り直し)
• 統一的な編集インターフェースの構築(0から構築)
• 全く動いていなかった編集画面の実装(テンプレート以外ほぼ最初から作り直し)
• 自動テスト可能なコードへの書き換え(もともと自動テストくそくらえなコードだった)

リリースして、実際にちゃんと動いているのも確認できました。
PHPUnitによる自動テストと、元の機能がつぶれていないか確かめる独自の自動結合テストもやっていたので、間違いはないはずなんですが、実際にリリースするまではやっぱりヒヤヒヤしましたね。
なんと言っても普段やっているのと比べて明らかに大きい変更でしたからね。

さて、本当のスタートはここからです。
ここしばらくやっていた内部構造リニューアルは、これまでの機能を潰さない書き換えで、編集画面をまともにした以外は、めぼしい新機能などなかったのです。
本当にやりたかったのは、言うまでもなく、新機能の実装。
見通しが悪く筋の通っていなかったコードのままでは、新機能を実装しても厄介な問題を生むだけというのは目に見えていたので、内部構造を一新したのです。

今まで弱かった、職業関連の情報とか、錬金釜や交換アイテムなど、充実させていきたいですね。

ローカル環境で作ったComposer環境をそのままサーバーにアップするのはやめて、ローカルはローカル、サーバーはサーバーで別個にComposer環境を作ることにしました。
ローカルはPHPUnitつきで、サーバーは--no-devで。

PHPUnit

今更ながら、導入してみました。
ちゃんとしたテストはまだ作っていないですが、お試しでちょっとしたサンプルコード的なものを仮に作っておきました。
サイト全体に適用するのはさすがに無理があるので、これから作る部分から順次作っていくことになりそうです。

ドラゴンクエスト10

テンの日。
あと、ァォィョッュもバトエンのクエストをクリアして、全員バトエンできるようになりました。