คุณและผู้ตอบแบบสอบถามส่วนใหญ่มองว่านี่เป็นปัญหาการสื่อสารระหว่างเพื่อนร่วมงานสองคน แต่ฉันไม่คิดว่าจะเป็นเช่นนั้น สิ่งที่คุณอธิบายฟังดูคล้ายกับขั้นตอนการตรวจสอบโค้ดที่แย่มากกว่าสิ่งอื่นใด
ขั้นแรกคุณพูดถึงว่าเพื่อนร่วมงานของคุณเป็นคนที่สองในการบังคับบัญชาและคาดว่าเขาจะตรวจสอบรหัสของคุณ นั่นเป็นเพียงความผิด ตามคำนิยามการตรวจทานโค้ดเพียร์นั้นไม่ได้เป็นลำดับขั้นและแน่นอนว่าไม่ใช่เพียงแค่การค้นหาข้อบกพร่อง พวกเขายังสามารถมอบประสบการณ์การเรียนรู้ (สำหรับทุกคนที่เกี่ยวข้อง) โอกาสในการปฏิสัมพันธ์ทางสังคมและพิสูจน์ว่าเป็นเครื่องมือที่มีคุณค่าสำหรับการสร้างความเป็นเจ้าของรหัสโดยรวม คุณควรตรวจสอบรหัสของเขาเป็นครั้งคราวเรียนรู้จากเขาและแก้ไขเขาเมื่อเขาผิด (ไม่มีใครทำให้ถูกต้องทุกครั้ง)
นอกจากนี้คุณพูดถึงว่าเพื่อนร่วมงานของคุณทำการเปลี่ยนแปลงได้ทันที นั่นเป็นสิ่งที่ผิดเช่นกัน แต่แน่นอนคุณก็รู้อยู่แล้ว คุณคงไม่ถามคำถามนี้ถ้าวิธีการกังฟูของเขาไม่ใช่ปัญหา อย่างไรก็ตามฉันคิดว่าคุณกำลังมองหาทางออกในที่ที่ผิด เพื่อความซื่อสัตย์อย่างสมบูรณ์เพื่อนร่วมงานของคุณเตือนฉันเล็กน้อย ... ฉันและสิ่งที่ทำงานให้ฉันในสถานการณ์ที่คล้ายกันคือกระบวนการตรวจสอบที่กำหนดชัดเจนและมั่นคงและชุดเครื่องมือที่ยอดเยี่ยม คุณไม่ต้องการหยุดเพื่อนร่วมงานของคุณจากการตรวจสอบรหัสของคุณและขอให้เขาหยุดและพูดคุยกับคุณก่อนที่การเปลี่ยนแปลงเล็กน้อยจะไม่ทำงาน อาจนานสักครู่ แต่ในไม่ช้าเขาก็จะไปถึงจุดที่มันจะน่ารำคาญและคุณจะกลับมาที่จุดเริ่มต้นหรือแย่กว่านั้น: เขาจะหยุดตรวจสอบรหัสของคุณ
กุญแจสำคัญในการแก้ไขที่นี่อาจเป็นเครื่องมือตรวจสอบรหัสเพียร์ ฉันมักจะหลีกเลี่ยงคำแนะนำผลิตภัณฑ์ แต่สำหรับรีวิวรหัสCrucible ของ Atlassianเป็นเครื่องมือช่วยชีวิตจริงๆ สิ่งที่อาจดูเหมือนง่ายมากและมันคือ แต่นั่นไม่ได้หมายความว่ามันไม่น่ากลัว มันเชื่อมโยงกับพื้นที่เก็บข้อมูลของคุณและให้โอกาสคุณในการตรวจสอบแต่ละเซ็ตการแก้ไขไฟล์หรือกลุ่มของไฟล์ คุณไม่ได้รับการเปลี่ยนรหัสใด ๆ แต่คุณแสดงความคิดเห็นในทุกสิ่งที่ไม่ถูกต้อง และถ้าคุณต้องเปลี่ยนรหัสของคนอื่นคุณสามารถแสดงความคิดเห็นกับชุดการเปลี่ยนแปลงที่อธิบายการเปลี่ยนแปลงของคุณ วิดีโอแนะนำที่หน้าผลิตภัณฑ์ของ Crucible นั้นคุ้มค่ากับการดูถ้าคุณต้องการรายละเอียดเพิ่มเติม การกำหนดราคาของ Crucible นั้นไม่ได้มีไว้สำหรับทุกคน แต่มีเครื่องมือตรวจสอบแบบ peer ที่ใช้ได้ฟรีมากมาย สิ่งหนึ่งที่ฉันเคยร่วมงานและมีความสุขคือคณะกรรมการพิจารณาและฉันแน่ใจว่าคุณจะพบคนอื่น ๆ อีกมากมายด้วยการค้นหาโดย Google อย่างง่าย
เครื่องมืออะไรก็ตามที่คุณเลือกมันจะเปลี่ยนกระบวนการของคุณโดยสิ้นเชิง ไม่จำเป็นต้องหยุดออกจากเก้าอี้ขัดจังหวะบุคคลอื่นและหารือเกี่ยวกับการเปลี่ยนแปลง สิ่งที่คุณต้องทำคือกำหนดเวลาปิดทุกสัปดาห์และอ่านความคิดเห็น (สัปดาห์ละครั้งเป็นเพียงข้อเสนอแนะคุณรู้ตารางเวลาและกิจวัตรประจำวันของคุณดีกว่าที่ฉันทำ) ที่สำคัญกว่านั้นความเห็นหลักจะถูกเก็บไว้ในฐานข้อมูลและคุณสามารถเรียกดูได้ตลอดเวลา พวกเขาไม่ได้มีการพูดคุยกันรอบ ๆ ตัวทำน้ำเย็น กรณีการใช้งานที่ชื่นชอบสำหรับบทวิจารณ์เก่าคือเมื่อแนะนำสมาชิกในทีมคนใหม่ให้กับโค้ดเบสของเรา เป็นเรื่องที่ดีเสมอเมื่อฉันสามารถนำใครบางคนใหม่ ๆ ผ่าน codebase ชี้ให้เห็นว่าเราติดอยู่ตรงไหนเรามีความคิดเห็นที่แตกต่างกัน ฯลฯ
คุณพูดถึงว่าคุณไม่สามารถอ่านรหัสของเพื่อนร่วมงานคนนี้ได้เสมอ นั่นทำให้ฉันรู้ว่าคุณไม่มีมาตรฐานการเข้ารหัสทั่วไปและนั่นเป็นสิ่งที่ไม่ดี อีกครั้งคุณอาจเข้าหานี้เป็นปัญหาคนหรือคุณสามารถเข้าใกล้นี้เป็นปัญหากระบวนการและอีกครั้งฉันขอแนะนำอย่างยิ่งหลัง รวมทีมของคุณเข้าด้วยกันและปรับใช้รูปแบบการเข้ารหัสทั่วไปและกำหนดมาตรฐานโดยเร็วที่สุด ไม่สำคัญว่าถ้าคุณเลือกชุดมาตรฐานที่ใช้กันทั่วไปในระบบนิเวศการพัฒนาของคุณหรือคุณคิดขึ้นเอง สิ่งที่สำคัญจริงๆก็คือเพื่อให้มาตรฐานของคุณสอดคล้องและคุณยึดติดกับมาตรฐานเหล่านั้น มีเครื่องมือมากมายที่สามารถช่วยคุณได้ แต่นั่นเป็นการอภิปรายที่แตกต่างกันโดยสิ้นเชิง เพียงเพื่อให้คุณเริ่มต้น สิ่งที่ง่ายมากที่จะทำคือการขอใช้งานพร็อพเพอร์ตี้สไตล์ล่วงหน้าบนโค้ดของคุณ คุณสามารถเขียนรหัสต่อไปได้ตามที่คุณต้องการและปล่อยให้เครื่องมือ "แก้ไขมัน" โดยอัตโนมัติก่อนที่ใครจะเห็น
สุดท้ายคุณพูดถึงในความคิดเห็นว่าการจัดการไม่เชื่อว่าสาขา dev บุคคลมีความจำเป็น มีเหตุผลที่เราเรียกพวกเขาว่า "สาขา dev" และไม่ใช่ "สาขาการจัดการ" ฉันจะหยุดที่นี่เพราะไม่มีเหตุผลสำหรับเสียงโวยวายที่กำลังก่อตัวขึ้นในหัวของฉันที่จะออกไป
ทั้งหมดที่กล่าวมารู้ว่าฉันไม่สงสัยเพื่อนร่วมงานของคุณที่นี่เป็นความผิด นั่นไม่ใช่ประเด็นของฉันประเด็นของฉันคือกระบวนการพัฒนาทั้งหมดของคุณก็เป็นความผิดด้วยและนั่นคือสิ่งที่ง่ายต่อการแก้ไข ติดอาวุธด้วยตัวคุณเองด้วยเครื่องมือที่เหมาะสมสำรวจกระบวนการที่เป็นทางการและไม่เป็นทางการมากมายแล้วเลือกสิ่งที่เหมาะสมกับทีมของคุณ ในไม่ช้าคุณจะไปถึงจุดที่คุณจะรู้ว่า "ปัญหาของคนส่วนใหญ่" ไม่มีอยู่อีกต่อไป และโปรดอย่าฟังใครเลย (รวมถึงตัวคุณเอง) ที่ทำให้เกิด "เราเป็นทีมเล็ก ๆ เราไม่ต้องการข้อแก้ตัวทั้งหมด" ทีมนักพัฒนาที่มีความสามารถสามารถตั้งค่าเครื่องมือที่จำเป็นในเวลาน้อยกว่าหนึ่งสัปดาห์ทำให้ทุกอย่างเป็นไปโดยอัตโนมัติและไม่ต้องมองย้อนกลับไปอีกเลย
PS "ความเป็นเจ้าของรหัส" เป็นคำที่คลุมเครือถกเถียงกันอย่างต่อเนื่องและมันหมายถึงสิ่งที่แตกต่างกันสำหรับคนที่แตกต่างกัน คุณสามารถค้นหาคอลเลกชันที่ยอดเยี่ยมที่สุดของที่แตกต่างกัน (และบางครั้งถือ) ความคิดเห็นเกี่ยวกับC2