كيف تجعل مراجعات شفرتك ممتعة (وليست مروعة)

لقد أنجزت نصيبي العادل من مراجعات الكود. بالمشاركة العادلة ، أعني الكثير. هل سبق لك أن أجريت مراجعة كود لميزة ملحمية؟ عندي. لم تكن تجربة رائعة ، لأنه بحلول الوقت الذي كان من المقرر فيه مراجعة الملف رقم 100 ، كنت مرهقًا بالفعل عند النظر إلى الكود. كنت قريبًا جدًا من مجرد قول "نعم ، يبدو جيدًا بالنسبة لي" ثم منح موافقتي.

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

أنا أستطرد رغم ذلك. الألم ليس ما يفترض أن أتحدث عنه.

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

عظيم! آمل أن تكون قد أحببت مقالي. لنذهب جميعًا إلى المنزل ...

انتظر توقف !

كان الأمر يتعلق بكيفية جعل الأمر أقل رعبًا ، ولكن ماذا عن جعله ممتعًا؟

استخدام الميمات في مراجعات الكود

إخلاء المسؤولية: فيما يلي أفكاري الخاصة وليست أفكار صاحب العمل.

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

يمكنني سماع دحرجة عينك من هنا! تسمعني رغم ذلك. ما هو التفسير الأفضل من الارتباط بشخص آخر شرحه بالفعل بكل سرور؟

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

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

كان من الأسهل فتح أداة البحث ، وسحب الدوج إلى تعليق ، ونشره ، بدلاً من مجرد كتابة الكلمة القديمة Please remove the extra whitespaceفي طلب السحب (PR) ، خاصةً إذا كان هناك العديد من الكلاب.

ألا يمكن أن يأتي بنتائج عكسية؟

يمكن تماما. أنا لا أقول أنه يجب على الجميع متابعة فريقنا. يعتمد الأمر حقًا على شخصية الفريق. يبلغ متوسط ​​عمرنا ست سنوات أصغر مني ، مما يعني أن معظمهم يمكن أن يرتبط بالميمات. ستكون قصة مختلفة إذا كان متوسط ​​عمر فريقك 40 عامًا (ما لم يكن بالطبع من سكان 4chan أو محدثين بأحدث الميمات!).

أنت تعرف أفضل زملائك في الفريق حقًا. بعد بضعة أشهر من التواجد مع بعضنا البعض (خاصة وأننا كنا نتزاوج بنسبة 100 ٪ تقريبًا) ، شعرنا نوعًا ما بروح الدعابة لبعضنا البعض ، إذا جاز التعبير. لقد جعلنا ذلك مرتاحين في رؤية الميمات في العلاقات العامة لدينا والضحك عليها (أثناء إصلاح المشكلة ، بالطبع).

ما الفوائد التي تحصل عليها من الميمات؟

حسنًا ، إحدى الفوائد أنه يجعل المراجعة أسرع (على الأقل بالنسبة لي). الشخص الذي قدم أيضًا PR يفهم في الواقع ما يحتاج إلى القيام به بشكل أسرع. بدلاً من قراءة بضع كلمات ، ترى صورة - ونعلم جميعًا أن الصور تتحدث بصوت أعلى من الكلمات. فيما يلي بعض الأمثلة من personal image wallet™:

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

هذا خاص بـ Ruby - pretzel dotالمشغل هو في الأساس اختصار لـ a try-catch. أنا فقط أبرز المقتطف الذي يحتاج إلى قطعة مملحة وألصق هذه الصورة. سريع وسهل!

هذان الشخصان أستخدمهما بالتبادل ومرة ​​أخرى ، في الغالب في Ruby حيث نضيف دائمًا # frozen_string_literal: trueالتعليق السحري في أعلى الملف. مع وجود Rubocop في القيادة ، على الرغم من ذلك ، فقد رأينا القليل من Elsa و Mr. تظهر أيضًا لأي ثابت يحتاج إلى تجميد.

هذا واضح ومباشر (آمل). عندما يرى الزملاء هذا ، فهذه علامة على وجود خطأ مطبعي. عادةً ما أسلط الضوء على الخطأ المطبعي أيضًا ، لذا من السهل رؤيته (وإصلاحه).

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

هذا كل ما لدي أيها الناس

في الختام ، الأمر متروك لك حقًا إذا كنت ترغب في الاستمتاع ، أو مجرد البقاء جادًا ومهنيًا في العمل. قد يعتقد البعض أن الميمات غير مهنية - ومن المؤكد أنها قد تكون كذلك في مهام العمل الرسمية. يمكن للمرء أن يجادل في أن طلب السحب يمكن أن يكون أو يكون شيئًا رسميًا متعلقًا بالعمل ، لكنني أعتقد أنه ينطبق فقط إذا كنت تستخدمه للتوثيق أو المراجعات. بخلاف ذلك ، أعتقد أنها "حرية التعبير" (نعم ، سحبت تلك البطاقة!) ويجب أن تؤخذ كما هي.

بالنسبة لي ، فإنه ينقل ما أريد قوله: إزالة المسافات البيضاء الزائدة ، أو إصلاح التهجئة ، أو تجميد ثابت ، أو إزالة سطر أو سطور من التعليمات البرمجية. لست مضطرًا للكتابة كثيرًا ، زميلي ليس مضطرًا لقراءة مجموعة من الكلمات ضد مجموعة أخرى من الكلمات ، وهذا يخفف من الحالة المزاجية قليلاً. الجميع يفوز! بذلك ، سأُنهي هذا بصورة أخرى من صوري: