Skip to content
This repository was archived by the owner on Jan 13, 2023. It is now read-only.

resolved home work #4

Closed
wants to merge 1 commit into from
Closed

resolved home work #4

wants to merge 1 commit into from

Conversation

defaust
Copy link

@defaust defaust commented Aug 28, 2017

Тарас привет.

Я вроде решил домашку, yо мне не нравится, то что я не приминил Optional.
Можешь на одном из методов (findOne() or findByEmail()) показать как его грамотно приминить в function. Я по этой причине не делал try{}catch{}final{}.

@tboychuk
Copy link
Collaborator

@defaust, hi.

  1. Please use English for comments
  2. Optional is often used for optional dependencies in the domain model. Data access layer methods could also be wrapped with Optional, but it depends more on your design. Your method findOne() could either throw an exception. It's all up to you

@tboychuk tboychuk closed this Jul 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants