حارس الجودة

function processOrder(order) {
  if (order) {
    if (order.items) {
      if (order.items.length > 0) {
        if (order.customer) {
          if (order.customer.active) {
            // Finally, the actual logic...
            return calculateTotal(order)
          }
        }
      }
    }
  }
  return null
}

5 مستويات من التداخل - صعب القراءة والصيانة

function processOrder(order) {
  if (!isValidOrder(order)) return null
  return calculateTotal(order)
}

function isValidOrder(order) {
  return order?.items?.length > 0
    && order?.customer?.active
}

استخراج شرط الحماية، تسطيح الهيكل

function calculateDiscount(total) {
  if (total > 100) return total * 0.1
  if (total > 50) return total * 0.05
  return 0
}

أرقام سحرية: ماذا تعني 100، 50، 0.1؟

const DISCOUNT_TIERS = {
  PREMIUM: { threshold: 100, rate: 0.10 },
  STANDARD: { threshold: 50, rate: 0.05 },
}

function calculateDiscount(total) {
  const tier = Object.values(DISCOUNT_TIERS)
    .find(t => total > t.threshold)
  return tier ? total * tier.rate : 0
}

استخراج الثوابت بأسماء ذات معنى

function calc(d, t) {
  const r = d / t
  const x = r * 3.6
  return x
}

أسماء مبهمة: d، t، r، x لا تخبرك بشيء

const MS_TO_KMH = 3.6

function calculateSpeedKmh(distanceMeters, timeSeconds) {
  const metersPerSecond = distanceMeters / timeSeconds
  return metersPerSecond * MS_TO_KMH
}

أسماء وصفية توضح الغرض