if (page != null) {
return page;
} else {
throw new EntityNotFoundException("There is no page with given id");
}
- нужно писать наоборот: сначало утверждение (page==null), а потом уже return. код читается лучше
- EntityNotFoundException вынес бы во внутренний, т.к отдельного файла для него много
if (page != null) {
return page;
} else {
throw new EntityNotFoundException("There is no page with given id");
}