จุดประสงค์ของการตรวจสอบรหัสคืออะไร

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

ดังนั้นจุดประสงค์ของการตรวจสอบรหัสคืออะไร

มีสาเหตุหลายประการที่ทำให้คุณต้องการตรวจสอบโค้ด:

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

มีหลายกรณีธุรกิจสำหรับการทำรีวิว:

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

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

ความคิดเห็นเกี่ยวกับโค้ดที่มีเครื่องมือสำหรับการถ่ายทอดความรู้

• เมื่อนักพัฒนาตรวจสอบโค้ดของกันและกันพวกเขาจะคุ้นเคยกับทุกส่วนของระบบ สิ่งนี้จะลดปัจจัยบัสของโครงการและทำให้นักพัฒนามีประสิทธิภาพมากขึ้นเมื่อต้องทำการบำรุงรักษาในส่วนของระบบที่พวกเขาไม่ได้เขียน

• เมื่อโปรแกรมเมอร์อาวุโสทบทวนรหัสของรุ่นอาวุโสโปรแกรมเมอร์อาวุโสสามารถรับกลมิฉะนั้นเรียนรู้ผ่านประสบการณ์เท่านั้น นอกจากนี้ยังสามารถทำงานเป็นแก้ไขรหัสที่ซับซ้อนเกินไป

  การตรวจสอบรหัสอย่างละเอียดจะต้องมีการตรวจสอบบ่อยครั้งกับเอกสารต่างๆ มันเป็นวิธีที่ยอดเยี่ยมในการเรียนรู้ภาษาหรือ API

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

บทวิจารณ์โค้ดไม่ได้เกี่ยวกับ:

• ... การค้นหาข้อบกพร่อง นั่นคือสิ่งที่การทดสอบมีไว้สำหรับ มันจะยังคงเกิดขึ้นบ่อยครั้งที่การตรวจสอบรหัสพบปัญหาบางอย่าง

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

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

ฉันต้องการเพิ่มสองด้านที่ไม่ครอบคลุมโดยคำตอบที่ยอดเยี่ยมอื่น ๆ :

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

อีกเหตุผลที่ดีคือการพัฒนาความปลอดภัยที่ดีขึ้น หนึ่งต้องดูที่การข้ามไปของ Apple ล้มเหลว (บรรทัดของรหัสซ้ำโดยบังเอิญ) หรือHeartbleed bug (ความล้มเหลวขั้นพื้นฐานในการตรวจสอบอินพุต) เพื่อทำความเข้าใจถึงความสำคัญของการตรวจสอบรหัสที่เหมาะสมในวงจรการพัฒนาที่ปลอดภัย