คุณจัดการกับโค้ดที่ไม่ดีอย่างตั้งใจได้อย่างไร?

มีเรื่องราวมากมายเกี่ยวกับรหัสที่ไม่ดีโดยเจตนาไม่เพียง แต่ใน TheDailyWTF แต่ยังเกี่ยวกับ SO กรณีทั่วไปรวมถึง:

• การมีโครงสร้างที่สิ้นเปลืองเวลาเปล่าประโยชน์ (เช่นการวนลูปที่ว่างเปล่านับเป็นมูลค่ามหาศาล) ดังนั้นโปรแกรมเมอร์สามารถ "เร่งความเร็ว" แอปพลิเคชันได้อย่างง่ายดายโดยการลบออกเมื่อพวกเขาถูกมอบหมายให้
• จัดทำเอกสารที่ทำให้เข้าใจผิดโดยเจตนาผิดหรือไม่มีเอกสารเพื่อสร้างคำขอการสนับสนุนที่มีราคาแพง
• การสร้างข้อผิดพลาดอย่างรวดเร็วหรือแย่กว่านั้นคือการสร้างแม้ว่าทุกอย่างจะทำงานได้ดีการล็อกแอปพลิเคชันดังนั้นจึงจำเป็นต้องมีการโทรเพื่อขอรับการสนับสนุนที่มีราคาแพง

คะแนนเหล่านี้แสดงทัศนคติที่เป็นอันตรายไม่มากก็น้อย (แม้บางครั้งจะเกิดขึ้นโดยบังเอิญ) โดยเฉพาะอย่างยิ่งประเด็นแรกที่เกิดขึ้นค่อนข้างบ่อย

หนึ่งจะจัดการกับโครงสร้างดังกล่าวได้อย่างไร ไม่สนใจปัญหาหรือลบรหัสที่ละเมิดออก แจ้งผู้จัดการของพวกเขาหรือพูดกับคนที่แนะนำ "คุณสมบัติ"?

รหัสที่ไม่ดีส่วนใหญ่เกิดจากการขาดความเข้าใจและวิธีแก้ปัญหาคือการศึกษา

รหัสที่ไม่ดีโดยเจตนานั้นแตกต่างกันอย่างสิ้นเชิงเนื่องจากสิ่งที่ไม่เกี่ยวข้องอย่างสมบูรณ์กับประสบการณ์ของผู้เขียนโค้ดหรือส่วนที่เหลือของโครงการ ดังนั้นคุณต้องค้นหาสาเหตุที่พวกเขาก่อวินาศกรรมรหัสตามวัตถุประสงค์และจัดการกับปัญหานั้น ซึ่งหมายความว่าบ่อยครั้งที่ไม่มีการเมืองในสำนักงานและนั่นเป็นสถานการณ์ที่ไม่น่าพอใจสำหรับทุกคน

ฉันจะจัดการด้านการเมืองอย่างไรขึ้นอยู่กับสถานการณ์ (ไม่ระบุข้างต้น) หลายอย่าง ฉันจะจัดการกับรหัสได้อย่างไรก่อนอื่นตรวจสอบให้แน่ใจว่าฉันไม่ใช่คนเข้าใจผิด - ว่ามันเป็นรหัสที่ไม่ดีจริง ๆ - จากนั้นแก้ไขข้อบกพร่องที่เห็นได้ชัด ถ้าเป็นไปได้อย่างสมเหตุสมผลให้เขียนการทดสอบว่ารหัสไม่ดีจะล้มเหลว การตรวจสอบอีกครั้งว่าฉันเข้าใจถูกต้องจะหมายถึงการพูดคุยกับคนที่เขียนรหัส ควรทำด้วยวิธีที่สุภาพและสุภาพโดยไม่คาดคิดและอาจช่วยหาเหตุผล (การเมือง) ที่จำเป็นในภายหลัง

การจัดส่งสินค้าสำคัญกว่าความสมบูรณ์แบบของหอคอยงาช้าง แต่มีสองประเด็นที่ควรคำนึงถึง การแก้ไขข้อบกพร่องที่เห็นได้ชัดนั้นทำให้คุณได้ผลลัพธ์ 80% พร้อมความพยายาม 20% และผลไม้ชนิดแขวนต่ำชนิดนี้ไม่ค่อยน่าสนใจ แต่ที่สำคัญกว่านั้นคือหากคุณไม่ได้ระบุเหตุผล (ทางการเมือง) พื้นฐานรหัสที่ไม่ดีโดยเจตนาจะถูกเขียนขึ้นและก่อให้เกิดปัญหาเพิ่มเติม - และอาจป้องกันการจัดส่ง

ฉันไม่เคย (ใน 20 ปีที่แปลก) พบกับรหัสที่ไม่ดีอย่างตั้งใจ แต่ตัวอย่างที่คุณอ้างถึง (อย่างน้อยก็สำหรับฉัน แต่ IANAL) จะพยายามหลอกลวงทั้งนายจ้างหรือลูกค้าดังนั้นคุณอาจมีกฎหมาย ข้อผูกพันในการชี้ให้ผู้จัดการของคุณทราบ

