مراجعة الكود بمساعدة الذكاء الاصطناعي

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

قبل أن تبدأ

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

1طلب مراجعة يعطي نتيجة

قارن الطلبين. الأول يعطي مديحًا عامًا، والثاني يعطي ملاحظات تُختبر:

Bad:
  "Review this code."

Good:
  Context: Flask API, Python 3.12, PostgreSQL. This diff adds an
  endpoint that lets a user download their own invoices.
  Check, in order:
  1. Can a user reach another user's invoice?
  2. Unhandled errors and missing input validation.
  3. SQL built from strings.
  4. Edge cases: empty list, very large list, missing id.
  For each finding give: file:line, severity (high/medium/low),
  a concrete request that triggers it, and the smallest fix.
  If you find nothing for an item, say so. Do not comment on style.

  <diff here>

2فرز الملاحظات

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

3مراجع آلي صغير

الأدوات الثابتة حتمية: نفس المدخل يعطي نفس النتيجة دائمًا. هذا مراجع مصغّر بمكتبة ast يكشف نمطين خطرين: except: العارية والقيمة الافتراضية القابلة للتغيير:

import ast

source = """
def add_item(item, basket=[]):
    basket.append(item)
    return basket

def load(path):
    try:
        return open(path).read()
    except:
        return None
"""

for node in ast.walk(ast.parse(source)):
    if isinstance(node, ast.ExceptHandler) and node.type is None:
        print(f"line {node.lineno}: bare except hides every error")
    if isinstance(node, ast.FunctionDef):
        for default in node.args.defaults:
            if isinstance(default, (ast.List, ast.Dict, ast.Set)):
                print(f"line {node.lineno}: mutable default in {node.name}()")

مثال محلول

المسألة: المساعد قال «السطر 14 عرضة لحقن SQL». التحقّق: السطر يستخدم cursor.execute("... WHERE id = %s", (user_id,))، أي استعلام بمعاملات. التصنيف: خاطئة، فالمعاملات تمنع الحقن. وفي نفس الرد قال «لا يتحقق أن الفاتورة تخص المستخدم». التحقّق: طلب /invoices/7 من مستخدم آخر أعاد 200. التصنيف: صحيحة وخطيرة، فاكتب الاختبار وأضف AND owner_id = %s.

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

اختبر فهمك

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

تحقق من فهمك