I have some questions about codeing for the asignment.
1.) Do you think al lot of boilerplate code affects the score, since it could be interpreted as unecessary nesting depth increasement?
2.) All this Exception handling seems to be poluting methods and constructors signatures. For example, the constructor for my hotel room DTO:
Since there is a lot of parsing from strings a lot can go wrong. Are all those exceptions an overkill?
1/ having a lot of boilerplate code is never a good thing
2/ Only the ParseException is a checked exception, all other exceptions should not occur in a well-developed application (and thus should not be caught anywhere). I decided not to parse dates and just handle them as string (which removes the need of handling/declaring the ParseException)