ขึ้นอยู่กับวัฒนธรรมของ บริษัท บ่อยครั้งที่ไม่ใช่งานของคุณที่จะแก้ไขและทำความสะอาดโค้ดที่ไม่ดีทั้งหมด

จากCoders at Workความคิดของ Jamie Zawinski ในเรื่อง overengineering ซึ่งสามารถนำไปใช้ในสถานการณ์นี้:

ในตอนท้ายของวันจัดส่งสิ่งร่วมเพศ! เป็นเรื่องดีมากที่จะเขียนโค้ดของคุณใหม่อีกครั้งและทำให้มันดีขึ้นและในครั้งที่สามมันจะสวยมาก แต่นั่นไม่ใช่ประเด็น - คุณไม่ได้อยู่ที่นี่เพื่อเขียนโค้ด คุณมาที่นี่เพื่อจัดส่งผลิตภัณฑ์

มีโคเดอเรเตอร์และโค้ดที่ไม่ดีมากมายและพยายามแก้ไขทั้งหมดเมื่อคุณเจอโครงการ / งานปัจจุบันอาจเสียค่าใช้จ่ายหากผลิตภัณฑ์ "ใช้งานได้" บ่อยครั้งที่เราทุกคนเป็นแค่โปรแกรมเมอร์เทปพันสายไฟ

ดูเพิ่มเติมที่โพสต์ของ Joel Spolsky: The Duct Tape Programmer

ทัศนคตินั้นเป็นอาการของบางสิ่งที่แย่กว่านั้น

• ฝ่ายบริหารสนับสนุนการแข่งขันของนักพัฒนาหรือไม่?

• จิตวิญญาณของทีมอยู่ที่ไหน

• มีการมอบหมายงานจากบุคคลอื่นนอกเหนือจากทีมหรือไม่

• ...

ไม่ว่าในกรณีใดการลบรหัสที่กระทำผิดนั้นไม่เพียงพอ การบ่นกับผู้จัดการของเขาจะไม่ช่วยพัฒนาจิตวิญญาณของทีม

ฉันจะพยายามพูดกับบุคคลนั้นโดยตรงและพยายามเข้าใจว่าทำไมโดยการถามคำถามมากมายโดยไม่ตัดสินเขา ทีมงานทั้งหมดต้องทำโดยไม่ใช้ความก้าวร้าว

ในกรณีส่วนใหญ่พฤติกรรมเชิงสร้างสรรค์นั้นทำให้เกิดปัญหาจริง (ยิ่งแย่กว่านั้น) ภายใต้ความสว่างและจากนั้นคุณสามารถทำงานได้

ถ้ามันใช้ไม่ได้ผลจริงๆ ลบนักพัฒนานั้นออกจากทีม

ถ้าฉันคิดว่ามันเป็นความตั้งใจฉันอาจจะยิงผู้ชาย! หากเป็นผลมาจากคนที่ไม่ได้เป็นโปรแกรมเมอร์ที่ดีพอที่ฉันจะทำงานกับทักษะของเขา ถ้ามันถูกผลักจากด้านบนฉันอาจจะเริ่มหางานใหม่

หนึ่งจะจัดการกับโครงสร้างดังกล่าวได้อย่างไร ไม่สนใจปัญหาหรือลบรหัสที่ละเมิดออก แจ้งผู้จัดการของพวกเขาหรือพูดกับคนที่แนะนำ "คุณสมบัติ"?

ขึ้นอยู่กับบริบทหนึ่งในนั้นอาจเหมาะสมที่สุด ความเป็นไปได้อื่น ๆ รวมถึงการขอให้โอนไปยังโครงการอื่นรับงานใหม่และการกระทำต่าง ๆ ของศีลธรรมที่น่าสงสัยและ / หรือถูกกฎหมาย

อย่างไรก็ตามเนื่องจากเราไม่ทราบข้อเท็จจริงที่แท้จริงและคนที่เกี่ยวข้องจริง ๆ ไม่มีทางที่คนที่อยู่ในตำแหน่งที่คุณกำลังอธิบายควรให้ความสนใจมากกับคำแนะนำของเรา / 2 เซนต์ที่คุ้มค่า

หากนี่เป็นสถานการณ์จริงที่คุณกำลังพูดถึงมันอาจคุ้มค่าที่จะพูดกับผู้จัดการของคุณอย่างเงียบๆ ขอคำแนะนำจากพวกเขาเกี่ยวกับสิ่งที่คุณควรทำ หากเป็นไปได้พยายามที่จะทำให้การสนทนาเกี่ยวกับสิ่งที่คุณสามารถทำได้ / ควรทำไม่ได้เกี่ยวกับการชี้นิ้ว ถ้าเป็นไปได้อย่าตั้งชื่อ มีโอกาสพอสมควรที่ผู้จัดการของคุณมีปัญหาแล้ว

แต่ด้านพลิกคือคุณอาจจะระเบิดสัดส่วนนี้ ลองคิดดูให้ดีก่อนจะทำอะไร คิดถึงผลที่ตามมารวมถึงความเป็นไปได้ที่ขั้นตอนใด ๆ ที่คุณทำอาจย้อนกลับมาหาคุณ ... แย่มาก