Code Reviews - معاينة الكود
Вставка
- Опубліковано 23 січ 2025
- مساء الخير, ازيكم
واحدة من الحاجات اللي ناس كثير علقت عليها في الفيديو اللي فات كانت النصيحة الأولى بتاعة Don’t Fall in Love with Your Code بالذات انها متعلقة جداً بمدى تقبل المبرمجين للنقد و التحسينات اللي بتتقال ليهم في ال Code Reviews.
و احنا اتكلمنا قبل كده عن ال Code Reviews في حلقة سابقة من #لايف_مع_القهوة, بس لأن السلسلة دي كانت أغلبها عفوية و متسجلة في مكان مفتوح, مش كل الناس بتقدر تتابعها.
يعني ايه Code Review؟
هي مرحلة من مراحل تطوير الكود و اللي فيها المطور بيطلب من زملاؤه مراجعة التغيير اللي ناوي يعمله, و يقدموا أي نقد أو تحسينات ممكنة لضمان ان الفريق شاف التغييرات قبل ما يتم دمجها في ال Codebase.
تقدر تقول هي مراسم تسليم المسؤولية من مسؤولية شخصية لمسؤولية مشتركة بين الفريق, و ده بيتطلب ان الفريق يبقى عنده فرصة يبدي رأيه و يتأكد ان الكود مش بس بيأدي المطلوب منه لكن كمان بيتبع المعايير المتفق عليها.
قبل ما نقول النصائح محتاجين الأول نتأكد ان الفريق بتاعك عنده مجموعة حاجات أساسية
1. فيه اتفاق على ال Code Style اللي الفريق حيستخدمه في خلال المشروع, و ده بيبقى عموماً معتمد بشكل أساسي على
a. اللغة اللي بتستخدمها و اللي في أغلب الوقت بيكون ليها Code Style مفضل
b. بعض الاضافات الخاصة بالشركة و بالفريق
c. ينصح بشدة ان يكون فيه ادوات زي ال Linters أو ال Code Style Checkers و اللي بتتأكد من ان أي كود لازم يتبع معايير معينة من أول استخدام spaces و لا tabs لحد ازاي تختار أسامي المتغيرات و تقسم الكود ازاي. ده بيوفر وقت كثير جداً أثناء المراجعة, لأن الفريق ساعتها حيركز على حاجات أهم من شكل الكود.
2. ان يكون فيهTest Automation, و ساعتها بتطلب من كل مطور مقدم تغيير في الكود انه يقدم مع التغيير بتاعه ال Tests اللي بتتأكد ان الكود شغال و بيأدي الوظيفة المطلوبة.
Our facebook Page
/ askdeveloper
On Sound Cloud
/ askdeveloper
Please Like & Subscribe - Наука та технологія
عظيم جدااا .. فكرة الفيديو القصير اللي بيركز على توبك معين ف وقت قليل دي عظيمة حاول تكتر منها جنب اللايف وعامة اي حاجة منك عظمة يا زعيم ^_^
كنت معجب جدا بفكره لايف مع القهوة، بس شايف ان نوعية الفيديوهات القصيرة افيد و فوكاسد اكتر، فعاش يعني.
ملحوظة: جرب تستخدم jump cuts في المونتاج + ان الexposure بتاع الكاميرا يكون متثبت على value واحدة مايبقاش auto عشان الباكجرواند ماتفضلش تقيد و تطفي مع حركتك..
طب استخدام لل eslint يعتبر جزء من الكود رفيو؟
شكرا جزيلاً على المجهود الواضح - استمر ربنا يبارك فيك
تسلم علي المجهود و الاضائة جميلة جدا ❤️
ما شاء الله - ممتاز
ننتظر منك أكثر - بالتوفيق
عظيم يابشمهندس معلومات زي دي من شخص زيك اتمني المحتوي ده كتير منه
ممكن تعمل فيديو عن الفرق بين هندسة قسم حاسبات
وكلية حاسبات ومعلومات لإنى فى ثانوية عامة ومحتار جدا بين الكليتين ومش عارف اختار
شكرا جدا يا باشمهندس
تنويع المحتوى دة مثري جدا ❤️
شكرا يا باش مهندس،،، فيديو جميل ارجوك كثر فيديوهات من هذا النوع
السلام عليكم.
لو كان لديا معاينة لكود كثير ووقت ضيق ( مضغوط بالوقت ) هل هناك حيلة لمعاينة الكود دون قرائة كله ؟
شكرا.
عاش والله استمر وياريت الحلقات القديمه اوى ومهمه تتعاد بالشكل ده و ربنا يوفقك
عاش جدا .. هو فيه جزء توارد افكار مافيش كلام لاني كنت بافكر في حاجة شبه كدا ممكن ابقى اشيرها معاك ان شاء الله 😊 .. ممكن بس لما تفتح البراوزر تعمل zoom 150% لانه الخطوط صغيرة جدا حتى على شاشة الكمبيوتر
Medhat Dawoud هو تفاصيل الكود نفسه مش مهمة علشان كده ما كانش فيه اهميه لإظهاره، الهدف كان الفكرة العامة
شكرا يا باشمهندس ع مجهودك .. ياريت تتكلم عن الBugs الي نخلي بالنا منها واحنا شغالين زي م حضرتك قولت زي ال CSRF , XFrame و غيره
video mofed gedan ya bashmohands shokrun gedan
شكرا.
ملاحظة : ممكن في المقطع القدمة تحسن تمركزك في الشاشة يعني يكون راسك أقرب بقليل للحافة العليا للشاشة.
Chadi_Ben اصلح الكرسي بس علشان هو بينزل لوحده 😂
@@bashmohandes هههههههههههههه
لله يعينك
@@bashmohandes والاضاءة كمان لو سمحت
شكرا لك على هذا الموضوع المهم وعلى اصلاح مشكلة الصوت
الفيديو فعلا جميل جدا ❤ وكنت عاوز اسالك ع شغل TECHNICAL SUPPORT &HELP DESK اي الJOB REQUIREMENT والشغلانة دي بيعملوا فيها اي 😂 بالتفصيل الممل ممكن تبقي تذكرها ف حلقة
مع اني still حمار ف حاجات كتير most of the time بس ال guide lines دي بتخليني cultured بطريقة او ب اخرى
الف شكر يا هندسة والله 🧡
لاتفهمني غلط بس يفضل تكتب ياعربي لوحده او انجليزي لوحده
المكس كذا غير مريح للعين.
طيب هل ال كود ريفيو للحجات الى بعملها تغير بس ولا ينفع يكون لو انا بعمل بروجكت كامل وبوريه لحد هايكون ده مفيد بردو ولا لأ ؟؟
انا مبرمج php وعايز اخش فى test automated ومش عارف ابدء منين
عظيم جدا الفيديو يا محمد كالعاده عاش جدا جدا
انا بس عندي ملحوظه صغيره, ال resolution بتاع الشاشه اللي بتسجل منها مخللي الخط طالع صغير أوي. أنا عشان أتاكد إخترت أعلى كواليتي وخليت الفيديو full screen
لو ينفع تكبر الخط شويه في البراوزر يبقى عظيم جدا
اوك انا لسه شايف ان تحت في كومنت بيقول نفس النقطه وانت رديت عليه ف كده تمام
يا هندسة والله العظيم بحبك 😂❤
اعمل فيديو عن طبيعة شغلك فى Microsoft 👍❤
انا سبت مايكروسوفت من أكثر من ٣ سنين.
@@bashmohandes
ربنا يوفقك يا هندسة
بشمهندس محمد ترشح اي لمذاكره database ؟
ربنا يبارك فيك يا كبيرنا 💖
video quality ↗️↗️↗️↗️
شكرا جزيلا .
Thanks!
Keep it up!
شكرًا مجهود رائع
انت راجل عظيم❤
النوع ده من المحتوى ممتاز اضغط نفسك شوية عاوزين نستفيد منك أكبر قدر ممكن 3>
عاش استمر💛
عظمه والله 👍
حلو جداُ موضوع الفيديو السؤال اللي نفسي اعرف اجابته لو الشركة مش مهتمه بالكود رفيو وبتبقي شايفة انه بيعطل و مش ذو قيمة اوي واللي بينعكس علي الفريق كله انه بيبقي عاوز يخلص اكبر قدر من المهام بدون حتي الالتزام بمعاير الكود او ختي solution structre ساعتها ايه بيبقي ايه الحل لان الدنيا بتوصل في الاخر انه خلاص ربنا يسهل ونبطق الكلام ده في المشروع الجاي وهكذا في كل مشروع
السلام عليكم ورحمة الله وبركاته
الله يعطيكم العافية وجزاكم الله خير
لو سمحت، ما هو نوع المايك لديك؟
Sami Almuhammady Blue Yeti
انا بحترمك جدا ❤
فيديو جميل. بينقل تجربه العمل في بيئات محترمه. هل ممكن فيديو عن git و git hub من وجهت نظر حد في السوق و دمجه مع الشغل الفعلي في الشركات
بس اعتقد العنوان ممكن يكون
مراجعه* الكود
تسلم على مجهودك استمر
مفيد جدا
keep going 👌👍
I will try my best
السلام عليكم ورحمة الله وبركاته انا طالب حاسبات و معلومات و مش عارف ابدأ منين و اخد اي في الكرسات ونفسي اتخصص في أمن المعلومات و مش عارف ابدأ منين بالظبط ارجو الرد في أسرع وقت و شكرا ل حضرتك ❤️❤️
thanks for your effort
Thanks , keep it up
beautiful
Thank you
❤
شكراً علي المعلومات، محتاج تنزل الكاميرا شويه.
هو انا محتاج كرسي جديد، علشان بينزل لوحده بعد شوية 😂
God bless you
👍👍👍
Can you reply with the Go project Github link, please?
Ayman M. Soliman
github.com/bashmohandes/go-askme
هي مشكلة ال code reviews انها بتسحب وقت جامد جدا.
يعني انت ساعات بتبقى كاتب الكود بطريقة معينة بس حد يطلب منك تعدل حاجة في الكود وتلاقي ان الحاجة دي حتاخد وقت كبيير جدا في ال refactoring
كمان كتابة التعليقات والرد عليها اصلا بياخد وقت كبيير جدا ... غالبا محدش بيرد على التعليق علطول, ممكن بعد ربع ساعة أو نص ساعة.
الكلام ده يؤدي ان ال feature بتاخد وقت طوييل جدا عشان تطلع.
حاجة كمان لاحظتها (خصوصا في ال startups), وهي ان كاتب الكود بيبقى تحت ضغظ عشان بيحس ان ال velocity بتاعته بتقل جامد بسبب الوقت المستهلك في ال code review وده حيأثر على الاداء بتاعه قدام ال project manager و ال team lead,, بينما اللي بيعاين الكود بيبقى على قلبه مراوح ومش فارقه معاه. وده غالبا سبب تاني بيخلّي كاتب الكود يبقى defensive جدا في الكود الاصلي بتاعه.
نظام ال pair programming ممكن يبقى بيحل شوية من المشاكل دي
سلسلة عن التيم ورك
God bless you
When I suggest things I write
Suggestion:.......
شكراً لمجهودك :)
أنا كان عندي سؤالين لو سمحت
1- هل بتشوف إن من مسؤولية الـ reviewer إنه يتأكد إن الـ PR ده بيـ match الـ business requirements بأكبر نسبة ممكنة، و بالتالي ممكن يـ reject PR من بتوع زمايله لمجرد إنه فيه business scenarios ناقصة. و لا بتشوف إن دي مسؤلية الـ testers كاملةً؟
2- هل المفروض الـ code review يكون بيتضمن الـ review بتاع الـ unit tests و بالتالي لو في missed unit tests جت في دماغ الـ reviewer و ملقيهاش implemented حتى لو كانت صغيرة أو مش هتعمل crashes بس ساعتها يـ reject الـ PR amid increase of unit tests coverage و بعد كدا الـ developer يصلحها و يرجعهاله؟
و شكراً لوقتك و اهتمامك :)
رديت عليك في Egyptian Geeks 👍
@@bashmohandesلو رديت عليها هنا عشان نستفيد كمان
السلام عليكم.... ممكن ترسلي واتساب لمبرمج محترف في صناعة برامج سطح مكتب...في مصر وشكرا جزيلا
God bless you