วิธีค้นหารหัสที่ไม่ได้ใช้ / ไม่ได้ใช้ในโครงการ Java [ปิด]

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

ข้อเสนอแนะสำหรับกลยุทธ์ / เทคนิคทั่วไป (นอกเหนือจากเครื่องมือเฉพาะ) ก็มีค่าเช่นกัน

แก้ไข:โปรดทราบว่าเราได้ใช้เครื่องมือครอบคลุมรหัส (Clover, IntelliJ) แล้ว แต่สิ่งเหล่านี้เป็นความช่วยเหลือเล็กน้อย รหัสที่ตายแล้วยังคงมีการทดสอบหน่วยและปรากฏขึ้นตามที่ครอบคลุม ฉันเดาว่าเครื่องมือในอุดมคติจะระบุกลุ่มของรหัสที่มีรหัสอื่นน้อยมากขึ้นอยู่กับมันทำให้สามารถตรวจสอบเอกสารด้วยตนเองได้

ฉันจะใช้ระบบที่ทำงานอยู่เพื่อเก็บบันทึกการใช้รหัสจากนั้นเริ่มตรวจสอบรหัสที่ไม่ได้ใช้เป็นเดือนหรือเป็นปี

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

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

ปลั๊กอินคราสที่ทำงานได้ดีพอสมควรเป็นเครื่องตรวจจับรหัสที่ไม่ได้ใช้

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

CodeProเปิดตัวเมื่อเร็ว ๆ นี้โดย Google ด้วยโครงการ Eclipse ฟรีและมีประสิทธิภาพสูง ปลั๊กอินมีคุณลักษณะ ' ค้นหารหัส Dead ' ที่มีจุดเริ่มต้นหนึ่ง / หลายจุด ทำงานได้ดีทีเดียว

ฉันแปลกใจที่ProGuardไม่ได้ถูกกล่าวถึงที่นี่ เป็นหนึ่งในผลิตภัณฑ์ที่เป็นผู้ใหญ่ที่สุดในรอบ

ProGuardเป็นโปรแกรมลดขนาดไฟล์คลาส Java, optimizer, obfuscator และ preverifier มันตรวจจับและลบคลาสที่ไม่ได้ใช้ฟิลด์วิธีการและคุณลักษณะ มันเพิ่มประสิทธิภาพ bytecode และลบคำแนะนำที่ไม่ได้ใช้ เปลี่ยนชื่อคลาสฟิลด์และเมธอดที่เหลือโดยใช้ชื่อที่ไม่มีความหมายสั้น ๆ ในที่สุดมันจะยืนยันรหัสที่ประมวลผลล่วงหน้าสำหรับ Java 6 หรือสำหรับ Java Micro Edition

การใช้ประโยชน์บางอย่างของ ProGuard คือ:

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

ตัวอย่างที่นี่สำหรับรายการรหัสที่ตายแล้ว: https://www.guardsquare.com/en/products/proguard/manual/examples#deadcode

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

แต่มันเป็นคู่มือมาก

ใช้เครื่องมือครอบคลุมการทดสอบเพื่อทำโค้ดเบสของคุณจากนั้นเรียกใช้แอพพลิเคชั่นนั้นไม่ใช่การทดสอบ

EmmaและEclemmaจะให้รายงานที่ดีถึงคุณว่าเปอร์เซ็นต์ของคลาสใดที่เรียกใช้สำหรับการเรียกใช้โค้ดใด ๆ

เราได้เริ่มใช้Find Bugsเพื่อช่วยระบุ funk บางอย่างในสภาพแวดล้อมที่อุดมด้วยเป้าหมายของ codebase สำหรับการ refactorings ฉันจะพิจารณาโครงสร้าง 101เพื่อระบุจุดในสถาปัตยกรรมของ codebase ที่ซับซ้อนเกินไปดังนั้นคุณจึงรู้ว่าหนองน้ำจริงอยู่ที่ไหน

ในทางทฤษฎีคุณไม่สามารถกำหนดรหัสที่ไม่ได้ใช้อย่างไม่แน่นอน มีข้อพิสูจน์ทางคณิตศาสตร์ของเรื่องนี้ (นี่เป็นกรณีพิเศษของทฤษฎีบททั่วไปที่มากกว่า) หากคุณสงสัยให้ค้นหาปัญหาการหยุดชะงัก

สิ่งนี้สามารถประจักษ์เองในโค้ด Java ได้หลายวิธี:

• กำลังโหลดคลาสที่อิงจากอินพุตของผู้ใช้ไฟล์กำหนดค่ารายการฐานข้อมูล ฯลฯ
• กำลังโหลดรหัสภายนอก
• ผ่านต้นไม้วัตถุไปยังห้องสมุดบุคคลที่สาม;
• เป็นต้น

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

