سؤال

أنا أستخدم restharper في العمل. بعض زملائي لا.

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

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

أعتقد أنني أرى خياراتي على نطاق واسع

1) تاريخ التغييرات في التعليمات البرمجية المصدر ضروري للصيانة. التغيير بأقل قدر ممكن، أو الرجل التالي لن يكون له أمل في معرفة ما تغير. من يهتم بكمونة غير قابلة للوصول، واستخدام لا لزوم له من .tosting () إلخ. على أي حال.

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

3) البرتقال هو أحمر فقط ولكن أخف وزنا. F12 ثم Alt + Enter حتى أخضر.

4) ننسى البرتقالي، انظر إلى وظيفة خط الوحش 700 الوحش. ما هذا عام 1997؟ الوقت للحصول على مشغول ... وإذا كنت قد حصلت على الوقت، فقدم زميلك إلى صديقنا الحميم ومعلمه السيد فاولر.

أميل إلى الإقلاع بين الخيارات اعتمادا على مقدار الوقت الذي دفعته، إلى أي مدى أنا الآن مسؤول عن التعليمات البرمجية، وكيف يبدو رمز التعريف (الذي يمكن أن يجعلني أذهب لمدة 1 أو 4 عادة).

يبدو أن أحد الخيارات الأربعة يجب أن يكون هو الشخص الذي نسعى جاهدين، لكن ليس لدي أي فكرة

هل كانت مفيدة؟

المحلول

"Leave the campsite cleaner than you found it."

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

نصائح أخرى

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

ولكن إذا كنت تستطيع أن توافق جميعا، ثم نعم. تنظيف الكود كما تذهب.

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

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

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

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

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

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

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

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

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

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

كما قال غالبية الناس بالفعل، نعم، أفضل لإعادة تعادل واتركها مدها.

يساعد Refactoring الجميع على التحسن، ويجب أن يحصل الجميع على أدوات إعادة تكوين (Coderush بالنسبة لي).

ومع ذلك، إذا لم يتم مشاركة زملائك في حب إعادة صبط، فهذه فرصة جيدة لك لتنوير لهم :)

مرخصة بموجب: CC-BY-SA مع الإسناد
لا تنتمي إلى StackOverflow
scroll top