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

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

10 code-quality  teamwork 

ปิด. คำถามนี้เป็นคำถามปิดหัวข้อ ไม่ยอมรับคำตอบในขณะนี้ ต้องการปรับปรุงคำถามนี้หรือไม่ อัปเดตคำถามเพื่อให้เป็นหัวข้อสำหรับ Software Engineering Stack Exchange ปิดให้บริการใน6 ปีที่ผ่านมา หนังสือ Java ของฉันหลายเล่มมีอายุ 5 - 10 ปี มันยังช่วยอ่านหรือไม่ฉันควรใช้บางอย่างภายใน 2 ปี

10 books  code-quality 

พื้นหลังเล็กน้อย: ในฐานะผู้นำทีมฉันใช้ NDepend ประมาณสัปดาห์ละครั้งเพื่อตรวจสอบคุณภาพรหัสของเรา โดยเฉพาะอย่างยิ่งการครอบคลุมการทดสอบบรรทัดของโค้ดและการวัดความซับซ้อนตามวัฏจักรมีค่าสำหรับฉัน แต่เมื่อมันมาถึงการปรับระดับและรอบการพึ่งพาฉันเป็นบิต ... กังวลดี Patrick Smacchia มีบล็อกโพสต์ที่ดีซึ่งอธิบายถึงเป้าหมายของการปรับระดับ เพื่อความชัดเจน: ภายใต้ "วงจรการพึ่งพา" ฉันเข้าใจการอ้างอิงแบบวงกลมระหว่างสองเนมสเปซ ขณะนี้ฉันกำลังทำงานบนกรอบ GUI ที่ใช้ Windows CE สำหรับเครื่องมือฝังตัว - แค่คิดถึงแพลตฟอร์มกราฟิก Android แต่สำหรับเครื่องมือต่ำสุด เฟรมเวิร์กเป็นชุดประกอบเดี่ยวที่มีรหัสประมาณ 50,000 บรรทัด (ไม่รวมการทดสอบ) เฟรมเวิร์กแบ่งออกเป็นเนมสเปซต่อไปนี้: ระบบนำทางหลัก & ระบบย่อยเมนู ระบบย่อยของหน้าจอ (ผู้นำเสนอ / การดู / ... ) ชั้นควบคุม / Widget วันนี้ฉันใช้เวลาครึ่งวันในการพยายามนำรหัสไปสู่ระดับที่เหมาะสม [ขอบคุณ Resharper ที่ไม่มีปัญหาโดยทั่วไป] แต่ในทุกกรณีมีบางวงจรการพึ่งพาอาศัยอยู่ ดังนั้นคำถามของฉัน: คุณปฏิบัติตามกฎ …

10 code-quality 

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

10 code-quality  sdlc  product-management 

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

10 code-quality  collaboration  code-reviews 

(ฉันเคยเห็นคำถามนี้แต่คำตอบแรกเป็นเรื่องเกี่ยวกับคุณสมบัติอัตโนมัติมากกว่าเกี่ยวกับการออกแบบและคำตอบที่สองบอกว่าซ่อนรหัสการจัดเก็บข้อมูลจากผู้บริโภคซึ่งฉันไม่แน่ใจว่าเป็นสิ่งที่ฉันต้องการ / รหัสของฉันทำ ดังนั้นฉันต้องการฟังความเห็นอื่น ๆ ) ฉันมีเอนทิตีที่คล้ายกันมากสองรายการHolidayDiscountและRentalDiscountนั่นคือส่วนลดที่มีความยาวเนื่องจาก 'ถ้าอย่างน้อยที่สุดจะnumberOfDaysใช้percentส่วนลดได้' ตารางมี fks ไปยังเอนทิตีพาเรนต์ที่แตกต่างกันและใช้ในสถานที่ต่าง ๆ แต่ที่ที่ใช้จะมีตรรกะทั่วไปเพื่อรับส่วนลดสูงสุดที่เกี่ยวข้อง ตัวอย่างเช่น a HolidayOfferมีจำนวนHolidayDiscountsและเมื่อคำนวณต้นทุนเราต้องหาส่วนลดที่เกี่ยวข้อง RentalDiscountsเหมือนกันสำหรับเช่าและ เนื่องจากตรรกะเหมือนกันฉันต้องการเก็บไว้ในที่เดียว นั่นคือสิ่งที่วิธีต่อไปนี้เพรดิเคตและตัวเปรียบเทียบทำ: Optional<LengthDiscount> getMaxApplicableLengthDiscount(List<LengthDiscount> discounts, int daysToStay) { if (discounts.isEmpty()) { return Optional.empty(); } return discounts.stream() .filter(new DiscountIsApplicablePredicate(daysToStay)) .max(new DiscountMinDaysComparator()); } public class DiscountIsApplicablePredicate implements Predicate<LengthDiscount> { private final long daysToStay; public DiscountIsApplicablePredicate(long …

10 java  object-oriented  object-oriented-design  code-quality 

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

10 php  code-quality  mvc 

เรามีรหัสนี้ซึ่งเมื่อทำให้ง่ายขึ้นจะมีลักษณะเช่นนี้: public class Room { public Client Client { get; set; } public long ClientId { get { return Client == null ? 0 : Client.Id; } } } public class Client { public long Id { get; set; } } ตอนนี้เรามีสามมุมมอง 1) นี่เป็นรหัสที่ดีเพราะClientควรตั้งค่าคุณสมบัติ (เช่นไม่ใช่โมฆะ) ดังนั้นClient == nullจะไม่เกิดขึ้นและค่ารหัส0แสดงถึงรหัสเท็จอย่างไรก็ตาม (นี่คือความเห็นของผู้เขียนรหัส ;-)) …

