On 08/17/2016 01:16 PM, Daniel Fuchs wrote:
Now that you have various findByXXX methods return Optional<Level>, you
could make methods that use them more Optional-API-powered. For example,
findLevel could be written as a single expression:

Right. But do you mind if I leave that for another day?
I'm not too keen on multi-line code statements embedded
in lambdas - but I feel like we did enough restructuring
for this fix now ;-)

It's just that the functional style is actually more readable in this case. But never mind. It's just style.

Regards, Peter

Reply via email to