0210
instanceId → searchStates<Record<string, string>> 으로 복잡하게 관리하는 부분이 추가된 지점에 대해 구두리뷰하면서 hook이 한 컴포넌트에 여러번 호출되어야할 때 인스턴스별로 구분짓기 위한 설계라고 설명받았는데, 불필요한 복잡도를 높이는 코드라는 점에 대해 경계하는 태도가 부족했던 것 같다
의도상 잘 돌아가면 장땡이 아니라 복잡도에 대한 근거가 충분한지도 고려하여 리뷰해야한다
Files
instanceId → searchStates<Record<string, string>> 으로 복잡하게 관리하는 부분이 추가된 지점에 대해 구두리뷰하면서 hook이 한 컴포넌트에 여러번 호출되어야할 때 인스턴스별로 구분짓기 위한 설계라고 설명받았는데, 불필요한 복잡도를 높이는 코드라는 점에 대해 경계하는 태도가 부족했던 것 같다
의도상 잘 돌아가면 장땡이 아니라 복잡도에 대한 근거가 충분한지도 고려하여 리뷰해야한다