isMatch 코드 리뷰입니다.
- 내부에 wait 하는 루팅이 없으므로 Promise 로 감쌀 필요는 없어 보입니다.
- isMatch 는 동사의 중복으로 matches, filter 같은 이름이 일반적입니다.
- 이 함수는 item 을 검사하는 즉 필터링 하는 역할을 하는데, 필터가 제목, 작가 등등 여러가지입니다. 각각을 별도로 함수로 만들면 좋을 것 같네요. 리턴 값은 items 로 합니다.
- getDiscussionsByCreated 결과가 item 의 배열인데 item 을 객체화하고 아래처럼 코딩합니다.
items.filterTitle() // items ==> filter ==> filtered items
.filterAuthor()
.filterVote()
이상입니다.
코드리뷰 감사드립니다.
말씀하신 부분을 수정하여 반영하면 소스가 좀 더 말끔해 질 것 같네요 ^^