Comprehensive Code Review Expert

가장 큰 원인은 calculateTotal의 반복 조건입니다.

for (let i = 0; i <= items.length; i++)

배열의 마지막 유효 인덱스는 items.length - 1이므로, 마지막 반복에서 items[items.length]가 undefined가 됩니다. 그 결과 items[i].price 접근 시 예외가 발생합니다.

수정 예시는 다음과 같습니다.

function calculateTotal(items) {
  if (!Array.isArray(items)) {
    throw new TypeError('items must be an array');
  }

  return items.reduce((total, item) => {
    const price = Number(item.price);
    const qty = Number(item.qty);

    if (!Number.isFinite(price) || !Number.isFinite(qty) || qty < 0) {
      throw new TypeError('Invalid price or quantity');
    }

    return total + price * qty;
  }, 0);
}

function applyDiscount(total, coupon) {
  if (!coupon) return total;

  let discountedTotal;

  if (coupon.type === 'percent') {
    const percent = Number(coupon.value);

    if (!Number.isFinite(percent) || percent < 0 || percent > 100) {
      throw new RangeError('Invalid discount percentage');
    }

    discountedTotal = total * (1 - percent / 100);
  } else if (coupon.type === 'fixed') {
    const discount = Number(coupon.value);

    if (!Number.isFinite(discount) || discount < 0) {
      throw new RangeError('Invalid fixed discount');
    }

    discountedTotal = total - discount;
  } else {
    throw new TypeError('Unknown coupon type');
  }

  return Math.max(0, discountedTotal);
}

async function checkout(cart) {
  const subtotal = calculateTotal(cart.items);
  const total = applyDiscount(subtotal, cart.coupon);

  const response = await fetch('/api/order', {
    method: 'POST',
    headers: {
      'Content-Type': 'application/json'
    },
    body: JSON.stringify({ total })
  });

  if (!response.ok) {
    throw new Error(`Order failed: ${response.status}`);
  }

  console.log('주문 완료');
}

추가로 확인할 사항은 다음과 같습니다.

• fetch를 await하지 않아 API 요청이 실패해도 "주문 완료"가 먼저 출력됩니다.
• Content-Type 헤더가 없어 서버가 JSON 본문을 제대로 해석하지 못할 수 있습니다.
• 할인 금액이 상품 합계보다 커지면 음수 합계가 될 수 있습니다.
• ==와 문자열 비교 대신 ===를 사용하는 것이 안전합니다.
• 실제 결제 금액이라면 부동소수점 오차를 피하기 위해 원 단위가 아닌 정수 최소화폐단위(예: 센트)로 계산하는 것이 좋습니다.
• 최종 합계는 클라이언트 값만 신뢰하지 말고 서버에서 장바구니 상품과 쿠폰을 재검증해야 합니다.