質問: new Item( $item['id'], $item['title'], $item['price'], $item['slug'] ); } } public function find($id) { if (isset($this->items[$id])) { return $this->items[$id]; } throw new \InvalidArgumentException('不正なID'); } public function findBySlug($slug) {//省略} } ``` また、あわせてお聞きしたいのですが、ItemRepositoryクラスについてもっといい書き方は無いでしょうか? コンストラクタでItemのインスタンスをセットしてるのがちょっとイケてない気がしています。 よろしくおねがいします。
回答: 現状で問題ないならいいんじゃ? しいていえば、設定ファイル等を別途設けて それを処理時などに読み込むようにすれば、 ロジックとの分離ができていいかも。