desire456/spring-course-3

spring-course-3's Issues

В соседнем сервисе - final. Используйте единый подход

spring-course-3/src/main/java/com/example/api/service/AuthorizationService.java

Lines 19 to 21 in 62e718a

	private UserService userService;
	private PostService postService;
	private PostRepository postRepository;

Если вы вызываете set, обязателен ли save?

spring-course-3/src/main/java/com/example/api/service/PostService.java

Line 68 in 62e718a

return mapper.fromEntity(postRepository.save(post));

Вопрос по архитектуре

Столкнулся с такой проблемой: в сервисах возвращаю, например, в методе findById дто объект. Потом я хочу в другом месте его использовать, но уже хочу ентити получить, а не дто. В связи с этим вопрос как лучше поступить при написании сервиса, чтобы таких проблем не возникало: возвращать ентити везде, а в прикладных местах мапить в дто или писать в сервисах кучу методов с разными return типами? @coursar, подскажите, пожалуйста?

В сервисах точно не должно быть транзакций?

А если не анонимус?

spring-course-3/src/main/java/com/example/api/configuration/SecurityConfiguration.java

Line 70 in 62e718a

.antMatchers(HttpMethod.GET, "/api/posts", "/api/posts/{\\d+}").hasRole("ANONYMOUS")

Чем больше вложенность кода - тем сложнее "complexity" - тем выше вероятность ошибок

spring-course-3/src/main/java/com/example/api/service/LikeService.java

Lines 28 to 39 in 62e718a

	if (likeEntity != null) {
	if (!likeEntity.getLike().equals(like)) {
	likeEntity.setLike(like);
	} else {
	return likeMapper.fromEntity(likeEntity);
	}
	} else {
	likeEntity = new LikeEntity();
	likeEntity.setLike(like);
	likeEntity.setAuthor(author);
	likeEntity.setPost(post);
	}

Если на проекте будут юзать стат.анализаторы, придётся отказываться от кучи вложенных if-else.

Строки лучше в константы/enum

spring-course-3/src/main/java/com/example/api/service/AuthorizationService.java

Line 49 in 62e718a

return authorities.contains(new SimpleGrantedAuthority("ADMIN"));

Можно уложить в один JPQL запрос

spring-course-3/src/main/java/com/example/api/service/AuthorizationService.java

Lines 24 to 25 in 62e718a

	UserEntity author = userService.findByUsername(principal.getUsername());
	return postService.existByAuthorAndId(author, postRequestDto.getId());