Feature/hashtag 01 #9

sobaman · 2025-01-31T08:59:13Z

📌 과제 설명

post 의 content 로 부터 해시태그를 추출하여 등록하는 기능을 구현

👩‍💻 요구 사항과 구현 내용

해시태그 추출을 담당하는 클래스는 다른 곳에서도 필요할 수 있으니 별도의 클래스로 구분
해시태그 엔티티에 대한 유효성 검증 로직 구현
- 해시태그는 글자수는 10자를 초과할 수 없다
- 해시태그는� 영어,숫자,한글,_ 만 허용된다
post 서비스에서 등록 시점에 content 를 가져와서 해시태그 추출과 해시태그 생성을 진행
만약 해시태그가 이미 존재하면 저장하지 않고 가져오기만 실행

✅ 피드백 반영사항

✅ PR 포인트 & 궁금한 점

close #8

- 해시태그의 길이는 10글자 이하 - 영문,숫자,한글,_ 만 가능

- 다른 곳에서도 사용할 수 있는 기능이기에 별도의 클래스로 구현

- 해시태그가 DB에 존재하지 않을때만 생성 - 해시태그 content 로 부터 단건 조회하는 로직 생성

ChoiHyunSan

전체적으로 잘 작성해주셔서 이번 프로젝트 진행에 있어서 큰 도움이 된 것 같습니다!

특히나 에러 처리하는 부분들을 세분화해서 나타내기 편리하다는 점이 좋아 보였습니다:)
개인적으로 궁금한 부분들은 따로 리뷰를 남겨두었습니다

고생하셨습니닫!!

ChoiHyunSan · 2025-02-01T04:59:26Z

backend/src/main/java/com/example/backend/entity/HashtagEntity.java

+	public HashtagEntity(String content) {
+		validateContent(content);
+		this.content = content;
+	}
+
+	private void validateContent(String content) {
+
+		if (content == null || content.isBlank()) {
+			throw new HashtagException(HashtagErrorCode.EMPTY_CONTENT);
+		}
+		if (content.length() > 10) {
+			throw new HashtagException(HashtagErrorCode.TOO_LONG_HASHTAG_CONTENT);
+		}
+		if (!HASHTAG_PATTERN.matcher(content).matches()) {
+			throw new HashtagException(HashtagErrorCode.INVALID_HASHTAG_CONTENT);
+		}
+
+	}


HashtagEntity가 생성자를 통해 validateContent를 처리하는 것 같은데, 혹시 빌더 패턴으로 처리할 때에도 validateContent 메서드가 호출되고 있나요?? (제가 제대로 못 봤을 수도 있습니다... ㅜ)

HashtagEntity가 생성자를 통해 validateContent를 처리하는 것 같은데, 혹시 빌더 패턴으로 처리할 때에도 validateContent 메서드가 호출되고 있나요?? (제가 제대로 못 봤을 수도 있습니다... ㅜ)

아 맞네요 이부분은 수정하겠습니다 감사합니다

ChoiHyunSan · 2025-02-01T05:04:20Z

backend/src/main/java/com/example/backend/content/hashtag/service/PostHashtagService.java

+	public List<PostHashtagEntity> create(PostEntity post, Set<String> contents) {
+		List<HashtagEntity> hashtags = contents.stream()
+			.map(hashtagService::createIfNotExists)
+			.collect(Collectors.toList());
+
+		List<PostHashtagEntity> postHashtags = hashtags.stream()
+			.map(hashtag ->
+				PostHashtagEntity.builder()
+					.post(post)
+					.hashtag(hashtag)
+					.build()
+			).collect(Collectors.toList());
+
+		return postHashtagRepository.saveAll(postHashtags);
+
+	}


해시태그를 생성할 contents를 extract해서 create 메서드로 들고 와서, 각각 스트림을 통해 존재하지 않는 경우 생성하고 PostHashtagEntity를 지정해주고 있는데, 해시태그 생성 과정에서 validate exception과 같은 것이 발생하면 어떤 식으로 처리가 되는지 궁금합니다!

저는 해시태그를 생성하는 것은 게시글을 추가하면서 PostService 같은 곳에서 아마 Transactional 에노테이션을 설정할 것이기 때문에 설정하지 않으신 것으로 예상하고 있습니다 :)

해시태그를 생성할 contents를 extract해서 create 메서드로 들고 와서, 각각 스트림을 통해 존재하지 않는 경우 생성하고 PostHashtagEntity를 지정해주고 있는데, 해시태그 생성 과정에서 validate exception과 같은 것이 발생하면 어떤 식으로 처리가 되는지 궁금합니다!