ใน Eclipse Goto Windows> การกำหนดค่าตามความชอบ> Java> คอมไพเลอร์> ข้อผิดพลาด / คำเตือน
และเปลี่ยนเป็นข้อผิดพลาดทั้งหมด แก้ไขข้อผิดพลาดทั้งหมด นี่เป็นวิธีที่ง่ายที่สุด ความงามคือมันจะช่วยให้คุณสามารถทำความสะอาดรหัสในขณะที่คุณเขียน

รหัสหน้าจอ Eclipse:

IntelliJ มีเครื่องมือวิเคราะห์รหัสสำหรับตรวจจับโค้ดที่ไม่ได้ใช้ คุณควรลองสร้างฟิลด์ / เมธอด / คลาสให้มากที่สุดเท่าที่จะเป็นสาธารณะและจะแสดงเมธอด / ฟิลด์ / คลาสที่ไม่ได้ใช้มากขึ้น

ฉันจะพยายามค้นหารหัสที่ซ้ำกันเพื่อลดปริมาณรหัส

ข้อเสนอแนะสุดท้ายของฉันคือพยายามค้นหาโอเพนซอร์สซึ่งถ้าใช้จะทำให้โค้ดของคุณง่ายขึ้น

มุมมองโครงสร้าง101จะให้รายการ (และกราฟการพึ่งพา) ของ "เด็กกำพร้า" หรือ " กลุ่มเด็กกำพร้า" ใด ๆของชั้นเรียนหรือแพคเกจที่ไม่มีการอ้างอิงถึงหรือจากกลุ่ม "หลัก"

DCD ไม่ใช่ปลั๊กอินสำหรับ IDE บางตัว แต่สามารถเรียกใช้จากมดหรือสแตนด์อโลน ดูเหมือนว่าเครื่องมือแบบคงที่และจะสามารถทำในสิ่งที่ PMD และ FindBugs ไม่สามารถ ฉันจะลองดู

PS ดังกล่าวในความคิดเห็นด้านล่างโครงการมีชีวิตอยู่ในขณะนี้ในGitHub

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

• FindBugs นั้นยอดเยี่ยมสำหรับสิ่งนี้
• PMD (Project Mess Detector) เป็นอีกเครื่องมือหนึ่งที่สามารถใช้ได้

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

เครื่องมือครอบคลุมผู้ใช้เช่น EMMA แต่ไม่ใช่เครื่องมือแบบสแตติก (เช่นต้องใช้แอปพลิเคชันจริงผ่านการทดสอบการถดถอยและผ่านทุกกรณีข้อผิดพลาดที่เป็นไปได้ซึ่งก็คือเป็นไปไม่ได้ :)

ถึงกระนั้น EMMA ก็มีประโยชน์มาก

เครื่องมือครอบคลุมรหัสเช่น Emma, ​​Cobertura และ Clover จะใช้รหัสของคุณและบันทึกว่าส่วนใดของมันที่ถูกเรียกใช้โดยใช้ชุดการทดสอบ สิ่งนี้มีประโยชน์มากและควรเป็นส่วนหนึ่งของกระบวนการพัฒนาของคุณ มันจะช่วยให้คุณระบุว่าชุดทดสอบของคุณครอบคลุมรหัสของคุณได้ดีเพียงใด

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

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

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

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

มีโครงการ Java - Dead Code Detector (DCD) สำหรับซอร์สโค้ดมันใช้งานไม่ได้ แต่สำหรับไฟล์. jar มันดีจริงๆ รวมทั้งคุณสามารถกรองตามชั้นเรียนและตามวิธีการ

Netbeans ที่นี่เป็นปลั๊กอินสำหรับ Netbeans ตรวจจับรหัสตายตรวจจับรหัสตาย

มันจะดีกว่าถ้ามันสามารถเชื่อมโยงและเน้นรหัสที่ไม่ได้ใช้ คุณสามารถลงคะแนนและแสดงความคิดเห็นได้ที่นี่: Bug 181458 - ค้นหาคลาสสาธารณะวิธีการและเขตข้อมูลที่ไม่ได้ใช้

Eclipse สามารถแสดง / ไฮไลต์รหัสที่ไม่สามารถเข้าถึงได้ JUnit สามารถแสดงการครอบคลุมโค้ดให้คุณได้ แต่คุณต้องมีการทดสอบบางอย่างและต้องตัดสินใจว่าการทดสอบที่เกี่ยวข้องนั้นขาดหายไปหรือไม่ได้ใช้รหัสจริงๆ

ฉันพบเครื่องมือครอบคลุมโคลเวอร์ซึ่งใช้รหัสเครื่องมือและไฮไลต์รหัสที่ใช้และไม่ได้ใช้งาน ไม่เหมือนกับ Google CodePro Analytics แต่ก็ใช้ได้กับ WebApplications (ตามประสบการณ์ของฉันและฉันอาจไม่ถูกต้องเกี่ยวกับ Google CodePro)

ข้อเสียเปรียบเพียงอย่างเดียวที่ฉันสังเกตคือมันไม่ได้คำนึงถึงส่วนต่อประสานของ Java

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