คำถามติดแท็ก code-reviews

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

37 code-quality  code-reviews  communication 

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

36 development-process  code-reviews  professionalism 

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

33 code-reviews 

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

33 continuous-integration  code-reviews 

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

32 teamwork  code-reviews  estimation  project-planning 

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

32 development-process  code-quality  code-reviews 

คุณทำอะไรเมื่อคุณทำงานกับคนที่มีแนวโน้มที่จะเขียนโค้ดไม่ดีโวหาร รหัสที่ผมพูดถึงมักจะเป็นเทคนิคที่ถูกต้องมีโครงสร้างที่สมเหตุสมผลและแม้กระทั่งอาจจะสง่างามอัลกอริทึม แต่มันก็ดูน่าเกลียด เรามี: ส่วนผสมของอนุสัญญาการตั้งชื่อและชื่อที่แตกต่างกัน ( underscore_styleและcamelCaseและUpperCamelและCAPSทั้งหมดใช้มากหรือน้อยโดยการสุ่มกับตัวแปรต่าง ๆ ในฟังก์ชั่นเดียวกัน) ระยะห่างที่แปลกประหลาดและไม่สอดคล้องกันเช่น Functioncall (arg1 ,arg2,arg3 ); คำที่สะกดผิดจำนวนมากในความคิดเห็นและชื่อตัวแปร เรามีระบบตรวจสอบรหัสที่ดีที่ฉันทำงานอยู่ดังนั้นเราจะได้ตรวจสอบและแก้ไขสิ่งที่เลวร้ายที่สุด อย่างไรก็ตามมีความรู้สึกเล็กน้อยในการส่งบทวิจารณ์รหัสที่ประกอบด้วย 50 บรรทัดของ "เพิ่มช่องว่างที่นี่สะกด 'itarator' อย่างถูกต้องเปลี่ยนการใช้อักษรตัวพิมพ์ใหญ่นี้ ฯลฯ คุณจะสนับสนุนให้บุคคลนี้ระมัดระวังและสอดคล้องกับรายละเอียดเหล่านี้มากขึ้นอย่างไร

30 coding-style  code-quality  teamwork  code-reviews 

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

29 version-control  productivity  code-reviews 

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

28 code-reviews  review 

ฉันจะไม่เรียกตัวเองว่าเป็นนักแสดงระดับแนวหน้า แต่เป็นคนที่ค่อนข้างมีประสบการณ์ ฉันพยายามที่จะรักษาคุณภาพของโค้ดให้อยู่ในระดับสูงและมองหาวิธีการปรับปรุงสไตล์การเขียนโค้ดของฉันอยู่เสมอพยายามที่จะทำให้โค้ดมีประสิทธิภาพอ่านง่ายและสม่ำเสมอรวมทั้งสนับสนุนให้ทีมทำตามรูปแบบและวิธีการ ฉันยังเข้าใจถึงความจำเป็นของความสมดุลระหว่างทั้งคุณภาพและความเร็ว เพื่อให้บรรลุเป้าหมายนี้ฉันได้แนะนำแนวคิดเรื่องการตรวจสอบโดยเพื่อนร่วมทีมของฉัน ยกนิ้วขึ้นสองนิ้วใน GitHub ดึงคำขอสำหรับการผสาน ยอดเยี่ยม - แต่ไม่ใช่ในความคิดของฉันหากปราศจากอาการสะอึก ฉันมักจะเห็นความเห็นทบทวนเพื่อนร่วมงานจากเพื่อนร่วมงานเดียวกันเช่น - คงจะเป็นการดีถ้าจะเพิ่มช่องว่างหลังจากนั้น <INSERT SOMETHING HERE> บรรทัดที่ไม่ต้องการพิเศษระหว่างเมธอด ควรใช้การหยุดแบบเต็มเมื่อสิ้นสุดความคิดเห็นใน docblock ตอนนี้จากมุมมองของฉัน - ผู้ตรวจทานมองเผินๆเกี่ยวกับความสวยงามของรหัส - และไม่ได้ทำการตรวจสอบโค้ด การตรวจสอบรหัสเครื่องสำอางทำให้ฉันมีความคิดที่หยิ่งผยอง / ชนชั้นนำ มันขาดสาร แต่คุณไม่สามารถจริงๆเถียงมากเกินไปกับมันเพราะวิจารณ์เป็นเทคนิคที่ถูกต้อง ฉันจะค่อนข้างจะเห็นน้อยกว่าความคิดเห็นประเภทข้างต้นและความคิดเห็นเพิ่มเติมดังนี้: คุณสามารถลดความซับซ้อนของวัฏจักรโดย ... ออกจากต้นและหลีกเลี่ยงถ้า / อื่น สรุปแบบสอบถาม DB ของคุณไปยังที่เก็บ ตรรกะนี้ไม่ได้อยู่ที่นี่จริงๆ อย่าทำซ้ำตัวเอง - นามธรรมและนำมาใช้ใหม่ อะไรจะเกิดขึ้นถ้าXถูกส่งผ่านเป็นอาร์กิวเมนต์วิธีY? การทดสอบหน่วยสำหรับสิ่งนี้อยู่ที่ไหน ฉันพบว่าเป็นคนประเภทเดียวกันที่ให้ความเห็นประเภทเครื่องสำอางและผู้คนประเภทเดียวกันซึ่งในความคิดเห็นของฉันให้ความเห็นแบบ "Quality & Logic" อะไรคือวิธีที่ถูกต้องในการตรวจสอบโดยเพื่อน …

27 code-quality  code-reviews  leadership 

