الكود النظيف وإعادة الهيكلة

طريقان إلى نفس الفكرة

قبل أن تبدأ

الفكرة الجوهرية

1قبل وبعد

دالة حقيقية الشكل: أسماء غامضة، أرقام سحرية، وتداخل عميق. نعيد هيكلتها خطوة خطوة ونتأكد أن النتائج لم تتغير:

def calc(o):
    t = 0
    if o:
        if o["items"]:
            for i in o["items"]:
                t += i["p"] * i["q"]
            if o["member"]:
                t = t * 0.9
            if t > 200:
                t = t - 15
    return round(t, 2)

MEMBER_DISCOUNT = 0.10
BIG_ORDER_THRESHOLD = 200
BIG_ORDER_REWARD = 15

def subtotal(items):
    return sum(item["p"] * item["q"] for item in items)

def order_total(order):
    if not order or not order["items"]:
        return 0
    total = subtotal(order["items"])
    if order["member"]:
        total *= 1 - MEMBER_DISCOUNT
    if total > BIG_ORDER_THRESHOLD:
        total -= BIG_ORDER_REWARD
    return round(total, 2)

cases = [None, {"items": [], "member": False},
         {"items": [{"p": 14, "q": 2}], "member": True},
         {"items": [{"p": 50, "q": 5}], "member": False}]
for case in cases:
    assert calc(case) == order_total(case)
print("same behaviour on", len(cases), "cases:", [order_total(c) for c in cases])

2الأسماء والدوال الصغيرة

الاسم الجيد يغني عن التعليق. لو احتجت تعليقًا يشرح ما تفعله كتلة كود، فاستخرجها لدالة اسمها هو ذلك التعليق. والدالة التي تحتاج كلمة «و» لوصفها («تحسب وترسل») تفعل شيئين.

3التكرار والتجريد

قاعدة الثلاثة: أول مرة اكتب، ثاني مرة لاحظ التكرار، ثالث مرة استخرج. التجريد المبكر يربط أشياء متشابهة صدفة، ثم تتباعد فيمتلئ بالشروط.

مثال محلول

المسألة: if user: if user.active: if user.email: send(...). ثلاث مستويات تداخل. إعادة الهيكلة: if not user or not user.active or not user.email: return ثم send(...). التحقق: نفس الاختبارات الثلاثة (مستخدم None، غير نشط، بلا بريد) تمر. الكود صار سطرين يُقرآن من الأعلى للأسفل.

خرافة مقابل حقيقة

اختبر فهمك

بطاقات مراجعة سريعة

تحقق من فهمك