มันเป็นไปได้หรือไม่ที่จะฟอร์แมตโค้ดนักพัฒนาซอฟต์แวร์อื่นในขณะที่แก้ไข / เพิ่มโมดูล?

ในขณะที่พัฒนาในบรรยากาศของกลุ่มและการเพิ่มหรือแก้ไขคุณสมบัติในบางรหัสฐาน ถือว่าเป็นการล่วงละเมิดหรือไม่สุภาพที่จะทำการฟอร์แมทโค้ดนักพัฒนาก่อนหน้านี้เพื่อให้ได้มาตรฐานการเข้ารหัสปัจจุบันหรือไม่? ฉันเข้าใจว่ามาตรฐานมีการเปลี่ยนแปลงและอาจจะมีการเปลี่ยนแปลงต่อไป แต่มีใครบ้างที่ทำให้คุณขุ่นเคืองหากมีใครบางคนเข้ามาและเปลี่ยนรูปแบบรหัสของคุณ?

เพื่อความชัดเจนฉันไม่ได้พูดถึงการเปลี่ยนแปลงตรรกะใด ๆ เพียงแค่ยุ่งกับแท็บและช่องว่างและเช่นนั้น

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

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

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

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

แก้ไข: ในบริบทของคำถามนี้การจัดรูปแบบตามมาตรฐานมีความเหมาะสม ในกรณีที่ไม่มีมาตรฐานฉันขอแนะนำให้สนับสนุนมาตรฐานและไม่ทำการฟอร์แมตใหม่จนกว่าจะมีมาตรฐานสำหรับรูปแบบ การจัดรูปแบบใหม่ตามรสนิยม / มาตรฐานส่วนบุคคลไม่ควรกระทำด้วยรหัสที่เป็นของโครงการ

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

ฉันทำสิ่งนี้ตลอดเวลา รหัสเก่าควรจะอยู่ในมาตรฐานเดียวกันกับรหัสใหม่และถ้าคุณไม่แก้ไขในขณะที่คุณกำลังทำงานอยู่จะไม่มีใครทำ ฉันคิดว่าสิ่งนี้มีความสำคัญภายใต้กฎลูกเสือ

ฉันคิดว่านี่เป็นวิธีปฏิบัติที่ดีและเป็นส่วนที่จำเป็นในการบำรุงรักษาโค้ด

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

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

ใช่. โปรด "แก้ไข" รหัสตามที่เห็นสมควร เช่นเดียวกับโปรแกรมเมอร์อย่างจริงจังบอกไว้ในหนังสือของพวกเขาThe Pragmatic Programmerไม่มีหน้าต่างแตก หากรหัสไม่ถึงเกณฑ์ฉันถือว่ามันเป็นหน้าต่างที่เสียหาย

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

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

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

เนื่องจากคุณกำลังพูดถึงปัญหา "การจัดรูปแบบ" อย่างหมดจด (หมายถึงเราไม่ได้แก้ไขข้อบกพร่อง แต่ทำให้ดูสวยเป็นมาตรฐานของคุณเอง) ฉันคิดว่ามันขึ้นอยู่กับว่าคนดั้งเดิมยังคงรักษารหัสหรือไม่

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

ฉันทำงานในโครงการหนึ่งครั้งกับผู้พัฒนาที่เป็นเจ้าของมาก ในช่วงหลายปีที่ผ่านมาฉันได้พัฒนาวิธีการจัดรูปแบบของโค้ดที่ฉันรู้สึกว่าอ่านง่ายลดความผิดพลาดโดยนัยและการจัดทำเอกสารด้วยตนเองน้อยลง ในทางกลับกันผู้ชายคนนี้ชอบใช้คุณสมบัติโดยนัยทั้งหมดที่มีเส้นยาวซึ่งกว้าง 300 ตัวอักษรดังนั้นคุณต้องมีจอภาพขนาด 30 นิ้วให้อ่านเพราะเขาเชื่อว่าการนับบรรทัดนั้นสำคัญกว่าการอ่านได้เขาใช้เวลาครึ่งวัน พัดผ่านรหัสของฉันเปลี่ยนเป็น "มาตรฐานที่ต้องการ" ของเขา ... ในขณะที่ฉันยังคงพัฒนาในแบบคู่ขนานฉันมาในเช้าวันรุ่งขึ้นเพื่อหางานที่มีมูลค่าสองวันในรูปแบบที่ยุ่งเหยิงของเขามันหยาบคายและเสียเวลา

ตอนนี้ถ้านักพัฒนาหายไปและคุณมี "รูปแบบที่ดีกว่า" ไปเลย

ทำการฟอร์แมตโค้ดทุกครั้งถ้า IDE ของคุณสามารถทำได้

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

ตัวอย่างเช่นใน eclipse คุณสามารถรันตัวจัดรูปแบบก่อนและจัดการการนำเข้าสำหรับฐานรหัสทั้งหมด จากนั้นอย่าลืม ctrl + alt + f ก่อนที่คุณจะบันทึก