อธิบายโค้ดเก่าที่รับช่วงต่อมาพร้อมคอมเมนต์

โค้ดทำอะไร

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: เปล่า ๆ ซ่อนข้อผิดพลาดจริง ทำให้ log ไม่มีประโยชน์

โค้ดที่แก้แล้ว

ปัญหาที่ 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]) เป็นชื่อหัวคอลัมน์

โปรดทดสอบ การแก้แต่ละจุดกับสำเนาไฟล์ของคุณ: เราอ่านโค้ดแต่ไม่ได้รันจริง