شرح شيفرة قديمة موروثة مع تعليقات

ماذا تفعل الشيفرة

1. تقرأ orders.csv من ملف التصدير.
2. لكل سطر طلب، تبحث عن المنتج برمز SKU في stock.xlsx وتطرح الكمية.
3. تكتب المخزون المحدّث وتسجّل الإجماليات في import_log.txt.

البنية واضحة وأسماء الدوال جيدة. معظم المشكلات تتعلق بمدخلات لا تأتي بالشكل الذي يتوقعه السكربت.

الأخطاء ونقاط الضعف

#	السطر	الخطورة	المشكلة
1	48	عالية	يمكن استيراد ملف CSV نفسه مرتين؛ لا يوجد تحقق من أرقام الطلبات المعالجة سابقًا. هذا السبب الأرجح للمخزون السالب.
2	72	عالية	int(row["qty"]) يتعطل مع القيم الفارغة أو "2.0" في الطلبات المستردة.
3	95	متوسطة	البحث عن SKU حساس لحالة الأحرف: "gt-101" و"GT-101" يُعامَلان كمنتجين مختلفين.
4	130	متوسطة	يُفتح الملف دون encoding="utf-8"، فتتلف أسماء المنتجات العربية على Windows.
5	201	منخفضة	except: المجرد يخفي الأخطاء الحقيقية ويجعل السجل بلا فائدة.

مقاطع مصحّحة

المشكلة 1: تجاوز الطلبات المستوردة مسبقًا

processed = load_processed_ids("processed_orders.txt")
for row in reader:
    if row["order_id"] in processed:
        continue
    # ... المنطق الحالي ...
    processed.add(row["order_id"])
save_processed_ids("processed_orders.txt", processed)

السبب: يُطبَّق كل طلب مرة واحدة، حتى لو شغّلت السكربت على الملف نفسه مجددًا.

المشكلة 2: قراءة الكمية بأمان

raw = (row.get("qty") or "").strip()
try:
    qty = int(float(raw))
except ValueError:
    log.warning("الطلب %s: كمية غير صالحة %r، تم تجاوزه", row["order_id"], raw)
    continue

السبب: تُتجاوز الأسطر المستردة كليًا أو جزئيًا مع ملاحظة واضحة، بدل إيقاف الاستيراد كله.

المشكلة 3: توحيد رموز SKU

sku = row["sku"].strip().upper()

طبّقي .strip().upper() نفسها عند بناء قاموس المخزون.

الأسلوب وسهولة القراءة

• الدالة process() طولها 90 سطرًا؛ قسّميها إلى read_orders وapply_to_stock وwrite_report.
• استبدلي أرقام الأعمدة الثابتة (row[7]) بأسماء العناوين.

يُرجى اختبار كل إصلاح على نسخة من ملفاتك: قرأنا الشيفرة لكننا لم نشغّلها.