Skip to content

fix: extract magic number to named constant - #24

Merged
GoBeromsu merged 1 commit into
masterfrom
fix/extract-magic-number-reliability
Dec 28, 2025
Merged

fix: extract magic number to named constant#24
GoBeromsu merged 1 commit into
masterfrom
fix/extract-magic-number-reliability

Conversation

@GoBeromsu

Copy link
Copy Markdown
Owner

Summary

Magic Number를 명명된 상수로 추출

Magic Number란?

코드에 직접 박힌 숫자로, 그 의미를 알 수 없는 값

// Bad - 0.2가 무엇을 의미하는지 알 수 없음
if (apiResponse.reliability > 0.2) {

// Good - 상수명이 의미를 설명함
if (apiResponse.reliability > COMMON_CONSTANTS.MIN_RELIABILITY_THRESHOLD) {

왜 문제인가?

  1. 가독성: 0.2만 보면 "이게 뭐지?" 생각해야 함
  2. 유지보수: 나중에 임계값을 바꾸려면 코드 전체에서 0.2를 찾아야 함
  3. 일관성: 같은 값이 여러 곳에 있으면 서로 다르게 변경될 위험

변경 사항

api/constants.ts

export const COMMON_CONSTANTS = {
    // ...
    MIN_RELIABILITY_THRESHOLD: 0.2,  // ← 추가
};

main.ts

// Before
if (apiResponse && apiResponse.reliability > 0.2) {

// After
if (apiResponse && apiResponse.reliability > COMMON_CONSTANTS.MIN_RELIABILITY_THRESHOLD) {

이점

  • 코드를 읽으면 "최소 신뢰도 임계값"임을 바로 알 수 있음
  • 임계값 변경 시 한 곳만 수정하면 됨
  • IDE에서 상수를 검색하면 사용처를 모두 찾을 수 있음

Test Results

  • Build: ✅ Pass

Replace hardcoded `0.2` with `COMMON_CONSTANTS.MIN_RELIABILITY_THRESHOLD`
to improve code readability and maintainability.
@GoBeromsu
GoBeromsu merged commit aa94a55 into master Dec 28, 2025
1 check passed
@GoBeromsu
GoBeromsu deleted the fix/extract-magic-number-reliability branch April 11, 2026 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant