[고도화] 해시태그 검색 기능 고도화 #61
Comments
lmw7414
added a commit
that referenced
this issue
Jan 8, 2023
게시글에 달린 해시태그를 관리하기 위한 기능을 구현. 독특하게도 해시태그의 저장을 다루는 메소드가 없는데 해시태그의 저장은 현재 비즈니스 로직의 구조 상 게시글이 저장, 수정될 때만 일어나는 기능으로 연관관계 매핑을 통해 게시글 서비스에서 모두 처리가 되기 때문. * 해시태그 이름으로 해시태그 정보를 db 조회하기 * 본문에서 해시태그 파싱하기 * 게시글이 더 이상 없는 해시태그를 삭제하기 * TODO: 삭제 기능의 테스트를 강의 중에 만들지 않았다. 추가하면 좋을 듯.
lmw7414
added a commit
that referenced
this issue
Jan 8, 2023
테스트의 변화는 #53의 1801c6 에서 도메인 변경을 하면서 이미 구상하여 작업하였음 주로 게시글이 저장, 수정, 삭제될 때 해시태그가 함께 어떻게 움직여야 하는지를 묘사한다. 강의 진행하면서 도메인 변경이 얼마나 넓은 영역에 영향을 미치는지 보여주면서, 컴파일 에러를 해소하기 위해 일단 테스트 변경까지 모두 반영하게 되었다. 이런 과정을 거치면서 코드 작성 과정을 잘 구분하고 보여줄 시간이 부족하여 이런 커밋들이 발생했는데, 실전에서 이렇게 작업 내역을 나누면 커밋으로 이력을 추적하면서 볼 때 다소 불편할 것 같다. 또한 사이의 커밋이 테스트 실패하는 불완전한 상태로 진행된 것도 문제. 서비스의 변경은 테스트 변경과 함께 담길 수 있도록 하자. * TODO: `getHashtags()`의 로직은 개선 가능해보임
lmw7414
added a commit
that referenced
this issue
Jan 8, 2023
이 정보는 뷰에서 해시태그에 링크를 달 때 게시글의 검색 필터 정보로 쓰기 위해 필요하다. * TODO: `searchTypes`에서 재활용할 수 있는 데이터이므로 더 나은 방법이 있을지 고민해보자. 테스트의 변화는 #53의 1801c6 에서 도메인 변경을 하면서 이미 구상하여 작업하였음
lmw7414
added a commit
that referenced
this issue
Jan 8, 2023
* 게시글 페이지, 게시판 페이지: 해시태그에 링크를 달고, badge 스타일을 추가 * 게시글 추가 페이지: 해시태그 입력칸 삭제
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
해시태그 기능을 고도화한다.
#을 뺀 문자열을 저장할 수 있게 하기The text was updated successfully, but these errors were encountered: