A: コーディング — ガード節（早期リターン）× Result型設計

概要

問題

ECサイトの販促システムのクーポン適用ロジック。以下の「悪いコード」を読んで問題点を指摘し、改善してください。

背景

制約・前提条件

• Python 3.12 を使用
• ユーザー・カート・クーポンはドメインオブジェクトとして表現
• 割引は注文合計金額に対するパーセンテージ（0.0〜1.0）
• エラーは例外ではなくResult型で返す（raise不可）
• 金額計算は decimal.Decimal を使用

悪いコード (Before)

📋 コピーdef apply_coupon(user: dict, cart: dict, coupon: dict) -> dict:
    result = dict(cart)
    if coupon["is_active"]:
        if result["total"] >= coupon["min_order"]:
            if user["membership_days"] >= coupon["required_days"]:
                if result["total"] <= coupon.get("max_order", float("inf")):
                    if coupon["remaining_count"] > 0:
                        discount = result["total"] * coupon["discount_rate"]
                        result["total"] -= discount
                        result["applied_coupon"] = coupon["code"]
                        return {"ok": True, "cart": result}
                    return {"ok": False, "error": "使用回数上限超過"}
                return {"ok": False, "error": "注文金額が上限を超えています"}
            return {"ok": False, "error": "会員歴不足"}
        return {"ok": False, "error": "最低注文金額未満"}
    return {"ok": False, "error": "クーポン無効"}

ヒント（段階的開示）

def apply_coupon(user: dict, cart: dict,
                 coupon: dict) -> dict:
    if coupon["is_active"]:
        if cart["total"] >= coupon["min_order"]:
            if user["membership_days"] >= ...:
                # さらにネスト...
                return {"ok": True, "cart": ...}
    return {"ok": False, "error": "..."}

def apply_coupon(user: User, cart: Cart,
                 coupon: Coupon) -> CouponApplyResult:
    # ガード節: 失敗を先に弾く
    if not coupon.is_active:
        return CouponApplyResult(ok=False, ...)
    if cart.total < coupon.min_order:
        return CouponApplyResult(ok=False, ...)
    # ...残りのガード節
    # 正常系は最後の1行のみ
    return CouponApplyResult(ok=True, ...)

問題点分析

ガード節パターン — 制御フロー構造図

ネスト型（Before）とガード節型（After）の制御フローの違い。

Before（左）は成功条件を全てネストで積み重ね、失敗パスがコードの末端に散在する。After（右）は失敗条件をガード節で先頭に集め、正常系を最後の1行に収めることで「上から下に読める」コードになる。

模範解答

📋 コピーfrom dataclasses import dataclass
from decimal import Decimal, ROUND_DOWN
from typing import Final

MAX_ORDER_DEFAULT: Final[Decimal] = Decimal("9999999")


@dataclass(frozen=True, slots=True)
class User:
    user_id: str
    membership_days: int


@dataclass(frozen=True, slots=True)
class Cart:
    cart_id: str
    total: Decimal


@dataclass(frozen=True, slots=True)
class Coupon:
    code: str
    is_active: bool
    min_order: Decimal
    required_days: int
    discount_rate: Decimal
    remaining_count: int
    max_order: Decimal = MAX_ORDER_DEFAULT


@dataclass(frozen=True, slots=True)
class CouponApplyResult:
    ok: bool
    error: str = ""
    discount: Decimal = Decimal("0")
    applied_code: str = ""


def apply_coupon(user: User, cart: Cart, coupon: Coupon) -> CouponApplyResult:
    """クーポンをカートに適用し結果を返す（副作用なし）。

    全バリデーションをガード節で先に弾き、
    正常系は最後の1行に集約する。
    """
    # ガード節: 失敗条件を先に返す（早期リターン）
    if not coupon.is_active:
        return CouponApplyResult(ok=False, error="クーポンが無効です")
    if cart.total < coupon.min_order:
        return CouponApplyResult(ok=False, error=f"最低注文金額 {coupon.min_order}円 未満です")
    if user.membership_days < coupon.required_days:
        return CouponApplyResult(ok=False, error=f"会員歴 {coupon.required_days}日 以上が必要です")
    if cart.total > coupon.max_order:
        return CouponApplyResult(ok=False, error=f"注文金額が上限 {coupon.max_order}円 を超えています")
    if coupon.remaining_count <= 0:
        return CouponApplyResult(ok=False, error="クーポンの使用回数上限に達しました")

    # 正常系: ここまで到達したら全条件を満たしている
    discount = (cart.total * coupon.discount_rate).quantize(Decimal("1"), rounding=ROUND_DOWN)
    return CouponApplyResult(ok=True, discount=discount, applied_code=coupon.code)

📋 コピー# --- 実行例 ---
user = User(user_id="u001", membership_days=30)
cart = Cart(cart_id="c001", total=Decimal("3000"))
coupon = Coupon(
    code="SPRING20",
    is_active=True,
    min_order=Decimal("2000"),
    required_days=7,
    discount_rate=Decimal("0.20"),
    remaining_count=5,
)

result = apply_coupon(user, cart, coupon)
if result.ok:
    print(f"クーポン適用成功: {result.applied_code}")
    print(f"割引額: {result.discount}円")
    print(f"支払額: {cart.total - result.discount}円")
else:
    print(f"適用失敗: {result.error}")

# 出力:
# クーポン適用成功: SPRING20
# 割引額: 600円
# 支払額: 2400円

# 失敗ケース
cart_cheap = Cart(cart_id="c002", total=Decimal("1500"))
result2 = apply_coupon(user, cart_cheap, coupon)
print(result2.error)  # 最低注文金額 2000円 未満です

ポイント解説

設計パターン対応表

実務への応用

• 新条件の追加: 「特定商品カテゴリのみ・初回購入者のみ」追加時に、既存のネストを変えずガード節を1行追加するだけで対応できる
• DataDogメトリクス: CouponApplyResult を DataDogのカスタムメトリクスとして記録すれば、error フィールドを error_reason タグにして失敗理由別の分布が可視化できる
• BigQuery連携: 型が明確なdataclassから直接BigQueryへマッピングできる（Pydantic v2 の model_validate とも相性が良い）

次のステップ

• 参考: 「良いコード・悪いコードで学ぶ設計入門」第6章「条件分岐の単純化」・第8章「密結合」

今日のまとめ

自己評価

自分の回答

気づき・メモ