search

swsnu / swpp2019-team2

COSMOS
0 stars 3 forks source link

Tree CheckBox 관련 #79

Open lllY2Klll opened 4 years ago

lllY2Klll commented 4 years ago

현재 dev/budget-tree 브랜치에 search tag tree를 새로 구현해놓은 상태입니다. 이와 관련한 장점과 단점이 있는데

장점

  1. 구현 로직이 쉽다. (태그가 추가되어도 노드 정보만 바꾸면 됩니다. 현재 budget search page에 구현해놓은 checkbox tree는 search page의 tag status를 그대로 재현해놓은 것입니다.)

  2. 테스팅이 쉽다. (렌더링은 컴포넌트 하나로 하기 때문에 태그의 갯수가 늘어나도 테스트 케이스가 늘어나지 않습니다. 현재 구현된 것에 대한 커버리지도 100을 만들어놓은 상태입니다.)

  3. 이식하기 편하다. (마음만 먹으면 지금 budget search 페이지에서 쓰고있는 태그를 서치페이지에도 그대로 가져다 쓸 수 있습니다.)

  4. 선택된 노드의 정보를 스트링으로 가져올 수 있다. ( budget search 페이지의 checked node 렌더링 참고)

단점

  1. 스타일링을 새로 해야한다. (현재 전부 세로 가운데정렬 되어있고 버튼이 커스터마이즈 되어있는데 새로운 트리를 쓴다 생각하면 css수정이 필요합니다.)

  2. url규칙에 맞게 스트링 파싱을 해야할 수도 있다. (백엔드로 데이터 요청하는 url을 맞추려면 파싱 작업이 필요할지도 모릅니다. CheckBox.js를 보면 선택된 노드의 정보를 value어트리뷰트에 string으로 저장하는데 이 저장방식에 따라 url 요청 방식이 상이할 것 같아보입니다.)

  3. sonarcloud (현재 pull request를 보면 소나클라우드에서 duplicate code 에러가 나는데... 의미상 중복일 수 없는데 중복이라고 하는건 어떻게 해결해야 좋을 지 모르겠습니다.)

dev/budget-tree 클론해서 budget search page 확인해보고 피드백과 이걸 쓸지 말지 기탄없는 의견 바랍니다. (sonarcloud duplication 해결법도...)

aeri206 commented 4 years ago

https://www.npmjs.com/package/react-checkbox-tree

aeri206 commented 4 years ago

Search도 비슷하게 바뀌면 좋을 것 같습니다! 👍

추가/검토해야할것 같은 사항으로는