저는 해시태그를 생성하는 것은 게시글을 추가하면서 PostService 같은 곳에서 아마 Transactional 에노테이션을 설정할 것이기 때문에 설정하지 않으신 것으로 예상하고 있습니다 :)

맞습니다 postservice 에서 하나의 트랜잭션으로 묶여 실행될거라 해당부분에선 일단 걸지 않았습니다
hashtag 에서 검증예외가 발생하면 에러핸들러가 잡아서 던저주는 플로우를 생각하고 있었습니다
추후에 에러핸들러에는 따로 hashtagerror를 추가해주려고 합니다

Metronon

전체적으로 구조가 잘 짜여 있어서 배워가는게 많았습니다.
유효성 검증에 대한 테스트 케이스만 몇 가지 더 해보면 더욱 견고해질 것 같습니다!

Metronon · 2025-02-01T09:43:30Z

backend/src/main/java/com/example/backend/content/hashtag/service/HashtagService.java

+			.orElseThrow(() -> new HashtagException(HashtagErrorCode.NOT_FOUND));
+	}
+
+}


게시글 작성시와 해시태그 조회에서 이런식의 구성도 가능하군요.
배워갑니다 :)

Metronon · 2025-02-01T09:49:16Z

backend/src/main/java/com/example/backend/content/hashtag/exception/HashtagErrorCode.java

+public enum HashtagErrorCode implements ErrorCodeIfs {
+
+	INVALID_HASHTAG_CONTENT(HttpStatus.BAD_REQUEST, 400, "해시태그 특수문자를 포함할 수 없습니다."),
+	EMPTY_CONTENT(HttpStatus.BAD_REQUEST, 400, "해시태크 내용이 없습니다."),


오타있어용!

오타있어용!

감사합니다!

sobaman added 5 commits January 31, 2025 16:57

feat: hashtag 에러코드 및 exception 초기 구현

3ce3de3

feat: hashtag 엔티티에 유효성 검사 로직 추가

0782c6d

- 해시태그의 길이는 10글자 이하 - 영문,숫자,한글,_ 만 가능

feat: post content 에서 해시태그 추출하는 기능 구현

85b26bb

- 다른 곳에서도 사용할 수 있는 기능이기에 별도의 클래스로 구현

feat: hashtag 등록 기능 구현

aae5219

- 해시태그가 DB에 존재하지 않을때만 생성 - 해시태그 content 로 부터 단건 조회하는 로직 생성

test: 해시태그 등록 테스트 구현

b23d0b1

sobaman requested review from ChoiHyunSan, k-haechan and Metronon January 31, 2025 08:59

sobaman self-assigned this Jan 31, 2025

sobaman requested a review from joonaeng January 31, 2025 08:59

ChoiHyunSan reviewed Feb 1, 2025

View reviewed changes

Metronon reviewed Feb 1, 2025

View reviewed changes

sobaman added 3 commits February 3, 2025 09:33

refactor: 클래스말고 생성자에 빌더 적용으로 수정

33e600f

refactor: 에러코드 오타 수정

084a36a

refactor: 주석 추가

56ddd62

sobaman merged commit 5fa3bc1 into main Feb 3, 2025

Metronon deleted the feature/hashtag-01 branch February 5, 2025 01:45

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/hashtag 01 #9

Feature/hashtag 01 #9

sobaman commented Jan 31, 2025 •

edited by Metronon

Loading

ChoiHyunSan left a comment

ChoiHyunSan Feb 1, 2025

sobaman Feb 3, 2025 •

edited

Loading

ChoiHyunSan Feb 1, 2025

sobaman Feb 3, 2025

Metronon left a comment

Metronon Feb 1, 2025

Metronon Feb 1, 2025

sobaman Feb 3, 2025

Feature/hashtag 01 #9

Feature/hashtag 01 #9

Conversation

sobaman commented Jan 31, 2025 • edited by Metronon Loading

📌 과제 설명

👩‍💻 요구 사항과 구현 내용

✅ 피드백 반영사항

✅ PR 포인트 & 궁금한 점

ChoiHyunSan left a comment

Choose a reason for hiding this comment

ChoiHyunSan Feb 1, 2025

Choose a reason for hiding this comment

sobaman Feb 3, 2025 • edited Loading

Choose a reason for hiding this comment

ChoiHyunSan Feb 1, 2025

Choose a reason for hiding this comment

sobaman Feb 3, 2025

Choose a reason for hiding this comment

Metronon left a comment

Choose a reason for hiding this comment

Metronon Feb 1, 2025

Choose a reason for hiding this comment

Metronon Feb 1, 2025

Choose a reason for hiding this comment

sobaman Feb 3, 2025

Choose a reason for hiding this comment

sobaman commented Jan 31, 2025 •

edited by Metronon

Loading

sobaman Feb 3, 2025 •

edited

Loading