مراجعة الكود بلغة جافا: ما الذي تبحث عنه في كود الآخرين

لماذا مراجعة الكود ليست انتقادًا، بل تتعلق بالجودة

تخيل: تفتح طلب سحب (pull request) من زميل. هناك 500 سطر من الكود، ورائحة النسخ واللصق، وفي مكان ما نسي شخص ما إغلاق مورد. مهمتك ليست مجرد العثور على الأخطاء، بل جعل الكود أفضل. مراجعة الكود بلغة جافا ليست رقابة شرطية، بل إبداع مشترك. المراجع الجيد ينقذ الفريق من الديون التقنية، أما السيئ فيثبط الهمم أو يسمح بمرور الثغرات.

في هذه المقالة، سأشرح ما يجب التحقق منه في مراجعة الكود بلغة جافا، حتى لا تضيع الوقت في التفاصيل الصغيرة، ولكن لا تفوت الأمور الحرجة. ستحصل على قائمة تحقق جاهزة وأمثلة من الممارسة.

البنية والتصميم: انظر إلى الجذر

قبل الغوص في التفاصيل، قيّم الصورة العامة. البنية السيئة لا تُعالج بالتنسيق.

انتهاك مبادئ SOLID

تحقق مما إذا كانت الفئة قد تحولت إلى "كائن إلهي". مثال على انتهاك مبدأ المسؤولية الفردية:

public class OrderService {\n    public void createOrder(Order order) {\n        // التحقق\n        // حساب الخصم\n        // إرسال البريد الإلكتروني\n        // تسجيل السجلات\n        // الحفظ في قاعدة البيانات\n    }\n}\n

هنا خمسة أسباب للتغيير. الصحيح هو فصل تسجيل السجلات والإرسال إلى خدمات منفصلة.

التعقيد المفرط

أحيانًا يحل الكود مشكلة بسيطة بطريقة معقدة. اسأل نفسك: "هل يمكن أن يكون أبسط؟" إذا كانت الإجابة نعم، فاكتب ذلك في المراجعة. تذكر مبدأ KISS.

الأمان: ثغرات لا يجب تفويتها

مراجعة كود جافا يجب أن تتضمن التحقق من الأمان. حتى لو كان المشروع داخليًا.

حقن SQL

ربط السلاسل النصية هو شر:

// سَيء\nString query = "SELECT * FROM users WHERE name = '" + userName + "'";\nStatement stmt = connection.createStatement();\nResultSet rs = stmt.executeQuery(query);\n

الصحيح هو استخدام PreparedStatement:

// جيد\nString query = "SELECT * FROM users WHERE name = ?";\nPreparedStatement pstmt = connection.prepareStatement(query);\npstmt.setString(1, userName);\nResultSet rs = pstmt.executeQuery();\n

تسريب البيانات الحساسة

تحقق من عدم تسجيل كلمات المرور أو الرموز المميزة أو أرقام البطاقات. استخدم @ToString.Exclude في Lombok أو التجاهل الصريح.

مقارنة غير آمنة للكائنات

// سَيء (مقارنة المراجع)\nif (password == storedPassword) { ... }\n// جيد\nif (password.equals(storedPassword)) { ... }\n

الأداء: نقاط الاختناق

لا تقم بالتحسين المبكر، لكن لا تفوت أيضًا حالات التباطؤ الواضحة.

عمليات غير ضرورية في الحلقات

// سَيء: يتم استدعاء length() في كل مرة\nfor (int i = 0; i < list.size(); i++) { ... }\n// جيد\nint size = list.size();\nfor (int i = 0; i < size; i++) { ... }\n

استخدام Stream للمهام البسيطة

أحيانًا تكون الحلقة العادية أسرع في القراءة والأداء:

// مفرط\nlist.stream().filter(x -> x > 5).collect(Collectors.toList());\n// بسيط\nList<Integer> result = new ArrayList<>();\nfor (int x : list) {\n    if (x > 5) result.add(x);\n}\n

تسرب الذاكرة

تحقق مما إذا تم نسيان إغلاق InputStream أو Connection أو Session. استخدم try-with-resources:

try (Connection conn = getConnection();\n     PreparedStatement stmt = conn.prepareStatement(sql)) {\n    // العمل\n} catch (SQLException e) {\n    log.error("خطأ في قاعدة البيانات", e);\n}\n

قابلية القراءة والأسلوب: الكود ككتاب

قائمة التحقق لمراجعة الكود تتضمن بالضرورة قابلية القراءة. الكود يُكتب مرة واحدة، ويُقرأ عشرات المرات.

الأرقام والسلاسل السحرية

// سَيء\nif (status == 3) { ... }\n// جيد\nprivate static final int STATUS_COMPLETED = 3;\nif (status == STATUS_COMPLETED) { ... }\n

الطرق الطويلة