코드 품질 개선 기법: 세트 할인

이 게시물은 상태 업데이트 시점과 값의 조합을 제한하는 인터페이스 설계의 중요성을 다룹니다.

기존 SampledDataRecorder 문제점

• isActive, minImportanceToRecord, dataCountPerSampling 세 속성이 독립적으로 변경되어 혼란과 버그 발생
• currentDataCount 재설정이 일관되지 않음
• 비동기 업데이트 시 경쟁 상태 위험 존재

개선 방안: SamplingPolicy 클래스 도입

• minImportanceToRecord와 dataCountPerSampling을 하나의 클래스에 묶어 함께 업데이트
• isActive는 null로 표현하여 활성 상태 관리
• currentDataCount 재설정 조건 명확화 및 스레드 안전성 확보 용이

결론

상태 변경의 일관성 유지와 인터페이스 제한을 통해 코드 품질과 유지보수성을 높일 수 있음을 보여줍니다.