You are viewing a single comment's thread from:

RE: [소스코드] AUTOVOT - 큐레이션(보팅)

in #autovot6 years ago

isMatch 코드 리뷰입니다.

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



이상입니다.

Sort:  

코드리뷰 감사드립니다.

말씀하신 부분을 수정하여 반영하면 소스가 좀 더 말끔해 질 것 같네요 ^^

Loading...