9 c#  code-quality  null 

ฉันได้ยินมาว่าขอแนะนำให้ตรวจสอบข้อโต้แย้งของวิธีการสาธารณะ: หนึ่งควรตรวจสอบโมฆะถ้าเขาไม่คาดหวังโมฆะ? เมธอดควรตรวจสอบความถูกต้องของพารามิเตอร์หรือไม่ MSDN - CA1062: ตรวจสอบข้อโต้แย้งของวิธีสาธารณะ (ฉันมีพื้นหลัง. NET แต่คำถามไม่เฉพาะ C #) แรงจูงใจสามารถเข้าใจได้ หากโมดูลจะถูกใช้ในทางที่ผิดเราต้องการที่จะโยนข้อยกเว้นทันทีแทนที่จะเป็นพฤติกรรมที่คาดเดาไม่ได้ สิ่งที่รบกวนจิตใจฉันคือข้อโต้แย้งที่ผิดไม่ใช่ข้อผิดพลาดเดียวที่สามารถทำได้ในขณะที่ใช้โมดูล นี่คือสถานการณ์ข้อผิดพลาดบางประการที่เราจำเป็นต้องเพิ่มตรรกะการตรวจสอบหากเราทำตามคำแนะนำและไม่ต้องการการเพิ่มระดับข้อผิดพลาด: สายเรียกเข้า - อาร์กิวเมนต์ที่ไม่คาดคิด สายเรียกเข้า - โมดูลอยู่ในสถานะไม่ถูกต้อง โทรภายนอก - ส่งคืนผลลัพธ์ที่ไม่คาดคิด การโทรภายนอก - ผลข้างเคียงที่ไม่คาดคิด (การป้อนสองครั้งในโมดูลการโทร, การแยกสถานะการอ้างอิงอื่น ๆ ) ฉันได้ลองคำนึงถึงเงื่อนไขเหล่านี้และเขียนโมดูลง่าย ๆ ด้วยวิธีการหนึ่ง (ขออภัยไม่ใช่ -C # guys): public sealed class Room { private readonly IDoorFactory _doorFactory; private bool …

9 design  code-quality  coding-standards  exceptions  error-handling 

ใน C # usingคำสั่งจะถูกใช้เพื่อกำจัดทรัพยากรอย่างไม่ถูกต้องโดยไม่ต้องรอการเก็บขยะ ตัวอย่างเช่นอาจใช้เพื่อ: กำจัดคำสั่งหรือการเชื่อมต่อ SQL ปิดสตรีมปล่อยแหล่งข้อมูลอ้างอิงเช่นไฟล์ องค์ประกอบ GDI + ฟรี เป็นต้น ฉันสังเกตเห็นว่าusingมีการใช้งานมากขึ้นในกรณีที่ไม่มีสิ่งใดที่จะกำจัด แต่ที่มันสะดวกกว่าสำหรับผู้โทรที่จะเขียนusingบล็อกแทนที่จะเป็นสองคำสั่งแยกกัน ตัวอย่าง: MiniProfilerเขียนโดยทีม Stack Overflow ใช้usingเพื่อแสดงถึงบล็อกในโปรไฟล์: using (profiler.Step("Name goes here")) { this.DoSomethingUseful(i - 1); } ทางเลือกหนึ่งที่จะมีสองช่วงตึก: var p = profiler.Start("Name goes here"); this.DoSomethingUseful(i - 1); profiler.Stop(p); อีกวิธีหนึ่งคือการใช้การกระทำ: profiler.Step("Name goes here", () => this.DoSomethingUseful(i - 1)); ASP.NET …

9 c#  .net  code-quality  readability 

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

9 unit-testing  code-quality  code-smell 

ไม่กี่เดือนที่ผ่านมาตั้งแต่ฉันเริ่มต้นตำแหน่งในฐานะนักพัฒนาซอฟต์แวร์ระดับเริ่มต้น ตอนนี้ฉันผ่านโค้งการเรียนรู้บางอย่าง (เช่นภาษาศัพท์แสงไวยากรณ์ของ VB และ C #) ฉันเริ่มมุ่งเน้นที่หัวข้อลึกลับมากขึ้นเพื่อเขียนซอฟต์แวร์ที่ดีขึ้น คำถามง่าย ๆ ที่ฉันนำเสนอให้เพื่อนร่วมงานได้รับการตอบสนองด้วยการ ในขณะที่ฉันเคารพเพื่อนร่วมงานคนนี้ฉันไม่เห็นด้วยว่านี่เป็น "สิ่งผิด" ที่จะมุ่งเน้น นี่คือรหัส (ใน VB) และตามด้วยคำถาม หมายเหตุ: Function GenerateAlert () จะคืนค่าจำนวนเต็ม Dim alertID as Integer = GenerateAlert() _errorDictionary.Add(argErrorID, NewErrorInfo(Now(), alertID)) VS ... _errorDictionary.Add(argErrorID, New ErrorInfo(Now(), GenerateAlert())) ฉันเขียนตอนหลังและเขียนใหม่ด้วย "Dim alertID" เพื่อให้คนอื่นสามารถอ่านได้ง่ายขึ้น แต่นี่คือความกังวลและคำถามของฉัน: เราควรเขียนสิ่งนี้ด้วย Dim AlertID จริงๆแล้วมันต้องใช้หน่วยความจำมากขึ้น จำกัด แต่มากขึ้นและควรเรียกวิธีนี้หลาย ๆ …

9 programming-practices  code-quality  garbage-collection 

ฉันแค่อยากรู้ว่าอะไรคือความแตกต่างระหว่างการวิเคราะห์โค้ดแบบสแตติกและการตรวจสอบโค้ด แต่ละคนทำกันอย่างไร โดยเฉพาะอย่างยิ่งมีเครื่องมืออะไรบ้างสำหรับการตรวจสอบโค้ด / การวิเคราะห์แบบคงที่ของ PHP ในวันนี้? ฉันต้องการทราบเกี่ยวกับเครื่องมือที่ดีสำหรับการตรวจสอบโค้ดสำหรับภาษาใด ๆ

9 code-quality  terminology  code-reviews  quality 

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

9 design  code-quality  code-reviews  code-smell 

ฉันเพิ่งเข้าร่วมทีมพัฒนาขนาดเล็กที่ค่อนข้างได้ทำงานในโครงการเป็นเวลาหลายเดือนถ้าไม่ใช่ปี เช่นเดียวกับนักพัฒนาส่วนใหญ่ที่เข้าร่วมโครงการฉันใช้เวลาสองสามวันแรกในการตรวจสอบ codebase ของโครงการ โครงการ (บรรทัดภายในของโปรแกรมประยุกต์ทางธุรกิจ ASP.NET WebForms ขนาดกลางถึงขนาดใหญ่) คือการขาดคำที่มีความหมายมากขึ้นหายนะ มีสามปัญหาที่เห็นได้ชัดเจนทันทีกับมาตรฐานการเข้ารหัส: มาตรฐานหลวมมาก มันอธิบายเพิ่มเติมเกี่ยวกับสิ่งที่ไม่ควรทำ (อย่าใช้สัญกรณ์ฮังการี ฯลฯ ) มากกว่าสิ่งที่ต้องทำ มาตรฐานไม่ได้ปฏิบัติตามเสมอไป มีความไม่สอดคล้องกันด้วยรหัสการจัดรูปแบบที่มีทุกที่ มาตรฐานไม่เป็นไปตามหลักเกณฑ์ลักษณะของ Microsoft ในความคิดของฉันไม่มีค่าเบี่ยงเบนจากแนวทางที่กำหนดโดยนักพัฒนาของกรอบงานและผู้มีส่วนร่วมที่ใหญ่ที่สุดในการกำหนดภาษา สำหรับจุดที่ 3 บางทีมันทำให้ฉันกังวลมากขึ้นเพราะฉันใช้เวลาในการรับMCPDของฉันโดยให้ความสำคัญกับเว็บแอปพลิเคชัน (โดยเฉพาะ ASP.NET) ฉันยังเป็น Microsoft Certified Professional คนเดียวในทีม เนื่องจากสิ่งที่ฉันได้เรียนรู้ในการเรียนการสอนด้วยตนเองและการเรียนรู้นอกสถานที่ทั้งหมด (รวมถึงการเตรียมการสำหรับการสอบเพื่อรับใบรับรอง) ฉันยังเห็นหลาย ๆ ครั้งในรหัสของโครงการซึ่งสิ่งต่าง ๆ ไม่ได้ทำใน วิธีที่ดีที่สุด. ฉันอยู่ในทีมนี้มาหนึ่งสัปดาห์แล้ว แต่ฉันเห็นปัญหามากมายเกี่ยวกับ codebase ของพวกเขาที่ฉันคิดว่าฉันจะใช้เวลาต่อสู้กับสิ่งที่เขียนไปแล้วเพื่อทำสิ่งต่าง ๆ ใน "ทางของพวกเขา" มากกว่าที่ฉันจะเป็น ทำงานในโครงการที่ยกตัวอย่างเช่นปฏิบัติตามมาตรฐานการเข้ารหัสที่ยอมรับกันอย่างแพร่หลายรูปแบบสถาปัตยกรรมและแนวปฏิบัติที่ดีที่สุด สิ่งนี้นำมาสู่คำถามของฉัน: …

9 project-management  code-quality