フレームワークに頼らず素のPHPでオブジェクト指向を意識しながらコードを書いてみる（２）

2019/01/042020/05/13

こちらの記事の続きです。

フレームワークに頼らず素のPHPでオブジェクト指向を…（１）

DBにつなげっ

収集したFeedデータはDBへの保存を想定しております。

が、SQLite かもしれないし、MySqlかもしれないし、MySqlでもWordPress用テーブルのどこかかもしれないし・・。

ということで、Interface と Abstract Class を作ってOOP的にシュッとしたヤツを・・と思ったものの、この辺りで、自分のデザインパターンに関する知識のなさが露呈してきます。

教科書に書かれてるデザインパターンとやらをきちんと使った記憶が無いんですよね。

SingletonとかFacade辺りはフムフムと学んでみるものの、フレームワークなしでスクラッチ設計する、なんてプロジェクトに関わったことないので、うっすらした知識だけで終わってます。

とは言え、手段に振り回されて疲れるのも嫌なので（むかーしEJBで疲弊してた同僚思い出した・・）、目的を達成しつつ、データの保存先が変わっても対応できる、ぐらいにしておけば良いかな・・ぐらいで進めます。

まずは ConcreteClass として

StorageMySql

クラスを作って、DB接続のテストをしてみます。

（もしかして、データ取得部分はスタブにしておいて、Viewレイヤーからテスト＋実装する、という手順の方が良いのかもしれないけれど。）

TDDを意識しつつテスト作成。

PHPUnit を使いましょ、ということで composer.json に以下を追加。

"require-dev": {
    "phpunit/phpunit": "^7"
}

AbstractStorageクラスを継承した、

StorageMySql.php

を作成し、テストロジック

StorageMySqlTest.php

を作成して、とりあえず connect できるところまでテスト。

それにしても PSR に従って AbstractStorage というクラス名にしたんだけれど、クラス名に Abstractって単語が明確に含まれている抽象クラスに対して AbstractStorage::getInstance() って呼ぶのは、なんだか違和感があるんだけれど考えすぎだろうか。

抽象クラスのクラス名を単に Storage にして、 Storage::getInstance() ってしたくなるのだけれど・・。

なんだか気持ち悪かったので、Storage クラスを別に作って、呼び出すときには Storage:getInstance() で呼ぶようにしてみました。

Storage::getInstance() 内では、Config で指定されたDBタイプを判断して、それ用のClass(StorageMySqlなど)を生成して返す、なんていう風にしてみました。

が、このためにわざわざ1クラス準備するのもずいぶん冗長な気がするなぁ・・。

どうするか悩んだのだけれど、結局元に戻すことに。

「しばらく時間が経って、すっかり記憶をなくしているアンポンタンな自分（もしくは経緯を知らない、自分ではない誰か）が見た時に「すんなり理解しやすいかどうか？」を考えると、AbstractStorage::getInstance() で呼び出すようにした方が、分かり易いだろうな、という判断です。

最終的にはWordPressのプラグインにしたいので、データはMySqlに・・と思っていたんですが、WordPressのDBを汚染してしまう、というのはちょっと気が引けます。（自分のPlugin専用のTableを追加するのは気持ち悪いし、かといってWP_OPTIONSにシリアライズして大量のデータ突っ込むのもイヤですし・・）

考えてみれば、RSSフィードから取得したデータをキャッシュしておきたいだけなので、SQLite使えばいいや、ということで、もう一つ SQLite 用の ConcreteClass を追加しました。

StorageSQLiteTest.php

connect() のロジックだけ書き換えればＯＫ、ということで TemplateMethod パターンがうまい具合に使えているような気がします。

この時点では、Storageクラスに load() や save() みたいな読み書きメソッドが必要だろうなー、ぐらいにしか考えていませんでした。

（このあたり、結局あとで大幅に書き換えることになります。）

テストファーストを意識しつつ

以前、CodeRetriet というイベントに参加した際に、「細かくテストと実装を繰り返していく」という、TDDのテンポみたいなものを体験しました。

今回も、

まずはテストメソッドを実装し、
失敗させ、
ロジックをコーディングし、
テストして、
失敗したら直して、
またテストして
success でイヤッホー！

を繰り返していきます。

例えば、 Channel クラスを作るときは、こんなコードを書いてすぐテスト、とね。

class ChannelTest extends TestCase
{
    public function testCreateChannel()
    {
        $channel_1 = new Channel('https://example.com/rss/feed');
        $this->assertEquals($channel_1->getSubscribeUrl(), 'https://example.com/rss/feed');
        // title is null.
        $this->assertNull($channel_1->getTitle());
    }
}