デザインパターン

関連するデータやロジック同士が分散している状態のこと 単にデータを保持するだけの、なんら処理ロジックを持たない データクラス は低凝集です データクラスとコントローラークラスが分離しているような状態がそうらしい https://qiita.com/MinoDriven/items/5e69d9bd028aa350e2c4

ValueObjectとはデータのラッパーであり、正確には (原則的に)1個のインスタンス変数 インスタンス変数を初期化するコンストラクタ インスタンス変数を正常に操作することを保証したメソッド から構成されるクラスです。 /// <summary>税抜き金額</summary> public class AmountExcludingTax { private int _amount; public AmountExcludingTax(int amount) { // 初期値はコンストラクタで与える _amount = amount; } } // 税抜き金額1000円 var amountExcludingTax = new AmountExcludingTax(1000);

オブジェクトを「 newした時点で正しく利用可能な、即ち完全体 」となるよう適切な初期化ロジックをコンストラクタに実装する方法です。 税抜き金額のあるべき姿、要件を考えてみましょう。 0以上の整数であること となりますね。この要件を満たすもののみインスタンス変数に格納するようフィルタリングすれば良いのです。要件未達の値は例外を投げるよう実装します。 /// <summary>税抜き金額</summary> public class AmountExcludingTax { private int _amount; public AmountExcludingTax(int amount) { // コンストラクタで不正値を除外 if (!IsValid(amount)) { throw new ArgumentOutOfRangeException(); } _amount = amount; } // 税抜き金額のバリデーションを用意 private static bool IsValid(int amount) { // 正常値の要件を記述 return 0 <= amount; } } これで正常値のみインスタンス変数に格納できるようになりました。 どうも値の役割をクラスとして定義し、役割外の値や行動をもたせない。 完全に役割が保証されたクラスを定義するということが思想らしい JavaScriptでそれやるのきつくね？

/// <summary>税抜き金額</summary> public class AmountExcludingTax { private readonly int _amount; public int Value { get { return _amount; } } /// <summary>コンストラクタ</summary> /// <param name="amount">税抜き金額</param> public AmountExcludingTax(int amount) { if (!IsValid(amount)) { throw new ArgumentOutOfRangeException(); } _amount = amount; } /// <summary>税抜き金額を加算する</summary> /// <param name="amountExcludingTax">税抜き金額</param> /// <returns>税抜き金額</returns> public AmountExcludingTax Add(AmountExcludingTax amountExcludingTax) { return new AmountExcludingTax(_amount + amountExcludingTax._amount); } /// <summary>有効な税抜き金額であるかを返す</summary> /// <param name="amount">税抜き金額</param> /// <returns>有効な場合true</returns> private static bool IsValid(int amount) { return 0 <= amount; } }