Skip to content

Refactoring. if-else, exception to inner class #6

@andreevym

Description

@andreevym

if (page != null) {
return page;
} else {
throw new EntityNotFoundException("There is no page with given id");
}

  1. нужно писать наоборот: сначало утверждение (page==null), а потом уже return. код читается лучше
  2. EntityNotFoundException вынес бы во внутренний, т.к отдельного файла для него много

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions