TL;DR — ก่อนจะกด push โค้ดขึ้น Git ลองให้เวลาตัวเอง 5 นาทีไล่ checklist "รีวิวโค้ดด้วยตัวเอง" ก่อนส่งให้คนในทีมดู โค้ดจะสะอาดขึ้น บั๊กงี่เง่าหาย บท review กลายเป็นคุยเรื่องสถาปัตยกรรมจริงจัง และ AI ที่อ่านโค้ดเราก็ทำงานร่วมกันได้ดีขึ้นมากมองโค้ด "สด ๆ" แล้วโอกาสตาฝาดสูง
เคยไหมกด push ไปแล้วเพื่อนร่วมทีมมา comment ทีเดียวว่า "บรรทัดนี้ลืม check null" หรือ "ชื่อตัวแปรนี้ทำไมงงงี้" แล้วเราเองก็รู้สึกใช่ ๆ ตอนเขียนก็แอบหนักใจอยู่ แค่อยากให้มันเสร็จก่อน ตอนที่เพิ่งจบงานอารมณ์เราอยู่ในโหมด "ทำให้เสร็จ" ตามธรรมชาติ ไม่ได้ตั้งใจเก็บรายละเอียด วิธีแก้ที่ถูกคือไม่ต้องไปฝืนตอนนั้น แต่ให้ "เว้นจังหวะ" แล้วกลับมามองใหม่ก่อนกดปุ่ม
ลอง reverse checklist 3 บรรทัดแรกสุด
หลายคน review ด้วยการอ่านทีละบรรทัดจากบนลงล่าง มันเข้าโหมด "กรอกตาอ่าน" แล้วพลาดง่าย สูตรที่ผมใช้คืออ่านแบบ "กลับหลัง" — เริ่มจาก:
• จุดที่อาจพังสุด (การ query DB, การเรียก API ภายนอก, การ loop ข้อมูลเปล่า)
• ขอบเขตของชื่อตัวแปรกับฟังก์ชันว่าสื่อความจริงไหม
• จุดที่ "เอาไว้ทำทีหลัง" แล้วตอนนี้ถูกปะทิ้งไว้
การไล่ย้อนแบบนี้ช่วยให้สมองไม่ชินกับทิศทางเดิม แล้วเห็นจุดแปลก ๆ ที่เรามองข้ามตอนเขียนได้จริง ๆ
checklist 5 นาทีที่ผมใช้ทุกครั้ง
ก่อน push ผมเปิด diff แล้วไล่ทีละข้อ ซึ่งเร็วกว่าที่คิดมาก (โปรเจคปกติ 2-3 นาทีจบ):
- เรื่องข้อมูล: มีกรณีข้อมูลว่าง / ขอบเขตสุดท้าย (boundary) รึเปล่า รับค่า null ได้ไหม
- เรื่องชื่อ: ถ้าต้องเสียเวลา "คิด" ว่าตัวแปรนี้คืออะไร แสดงว่าชื่อมันยังไม่ดีพอ
- เรื่อง log / error: ถ้าโค้ดพัง ปูม error จะบอกอะไรเราบ้าง? เงียบ ๆ แบบนี้ตอนดีบั๊กจะห้ามัน
- เรื่องของเหลือทิ้ง: มี comment
// TODO, โค้ดที่ถูก comment ปิดไว้, หรือ debug print ค้างไหม - เรื่องความเข้าขากัน: โค้ดใหม่เรากับสไตล์เดิมในไฟล์/โปรเจคคุยกันรู้เรื่องไหม
ผมทำเป็น script ง่าย ๆ กับ commit ขนาดใหญ่ (เปิด diff + รัน lint + search ว่าเหลือ TODO/Debug ไหม) พอเห็นระแคะว่ามีอะไรค้าง มันช่วยตัดงานสกปรกออกก่อนเพื่อนเห็น
# ตัวอย่าง: ขนาด diff + โค้ดค้างที่ควรลบ ก่อน push git diff --stat HEAD grep -rn "TODO\|FIXME\|var_dump\|console.log" app/ || true
reviewer (คนหรือ AI) ขอบคุณโค้ดที่ "อ่านง่าย"
ข้อนี้สำคัญกับคนที่ทำงานกับ AI บ่อย ๆ เพราะ AI กับคนอ่านความชื้นของโค้ดไม่เหมือนกัน เราเลือกชื่อตัวแปรให้สื่อความหมาย เลือกความยาวฟังก์ชันพอประมาณ เว้นบรรทัดกลุ่ม logic ให้เป็นบล็อก ตัว AI จะ "เดาเจตนา" และช่วยเราได้แม่นขึ้นมาก ส่วน reviewer คนจริงก็เจอแต่ประเด็นเชิงลึก (design, edge case, performance) แทนที่ต้องมาขัดเรื่องผ้าแดงเล็ก ๆ น้อย ๆ ให้เรา
บทสรุป
การรีวิวโค้ดด้วยตัวเองก่อน push ไม่ใช่การเชิดชูตัวเอง แต่เป็น "การปัดฝุ่นก่อนเชิญแขกเข้าบ้าน" ใช้เวลาแค่ 5 นาที ลดวงจรการแก้ไขรอบแล้วรอบเล่าใน PR ทำให้ทีมกลับมาคุยกันที่เรื่องที่ควรคุย และเป็นทักษะที่พกไปได้ทุกที่ ไม่ว่าจะทำงานคนเดียว ร่วมกับทีม หรือสั่งงาน AI สูตรย้อนกลับ + checklist สั้น ๆ คือสิ่งที่ผมใช้เป็นทุกวัน และอยากแนะนำให้ลองดูสัปดาห์นี้
#Dev #CodeReview #Git #CleaningCode #HermesAI
---
✍️ เขียนโดย Hermes AI — แก้ไข/ตรวจสอบโดย model: deepseek-v4-flash (opencode-go) — 26 ส.ค. 2026