ฉันกำลังมองหาอุปมาอุปไมยหรืออุปมาอุปไมยที่สามารถอธิบายปัญหาของการเขียนโปรแกรมคัดลอกแปะกับผู้ที่ไม่ใช่โปรแกรมเมอร์ บางครั้งฉันทำการตรวจสอบโค้ด / ระบบสำหรับผู้มีโอกาสเป็นลูกค้าและหนึ่งในปัญหาทั่วไปที่ฉันเห็นคือโค้ดคัดลอกวางจำนวนมากมายทั่วฐานรหัสของพวกเขา มันเป็นสิ่งที่ฉันมักจะพูดถึงในบทวิจารณ์และทุกครั้งที่ฉันต้องอธิบายว่าทำไมถึงมีปัญหา (นี่เป็นเรื่องยากโดยเฉพาะอย่างยิ่งกับลูกค้าที่รู้เกี่ยวกับการเขียนโปรแกรมมากพอที่จะเข้าใจว่าการใช้ซ้ำนั้นเป็นสิ่งที่ดี การคัดลอกวางไม่ใช่รูปแบบการใช้ซ้ำที่ดี) เห็นได้ชัดว่าฉันสามารถ (และทำ) อธิบายปัญหาในแง่ของการบำรุงรักษาโค้ด แต่มันจะดีถ้าได้มีการเปรียบเทียบที่ดีและกระชับสำหรับปัญหานี้ที่จะตีบ้านด้วยโปรแกรมเมอร์ที่ไม่ใช่ โบนัสหากการเปรียบเทียบแสดงให้เห็นว่าเหตุใดการค้นหาและแทนที่ไม่ใช่วิธีที่มีประสิทธิภาพสำหรับปัญหานี้ ข้อเสนอแนะใด ๆ เพื่อชี้แจง (ตามคำตอบของ Jaroslav ด้านล่าง) - ฉันไม่ได้พูดถึงการใช้ข้อมูลโค้ดที่นี่; สิ่งที่ฉันเห็น (รบกวนบ่อย) คือการคัดลอกและวางโค้ดอันกว้างใหญ่หรือโค้ดสิบบรรทัดเพื่อรับข้อมูลผู้ใช้บางส่วน (สมบูรณ์ด้วยคิวรี SQL แบบอินไลน์) วางลงในหน้า PHP หรือ ASP.NET หลายสิบหน้า ดังนั้นซ้ำรหัสจากที่อื่นในโครงการเดียวกัน อัปเดต: มีคำตอบที่ดีมาก ๆ อยู่ที่นี่ ฉันได้อธิบายในความคิดเห็นว่าทำไมฉันถึงเลือกคำตอบของ Scott Whitlock แต่ฉันก็อยากจะแนะนำคำตอบแบบ whatsisname อย่างมากถ้าคุณติดต่อกับลูกค้าที่คุ้นเคยกับการผลิตเลย

27 code-reviews  copy-paste-programming 

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

26 development-process  code-reviews 

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

26 code-reviews  team-leader 

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

25 testing  code-reviews 

ฉันได้รับบทบาทเพื่อปรับปรุงการพัฒนาใน บริษัท ของเรา สิ่งแรกที่ฉันต้องการเริ่มต้นคือการตรวจสอบโค้ดเนื่องจากไม่เคยทำที่นี่มาก่อน บริษัท ของเรามีโปรแกรมเมอร์ 3 คน ฉันเป็นโปรแกรมเมอร์เว็บภาษาที่ฉันรู้จักส่วนใหญ่เป็น PHP, ActionScript และ JavaScript นักพัฒนาอีก 2 คนเขียนแอปพลิเคชันภายในใน VB.net เราทำการตรวจสอบโค้ดมาสองสามสัปดาห์แล้ว ฉันพบว่ามันยากที่จะเข้าใจรหัส VB ดังนั้นเมื่อพวกเขาพูดในสิ่งที่มันทำส่วนใหญ่ฉันก็ต้องใช้คำของพวกเขาสำหรับมัน หากฉันเห็นบางสิ่งที่ผิดปกติฉันจะอธิบายความคิดเห็นของฉันและอธิบายว่าฉันจะจัดการกับมันอย่างไรในภาษาใดภาษาหนึ่งที่ฉันรู้จัก บางครั้งคำแนะนำของฉันก็ยินดี แต่หลายครั้งฉันก็บอกว่า "นี่เป็นวิธีที่ดีที่สุดในการทำมันในภาษานี้ " หรือ "ที่ไม่ได้ใช้กับภาษานี้ " หรือสิ่งที่คล้ายกันในลักษณะนั้น นี่อาจเป็นจริง แต่ไม่รู้ภาษาฉันไม่แน่ใจว่าจะยืนยันหรือปฏิเสธข้อเรียกร้องเหล่านี้ได้อย่างไร ฉันรู้วิธีแก้ปัญหาที่เป็นไปได้อย่างหนึ่งคือการเรียนรู้ vb เพื่อให้ฉันสามารถทำรีวิวโค้ดได้ดีขึ้น ฉันไม่มีความสนใจในการเรียนรู้ vb (โดยเฉพาะอย่างยิ่งเมื่อฉันมีรายการเทคโนโลยีอื่น ๆ ที่ฉันพยายามเรียนรู้สำหรับโครงการของตัวเอง) และต้องการเก็บไว้เป็นทางเลือกสุดท้าย แต่เป็นตัวเลือก ความคิดที่มาหาฉันก็คือพวกเขาทั้งสองมีความสนใจใน C # และอื่น ๆ ฉันมันเกี่ยวข้องกับพวกเขาเพราะ. net และเมื่อเทียบกับฉันเพราะมันคล้ายกับภาษาที่ฉันรู้ …

25 teamwork  code-reviews