search

matgip / TastyEstate

대한민국 부동산 평점 시스템
GNU General Public License v2.0
1 stars 0 forks source link

BaseChkBox에서 index를 받는 이유 #48

Closed jkpark closed 2 years ago

jkpark commented 2 years ago

https://github.com/matgip/TastyEstate/blob/e96ed916d1c0c9ec59c7794b4f6de50212c3c3fd/frontend/src/common/BaseCheckBox.vue#L28

BaseChkBox에서 index를 받을 이유가 없음.

ReviewDialogButton.vue 에서 chkBoxes array의 index를 전달해주기 위해 저렇게 해놓은거 같은데 저러면 재사용성 떨어질듯.

수정필요

jkpark commented 2 years ago

그리고 https://github.com/matgip/TastyEstate/blob/e96ed916d1c0c9ec59c7794b4f6de50212c3c3fd/frontend/src/common/BaseCheckBox.vue#L13 에서 key를 저렇게 받는 이유는 머임?? 몰라서 물어봄

junhong91 commented 2 years ago

:key를 받는거는 아마 안쓰면 check된게 리랜더링이 안될거야 아마도? 원래 그 이유로 key주니까

jkpark commented 2 years ago

근데 item-${i} 의 의미는머임? 걍 :key=i 해도 될텐데

junhong91 commented 2 years ago

ㅇㅇ 아마 상관없을걸 레퍼런스 가지고 온거라 그런거임

junhong91 commented 2 years ago

array로 짜놨는데, 사실 "string" to object형태의 자료구조가 있으면 그거 쓰면 index필요없어질거 같은데 더 좋은방법 찾아봐야할듯

junhong91 commented 2 years ago

onClick이 됬을때 DOM element의 값으로 확인 가능하면 index를 삭제할 수 있을 듯

jkpark commented 2 years ago

여래개의 chkbox group을 가지고 있는건 ReviewDialogButton.vue 에서의 문제고, baseCheckBox 에서는 index 같은 걸 넘겨받을 이유가 없지.. plain js에서 click listener어떻게 구현했는지 찾아봐야될듯

junhong91 commented 2 years ago

plain js에서 click listener어떻게 구현했는지 찾아봐야될듯

ㅇㅇ 바로 위에서 언급한 내용인데 onchange시에 돔 element에 접속해서 data나 prop값을 보면 인덱스가 필요앖아질듯?

junhong91 commented 2 years ago

index 삭제 commit: c3737a1827940cd15c88ba6e830a65d451a1e3d0