forumفتح موضوع جديد
فتح موضوع جديد

إزاي تبني ثقافة مراجعة الكود؟ ملاحظات من عشر سنين خبرة

RRıdvan Y***قبل 18 يومًا·48 رسالة·17.1 ألف مشاهدة#فريق#جودة#عملية
RRıdvan Y***خبير
المسمى الوظيفي
قائد فريق البرمجيات
تاريخ الانضمام
سبتمبر 2023
رسالة
196

Doki · تصميم الواجهات · 2023

#1

من زمان بقى ليدر فريق، هكتب بهدوء لأن أكبر غلطة في الموضوع ده هي الاستعجال.

مراجعة الكود مش تفتيش، دي أداة تعليمية. الفرق اللي مش بتقبل الجملة دي، العملية بتخلص دايماً في نفس النقطة: الشخص المخضرم بيصطاد الأخطاء، والشاب بيبدأ يدافع عن نفسه، المراجعات بتبطأ، وفي الآخر الكل يوافق من غير ما يقرأ.

جمعت شوية قواعد نفعت، هشاركها معاكم.

ابعتوا التعديلات على أجزاء صغيرة. محدش يقدر يقرأ فعلاً تعديل من 500 سطر، الكل بيكتب "يبدو جيداً". لما التعديل يكون أقل من 200 سطر، عدد المشاكل اللي بتظهر بيزيد بشكل ملحوظ.

اكتب التعليق على الكود مش على الشخص. بدل "ليه عملت كده"، اسأل "إيه اللي هيحصل هنا في الحالة دي". نفس المعلومة، بس الحوار بيكون مختلف تماماً.

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

CCerenمشارك
المسمى الوظيفي
خبير اختبار
تاريخ الانضمام
مارس 2024
رسالة
178
الأكثر إفادة#2

أنا معاك في الاختبار بس عندي سؤال: هل مكتوب ومحدد إيه اللي بتدوروا عليه في المراجعة؟

سبب سؤالي هو إن في فرق كتير بتعمل مراجعة كود، بس اللي بيتبص عليه بيختلف من شخص للتاني. واحد يبص على التسمية، واحد على معالجة الأخطاء، وواحد ميبصش على حاجة. وبعدين يقولوا "راجعنا"، ونفس الأخطاء تطلع للإنتاج.

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

أنا قست الرقم مش بتخمين: في الربع اللي بعد ما حطينا القائمة دي، عدد الأخطاء اللي طلعت للإنتاج نزل بشكل ملحوظ. القائمة مش سحر، بس بتخلي الكل يبص على نفس الحاجة.

KKayahanمشارك
المسمى الوظيفي
مطور متكامل
تاريخ الانضمام
يونيو 2024
رسالة
118
#3

أنا معاك في قاعدة الأجزاء الصغيرة بس صعبة عملياً. أحياناً التعديل بيكون كبير بطبيعته.

الحل عندنا بقى إننا قبل ما نبعت التعديل الكبير، نكتب النهج في صفحة واحدة ونشاركها. قراءة خمس دقايق بتوفر نقاش مراجعة ساعة.

أسوأ حاجة تكتشف إن النهج غلط بعد ما تكتب 500 سطر. في النقطة دي محدش عايز يرجع، والقرار السيء يطلع للإنتاج.

AAhmetعضو جديد
المسمى الوظيفي
طالب · برمجيات
نوع المنظمة
شركة عائلية
تاريخ الانضمام
يناير 2025
رسالة
48
#4

ممكن أقول حاجة كوني مبتدئ؟

أصعب حاجة عندي مش إني آخذ التعليق بشكل شخصي. لما يجيني تعليق مش فاهمه، بتكاسل أسأل تاني، بحس إني هبوظ نفسي. وبعدين أحاول أحزر وأصلح غلط.

مش عارف ده بس عندي ولا عند ناس تانية، بس يمكن يكون عند غيري كمان.

RRıdvan Y***خبير
المسمى الوظيفي
قائد فريق البرمجيات
تاريخ الانضمام
سبتمبر 2023
رسالة
196

Doki · تصميم الواجهات · 2023

#5

مش عندك بس، دي حاجة شائعة جداً، ومهمتنا إحنا نصلحها مش مهمتك.

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

التانية، لما يبقى أكتر من جولتين مراسلة، حولوا النقاش لمحادثة صوتية. اجتماع 15 دقيقة أسرع من 15 تعليق وأقل إجهاداً بكتير.

وقول كمان: مفيش حاجة اسمها سؤال غبي. السؤال اللي إنت بتسأله، تلات ناس في الفريق كمان عايزين يسألوه بس مش بيسألوا. لما تسأل، بتعمل خير لأربعهم مرة واحدة.

SSerhatمشارك
المسمى الوظيفي
مطور Go
تاريخ الانضمام
مايو 2024
رسالة
88
#6

ترك نقاش التنسيق للأداة لوحده أكبر مكسب. بيتظبط في أسبوع، وبيوفر وقت لسنوات.

SSinemخبير
المسمى الوظيفي
مدير مشاريع
تاريخ الانضمام
أكتوبر 2023
رسالة
176
#7

