オブジェクト指向システムの設計 172 [無断転載禁止]©2ch.net

uy ◆e6.oHu1j.o · 2016/07/09(土) 00:35:13.95

前スレ
オブジェクト指向システムの設計 171 [無断転載禁止]©2ch.net
http://echo.2ch.net/test/read.cgi/tech/1465636703/

**デフォルトの名無しさん** · 2017/07/18(火) 16:08:09.93

>>557
> 『「発注」すると、「キャンセルすると言う操作」を無効にする』ってロジックが居て初めて、注文はキャンセルを無効と自分で判断できる、では無いか？
ごめん、書き間違えたよ
正しくは、「発送されたらもうキャンセルできない」

あと、無効にする処理をどこに実装するかの話はしていないよ
ルールの実装をどこにするかの話

class Order {
　　public bool isCancelable() {}
　　public void cancel() {}
}

client:
order = new Order();
if (order.isCancelable()) {
　　order.cancel();
}
というコードの場合、以下の点がよろしくない
・cancelの正しい手続きを呼び出し側が知っておく必要がある
・その手続きに変更があった場合、Order::cancel()を呼び出している箇所全部を修正しなければならない
・誤った呼び出しをしてしまうと、データに不整合が発生する

class Order {
　　private bool isCancelable() {}
　　public void cancel() {
}
}

**デフォルトの名無しさん** · 2017/07/18(火) 16:11:30.47

途中で書き込んでしまった

それに対して、オブジェクト自身がルールを持っていれば
class Order {
　　private bool isCancelable() {}
　　public void cancel() {
　　　　if (!isCancelable()) {
　　　　　　// エラー処理（例えば例外をthrow）
　　　　}
　　　　// 実際のキャンセル処理
　　}
}

client:
order = new Order();
try {
　　order.cancel();
} catch () {
}

クライアントは、
・cancelの正しい手続きを知る必要はなく、単に呼び出すだけ
・その手続きに変更があっても、呼び出し側は変更する必要がない
・誤った呼び出しもない
というメリットがある

**デフォルトの名無しさん** · 2017/07/18(火) 16:14:53.99

一方、権限としての呼び出し可/不可の話はこれとは別
なぜなら、cancelの呼び出し権限があるかどうかは、Orderオブジェクト自身は知りようがないから

**デフォルトの名無しさん** · 2017/07/18(火) 16:18:09.27

もちろん、「自分の注文のみキャンセルできる」という単純な権限チェックなら、Orderモデルの範囲内で
チェックが可能なので、そう実装してもいい

class Order {
　　private bool isCancelable() {}
private string orderUserID() {}
　　public void cancel(string cancelUserID) {
　　　　if (!isCancelable()) {
　　　　　　// エラー処理（例えば例外をthrow）
　　　　}
　　　　if (orderUserID() != cancelUserID) {
　　　　　　// エラー処理（例えば例外をthrow）
　　　　}
　　　　// 実際のキャンセル処理
　　}
}

まあ大抵はロールによる権限管理とかのケースが多いだろうから、その場合はOrder自身は権限チェックできない