반응형
WIL을 따서 WID로 타이틀을 정했다. What I Did 라는 뜻이다.
이전 소스코드를 리팩토링하면서 새로 생긴 원칙들을 적용했다.
- 줄임말을 쓰지 않았다. 이전에 줄임말을 썼던 부분도 풀네임으로 바꾸었다.
- 타입 정의를 보다 명확하게 했다(일부 못한 부분도 있었음, 라이브러리가 제공하는 props의 경우).
- 클릭 시 데이터를 CRUD 하는 함수가 받는 파라미터를 아무 생각 없이 e 로 써두었었는데 이 함수가 받는 인수는 event나 element라기보다 target에 가깝다고 생각하여 target으로 명칭을 변경하였다. 보통은 event를 받아 event.target 또는 e.target으로 많이 썼는데 이 경우에는 인수에서 바로 data에 접근하기 때문에 target.data가 더 명확하게 의미가 전달될 것이라고 생각했다.
ShallowEqual 에 대해 다시 찾아보았다.
- useSelector에서 여러가지 상태값을 받아오는 경우 shallowEqual이라는 인자를 주는 이유에 대해 다시 찾아보았다. 여러가지 상태값을 받아올 때 필요한 상태값을 비교하여 컴포넌트를 적절하게 렌더링되도록 도와주는 친구라고 생각하고 있었는데 좀 더 상세하게 알아야겠다는 생각을 했다. (https://okky.kr/questions/1432062)
- 여기 있는 세번째 방법이 좋은 방법일까? (https://velog.io/@hyunn/useSelector-%EC%B5%9C%EC%A0%81%ED%99%94)
더 생각해볼 점
- 한 번에 한가지 일에만 집중하는 것이 아직 좀 어렵다. 기능을 완성한 뒤에 커밋을 하고, 또 리팩토링을 한 뒤에 커밋을 하고, 네이밍 변경 후에 다시 커밋을 했으면 좀 더 깨끗하게 커밋 관리를 할 수 있었을 것 같다.
- PR한 뒤에, 커밋은 커밋대로 기록이 남고 머지한 기록이 커밋으로 한줄 더 남는데 이거 왜 이러는지, 없앨수 있는지 알아봐야겠다.
- 오늘은 약간 흔들렸다. 정신없이 몰아치듯 일하기보다 바빠도 정확하게 일하기!
728x90
반응형
'TIL > WID: What I Did' 카테고리의 다른 글
2월 26일 WID : PR 메시지 포맷, button 규칙 지키기 (0) | 2024.02.26 |
---|---|
2월 22일, 23일 WID : 1px 잡기, 문서작업 (0) | 2024.02.23 |
2월 21일 WID : TabPanel 내용 보강, 업무적/개인적 todo list 작성 (2) | 2024.02.21 |
2월 20일 WID : Devextreme의 TabPanel을 정복하다! (0) | 2024.02.21 |
2월 16일 WID : 테스트, 테스트, 테스트, 그리고 Promise (1) | 2024.02.16 |