[고도화] 해시태그 검색 기능 고도화 #65
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
게시글에 달린 해시태그를 관리하기 위한 기능을 구현. 독특하게도 해시태그의 저장을 다루는 메소드가 없는데 해시태그의 저장은 현재 비즈니스 로직의 구조 상 게시글이 저장, 수정될 때만 일어나는 기능으로 연관관계 매핑을 통해 게시글 서비스에서 모두 처리가 되기 때문. * 해시태그 이름으로 해시태그 정보를 db 조회하기 * 본문에서 해시태그 파싱하기 * 게시글이 더 이상 없는 해시태그를 삭제하기 * TODO: 삭제 기능의 테스트를 강의 중에 만들지 않았다. 추가하면 좋을 듯.
테스트의 변화는 #53의 1801c6 에서 도메인 변경을 하면서 이미 구상하여 작업하였음 주로 게시글이 저장, 수정, 삭제될 때 해시태그가 함께 어떻게 움직여야 하는지를 묘사한다. 강의 진행하면서 도메인 변경이 얼마나 넓은 영역에 영향을 미치는지 보여주면서, 컴파일 에러를 해소하기 위해 일단 테스트 변경까지 모두 반영하게 되었다. 이런 과정을 거치면서 코드 작성 과정을 잘 구분하고 보여줄 시간이 부족하여 이런 커밋들이 발생했는데, 실전에서 이렇게 작업 내역을 나누면 커밋으로 이력을 추적하면서 볼 때 다소 불편할 것 같다. 또한 사이의 커밋이 테스트 실패하는 불완전한 상태로 진행된 것도 문제. 서비스의 변경은 테스트 변경과 함께 담길 수 있도록 하자. * TODO: `getHashtags()`의 로직은 개선 가능해보임
이 정보는 뷰에서 해시태그에 링크를 달 때 게시글의 검색 필터 정보로 쓰기 위해 필요하다. * TODO: `searchTypes`에서 재활용할 수 있는 데이터이므로 더 나은 방법이 있을지 고민해보자. 테스트의 변화는 #53의 1801c6 에서 도메인 변경을 하면서 이미 구상하여 작업하였음
* 게시글 페이지, 게시판 페이지: 해시태그에 링크를 달고, badge 스타일을 추가 * 게시글 추가 페이지: 해시태그 입력칸 삭제
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
해시태그 기능을 고도화하는 pr
고도화된 기능 디테일은 #61 참고
This closes #61