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