-
9/23 TIL | 방금 본 플레이리스트 기능 추가의 건.📝 기록/매일의 기록 2022. 9. 23. 22:39
어제저녁에 생각나서 새롭게 추가한 기능인 방금 본 플레이리스트 기능을 구현해보았다. 사실 도메인 모델 추가를 주된 목적으로 추가한 기능이었는데, 막상 작성하다 보니 History 객체 안에 담겨야 할 요소가 모호했다. 어차피 PlayList 객체의 리스트만을 갖고 있으면 되니까 굳이 이걸 따로 모델로 만들 필요가 있나? 하는 생각이 드는 것이다. 결국 고민 끝에 방문한 적이 있는 리스트만 관리하면 되는 것이니 굳이 하나의 모델을 더 생성하기보다는 HistoryService 클래스를 따로 만들어서 거기서 리스트를 추가, 필터 작업을 하게끔 작성하였다.
근데 오늘 리뷰에서 History를 도메인 모델로 추가했어야 했다는 리뷰를 주셨고 그래서 History 객체를 만들어서 수정해보는 중인데, 계속 이걸 굳이 이렇게 관리를 해야 하는 것인가 하는 의문이 계속 드는 것이다...... History 기능은 단지 방금 본 플레이리스트인지 확인하여 리스트 배열에 가지고만 있으면 되는 것이니까 이렇게까지 해야 하나 하는 생각에... 굳이 History 모델을 추가해야 하는지 아직도 이해가 가지 않아.. 코드를 짜다가도 계속 멈칫하는 상황 발생.. 다시 싹 다 지우고 테스트 코드를 다시 짜면서 역할과 책임을 다시 정리하고 이해가 가면 수정해보고 도무지 이해가 안 가면 질문 채널에 질문을 남겨보자!
일주일 간 프로젝트를 하면서 그리고 오늘 데모까지 많은 것을 알게 되고, 또 많은 감정을 느꼈는데 이건 이번 주 주간 회고에 좀 더 길게 작성해볼 생각이다. 무튼 고생 많았다. 앞으로도 더 고생하자!
'📝 기록 > 매일의 기록' 카테고리의 다른 글
9/25 TIL | CSS float, flex, grid 레이아웃 써보기. (0) 2022.09.25 9/24 TIL | Welcome Back, HTML & CSS! (1) 2022.09.24 9/22 TIL | 프로젝트 마감까지 16시간...⌛️ (1) 2022.09.22 9/21 TIL | 작업은 중요도 순! (feat. 철야 작업) (5) 2022.09.21 9/20 TIL | MVP를 위해서 사소한 것엔 흐린 눈을..🙈 (1) 2022.09.20