search

2017-Capstone-2nd-Viewreful-Team-12 / Master

1 stars 0 forks source link

이슈별 Recommend 코드에 대한 index 과정의 리팩토링 #4

Open skmmarine opened 6 years ago

skmmarine commented 6 years ago

nodeServer/routes/index.js

이슈 타입 배열화에 따른 1) 반복, 분기문 작성으로 기존에 작성된 불필요한 반복,분기문의 제거 2) 새로운 이슈 타입에 대한 확장이 용이해지도록 함

skmmarine commented 6 years ago

기존 이슈 타입을 배열로 정리, 이슈카운트를 위한 별도 배열 생성 이슈 타입의 각 이름은 result에서 가져오는 이름을 기준으로 설정

*실행 환경 미구축으로 인한 검증 미실시, 기존 코드 미삭제

skmmarine commented 6 years ago

1) Issue 및 recommendID 객체에 대한 정보은닉 필요 2) recommendIDArray 의 경우 여전히 배열 순서와 이슈타입을 묵시적으로 연관지어 사용하는데 Issue 객체가 recommendID를 인자로 가지는 것이 어색하여 하나의 객체로 표현할 수 없으며 다른 객체로 정의했을 때 어떻게 recommendIDArray의 각 인자가 이슈별 recommend 인지 표현할 수 있을지 생각해볼 것. 3)수정 코드에 대한 정상동작 확인 필요