nftug/yomlognext

読書ログをつけるアプリ

yomlognext's Issues

値オブジェクト `BookPage` の使用用途に応じた修正

TL;DR

• #1 に引き続き、 BookPage に危険なニオイを感じてしまったので。

• 単にページ数を指し示す用途と、進捗状況を指し示す用途で同じ値オブジェクトを使いまわしている。

  • 別の用途なのに同じクラスで管理しているのは、ちょっと危険なニオイがする。
  • nullableなフィールドが増えてきた段階でリファクタしておくべきだった。

Solution

• BookPage について、用途別に値オブジェクト及び関連するDTOを分けておくこと。

値オブジェクトのCQRS対応

値オブジェクトの扱いでちょっと悩んでいる。例として BookPage を取り上げる。

TL;DR

• データを永続化する際は値の検証を挟むが、単に取得されたデータから作成するときは挟まないでほしい。
• CQRSっぽく安全性を確保するなら、コマンド用とクエリ用でそれぞれ別に値オブジェクトを作成すべきではないか？

Background

• もし変なデータがDBに入ってしまったら、アプリから該当のデータを取得しようとすると例外で止まってしまう。
• コンストラクタにskipValidationというbool値を入れて検証の有無を切り替えているが、コードが複雑になってしまう。

Solution

• CQRSの原則に則って、バリデーションを挟まないQuery専用の値オブジェクト BookPageForQuery を新たに作成すべき？
• 検証が必要な際は、この値オブジェクトを継承して別クラスとして BookPageForCommand を作成すべきだろう。