プログラミングのマジックナンバーとは？3つの問題点と解消法

if (user.age >= 18) {
  allowAccess();
}

// マジックナンバーあり（読んでも意図が伝わらない）
function calcShippingFee(price) {
  if (price >= 5000) {
    return 0;
  }
  return 500;
}

// 定数化あり（意図が一目でわかる）
const FREE_SHIPPING_THRESHOLD = 5000; // 送料無料になる金額
const STANDARD_SHIPPING_FEE = 500;    // 通常送料

function calcShippingFee(price) {
  if (price >= FREE_SHIPPING_THRESHOLD) {
    return 0;
  }
  return STANDARD_SHIPPING_FEE;
}

// パターン1：ステータスコードの直書き
if (user.role === 1) { ... }        // 1って何の権限？
if (order.status === 3) { ... }     // 3って何のステータス？

// パターン2：時間・日数の直書き
setTimeout(callback, 3000);         // 3000ミリ秒？なぜ3秒？
if (daysSinceLogin > 90) { ... }    // 90日の根拠は？

// パターン3：計算式の中の直書き
const tax = price * 0.1;            // 0.1は消費税？手数料？
const salary = hours * 1500;        // 1500は時給？

// パターン4：文字数・件数制限の直書き
if (comment.length > 200) { ... }   // なぜ200文字？
const items = list.slice(0, 10);    // なぜ10件？

// 定数化後
const ROLE_ADMIN = 1;
const ORDER_STATUS_SHIPPED = 3;
const NOTIFICATION_DELAY_MS = 3000;
const SESSION_EXPIRY_DAYS = 90;
const TAX_RATE = 0.1;
const HOURLY_WAGE = 1500;
const MAX_COMMENT_LENGTH = 200;
const DEFAULT_LIST_LIMIT = 10;

// これはOK（文脈から意味が読み取れる）
for (let i = 0; i < array.length; i++) { ... }
if (result.indexOf('error') !== -1) { ... }

// これはNG（文脈から意味が読み取れない）
if (score >= 60) { allowPass(); }       // なぜ60？
if (retryCount > 3) { giveUp(); }       // なぜ3回？

{
  "rules": {
    "no-magic-numbers": [
      "warn",
      {
        "ignore": [0, 1, -1],
        "ignoreArrayIndexes": true,
        "enforceConst": true
      }
    ]
  }
}

// constants.js（定数をまとめるファイル）
export const TAX_RATE = 0.1;                  // 消費税率
export const FREE_SHIPPING_THRESHOLD = 5000;  // 送料無料のしきい値
export const SESSION_EXPIRY_DAYS = 90;        // セッション有効期限（日数）
export const MAX_COMMENT_LENGTH = 200;        // コメント最大文字数
export const ROLE_ADMIN = 1;                  // 管理者権限
export const ROLE_GENERAL = 2;                // 一般権限

function calcTotal(price, quantity) {
  const subtotal = price * quantity;
  const tax = subtotal * 0.1;
  const total = subtotal + tax;

  if (total >= 10000) {
    return total - 1000;
  }

  return total;
}

const TAX_RATE = 0.1;                    // 消費税率（10%）
const DISCOUNT_THRESHOLD = 10000;        // 割引適用の最低金額
const DISCOUNT_AMOUNT = 1000;            // 割引額

function calcTotal(price, quantity) {
  const subtotal = price * quantity;
  const tax = subtotal * TAX_RATE;
  const total = subtotal + tax;

  if (total >= DISCOUNT_THRESHOLD) {
    return total - DISCOUNT_AMOUNT;
  }

  return total;
}

function getMenuItems(user) {
  if (user.role === 1) {
    return ['ダッシュボード', 'ユーザー管理', '設定', 'ログ閲覧'];
  }

  if (user.role === 2) {
    return ['ダッシュボード', '設定'];
  }

  return ['ダッシュボード'];
}

const ROLE_ADMIN = 1;    // 管理者
const ROLE_STAFF = 2;    // スタッフ
const ROLE_GUEST = 3;    // ゲスト（デフォルト）

function getMenuItems(user) {
  if (user.role === ROLE_ADMIN) {
    return ['ダッシュボード', 'ユーザー管理', '設定', 'ログ閲覧'];
  }

  if (user.role === ROLE_STAFF) {
    return ['ダッシュボード', '設定'];
  }

  return ['ダッシュボード'];
}

function validateUserInput(name, email, age, bio) {
  const errors = [];

  if (name.length < 2 || name.length > 50) {
    errors.push('名前は2〜50文字で入力してください。');
  }

  if (age < 18 || age > 120) {
    errors.push('年齢は18〜120の範囲で入力してください。');
  }

  if (bio.length > 500) {
    errors.push('自己紹介は500文字以内で入力してください。');
  }

  return errors;
}

const NAME_MIN_LENGTH = 2;       // 名前の最小文字数
const NAME_MAX_LENGTH = 50;      // 名前の最大文字数
const AGE_MIN = 18;              // 利用可能な最低年齢
const AGE_MAX = 120;             // 利用可能な最高年齢
const BIO_MAX_LENGTH = 500;      // 自己紹介の最大文字数

function validateUserInput(name, email, age, bio) {
  const errors = [];

  if (name.length < NAME_MIN_LENGTH || name.length > NAME_MAX_LENGTH) {
    errors.push(`名前は${NAME_MIN_LENGTH}〜${NAME_MAX_LENGTH}文字で入力してください。`);
  }

  if (age < AGE_MIN || age > AGE_MAX) {
    errors.push(`年齢は${AGE_MIN}〜${AGE_MAX}の範囲で入力してください。`);
  }

  if (bio.length > BIO_MAX_LENGTH) {
    errors.push(`自己紹介は${BIO_MAX_LENGTH}文字以内で入力してください。`);
  }

  return errors;
}

// マジックストリングの例
if (user.status === 'active') { ... }       // 'active' という文字列が直書き
if (order.type === 'express') { ... }       // 'express' という文字列が直書き

const STATUS_ACTIVE = 'active';
const ORDER_TYPE_EXPRESS = 'express';

// これはマジックナンバーと変わらない（名前から意味が読み取れない）
const NUM = 5000;
const X = 0.1;

// これはOK（名前から意図が読み取れる）
const FREE_SHIPPING_THRESHOLD = 5000;
const TAX_RATE = 0.1;