質問編集履歴
1
よろしくないと思っている点に追記。
title
CHANGED
|
File without changes
|
body
CHANGED
|
@@ -26,5 +26,6 @@
|
|
|
26
26
|
|
|
27
27
|
よろしくないと思っている点。
|
|
28
28
|
・Optionalを取得して、わざわざisPresent()で値のあるなしを判定している。→ nullチェックやってるのと変わらないから、Optionlの思想(?)に反している。
|
|
29
|
+
※ ググるとOptional取得して、if(optional.isParent){ ... } は論外。と書かれていることが多かったので。
|
|
29
30
|
・正常業務で、値を使いたいときいちいちOptionlにget()をかけるのが微妙。かと言って、isPresent()で判定した後、正常処理の最初に、ローカル変数に入れるのも微妙。そもそも、Optionalを取得する流れの中で、getElse()とかで取得したり、nullの場合の処理も記述できないの?
|
|
30
31
|
・仮に、Optionalを取得する流れの中で値があるなら取得する、ない場合は処理を行うのができた場合、ない場合の処理はラムダ式(?)で書くことになるが、その中でログを出したりDB操作することに違和感がある。
|