تحذير من إدارة المشروع: اكتبوا مدة المراجعة في الخطة.

عندنا صار كذا، ما حسبنا المراجعة جزء من الشغل لفترة طويلة، كان في الخطة وقت التطوير بس. النتيجة: كل مراجعة كانت تتشوف كـ "شيء يبطئ الشغل" وتصير بشكل صوري.

من يوم ما كتبناها في الخطة، ما حد يعرقل المراجعة، لأنها صارت جزء من الخطة نفسها مو شيء يبطئ الشغل.

GGürkanمشارك
المسمى الوظيفي
قطاع الطاقة
نوع المنظمة
وكالة بوتيك
تاريخ الانضمام
نوفمبر 2023
رسالة
94
#8

عانيت من نفس الشيء.

AAyşegülمشارك
المسمى الوظيفي
فندق بوتيك
نوع المنظمة
ورشة تصنيع
تاريخ الانضمام
أغسطس 2024
رسالة
86
#9

بالضبط هكذا. إذا لم تقم بصياغة هذا كتابيًا من البداية، فستنشأ خلافات لاحقًا.

دعني أتركها كملاحظة، قد تكون مفيدة.

TTolga Y***خبير
المسمى الوظيفي
مسؤول التصدير
قطاع
مطبعة
نوع المنظمة
سلسلة متاجر
تاريخ الانضمام
أغسطس 2023
رسالة
3
#10

أوافقك الرأي تمامًا. فعل الجميع لشيء ما لا يعني أنه صحيح.

اكتبوا إذا كانت لديكم أسئلة، سأجيب قدر استطاعتي.

TTülay K***مشاركعضو المجتمع
تاريخ الانضمام
مارس 2023
رسالة
216
#11

حدث معي العكس تماماً، لذلك أكتب هذا. المسألة ليست الرقم، بل الأساس الذي بُني عليه.

إذا كتبت النتيجة هنا، فستفيد الآخرين أيضاً.

ŞŞerife K***محارب قديم
المسمى الوظيفي
مدير عيادة
قطاع
كهرباء وإلكترونيات
نوع المنظمة
شركة ناشئة حديثة التأسيس
تاريخ الانضمام
ديسمبر 2023
رسالة
128
#12

دعني أترك تنبيهًا. انظروا أولاً إلى البيانات المتاحة لديكم عند اتخاذ القرار.

لا تتحسن أي عملية لا يتم توثيقها، لأنك لا تعرف ما الذي يجب إصلاحه. إذا كان هناك من يفعلها بطريقة أخرى، فأنا فضولي لمعرفة ذلك.

KKaan G***خبيرعضو المجتمع
تاريخ الانضمام
يونيو 2023
رسالة
94
#13

أتابع الموضوع.

EElif B***مشارك
المسمى الوظيفي
منسق التوصيل
قطاع
الخدمات الصحية
نوع المنظمة
شركة ضمن مجموعة استثمارية
تاريخ الانضمام
ديسمبر 2023
رسالة
228
#14

عندنا نفس المشكلة.

AAlper E***مشاركعضو المجتمع
تاريخ الانضمام
ديسمبر 2024
رسالة
184
#15

تم الحفظ.

JJülide A***مشارك
المسمى الوظيفي
مدير المحاسبة
قطاع
مجوهرات
نوع المنظمة
شركة من 20 شخصاً
تاريخ الانضمام
مايو 2024
رسالة
103

Doki · فحص الثغرات الأمنية · 2026

#16

سأجمع الموضوع، لأن هناك عدة إجابات مختلفة. ابدأ بتجربة صغيرة، لا تربط كل شيء دفعة واحدة.

لو كنت مكانك، لسلك هذا المسار.

GGökhan A***مشارك
المسمى الوظيفي
منتج · أثاث
تاريخ الانضمام
أكتوبر 2023
رسالة
74
#17

أرجو المعذرة، لكن هذا لا ينطبق في كل الحالات. الحلول التي تنجح على نطاق صغير تنهار عند التوسع، تعلمت هذا متأخراً.

صححوا لي إن أخطأت.

YYasemin S***مشاركعضو المجتمع
تاريخ الانضمام
فبراير 2024
رسالة
42
#18

نادرًا ما تجد مقالاً يشرح الأمر بهذه الوضوح. لا تترددوا في السؤال، فالسائل يدفع ثمناً أقل دائماً.

إذا كان النطاق يتوسع، فيجب أن يتوسع إما الجدول الزمني أو الميزانية. لا يوجد خيار ثالث. إذا كان هناك من يفعلها بطريقة أخرى، فأنا فضولي لمعرفة ذلك.

RRecepعضو جديد
المسمى الوظيفي
سباك
نوع المنظمة
شركة من 20 شخصاً
تاريخ الانضمام
ديسمبر 2024
رسالة
22
#19

موضوع في الوقت المناسب تمامًا.

AAlper P***مشارك
المسمى الوظيفي
مدير الأنظمة
قطاع
عقارات
نوع المنظمة
شركة من 20 شخصاً
تاريخ الانضمام
أغسطس 2024
رسالة
85
#20

أخذت ملاحظة، شكرًا لك.

اكتب رداً