オブジェクト指向の考え方

Question

serviceクラスがあった際に
public Result calculate(Object obj1)
このようなメソッドがあったとします。

Resultがnullになるケースの対応方法として（もちろんOptionalを使ってもいいのですが）
calculateがexceptionを出すべきなのか。nullを返してもいいのか
呼び出し先でcheckするべきなのか。
引数のobj1に関してもobjのvalidationをcalculateメソッド内ですべきなのか
呼び出し先でそもそもcheckすべきなのか

Serviceクラスの責務などでよくわからなくなります。
なにかよくまとまっているものURLや本などもちろん解説も嬉しいです。

Javaの振る舞いは覚えたものの、この機能はここが持つべきじゃないみたいな議論でしばしばわからなくなります。

Answer 1

Resultがnullになることが業務上普通にあり得ることなのであれば、nullを返してもいいような気はしますね。
もちろんnullableなOptionalで返すべきです。
Resultがnullになることは明らかな異常事態なのであれば例外の方がいいのではないかと思います。
エラーハンドリングを強制させたいなら、Runtime系ではない例外がいいです。
でも、本当にここはケースバイケースという感じです。

validationに関してですが、ここは賛否両論ありそうですが、持論を申し上げますと
Serviceでvalidationは絶対NGです。
Serviceは業務ロジックです。入力値の検証は業務ではないので役割外ということです。
Serviceに不正な値がくればエラーが発生してプログラムは停止すべきです。

入力値の検証はプレゼンテーションロジックの役割です。
Serviceを呼ぶ前（Actionなど）のところでやってください。
ログインチェックや在庫チェックなどの業務チェックは別ですよ？
プレゼンテーションロジックとは入力値を検証したり、業務処理用に加工したり、
業務処理の結果を画面表示用に加工したりなどを担うロジックですね。

Comments

[退会済みユーザー]

横から失礼致します。ここでおっしゃられてる検証というのは、ただ値が不正か確認するだけではなく、不正な値だった場合は再入力させるまでを言う、という理解であってますか？

[root_jp]

そうです。validationです。

[root_jp]

不正といっても、例えばログイン時のパスワードが間違ってるから再入力させるとかは、業務が入ってきてますので、Service層でも問題ないですが、半角英数のみで入力されてるか？などをServiceまで持ち込まないでくださいね。ってことです。

[退会済みユーザー]

なるほど、ありがとうございます！('_'*

[anonymous001]

ありがとうございます！！
例えばserviceが複数のservoceを依存関係にもち injectionされてた際に。
Result result = serviceA.calculate(obj1,obj2)
NewResult a = serviceB.filter(result)
こんな記述があったときに obj1,obj2のvalidationやnull checkやresultのcheckをここですべきなのか
serviceAやB内ですべきなのかってどうやって判断していますか？

[root_jp]

何度も言っていますが、validationはcalculateする前に終わらしておいてください。
resultのチェックはどういったものがあるか分からないので何とも言えませんが、serviceA内で例外を投げるのであれば、ここでキャッチするしかありません。
例外ではなくnullを返すとして、resultがnullであるということを想定内の１つの結果として扱うのであれば、filter内にそのまま渡してNewResultとして何のエラーかを表してもいいと思います。

しかし、filterするまでもない、NewResult化するまでもないとか言うレベルのことなのであれば
ここでif切ってリターンして、filter実行前に処理すべきじゃないでしょうか。

Answer 2

場合によってやり方を変えます。
場合によっては例外を出すメソッドと出さないメソッドの二種類を用意します。