เหตุผลใดในการล้างการนำเข้าที่ไม่ได้ใช้ใน Java นอกเหนือจากการลดความยุ่งเหยิง

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

(ฉันถามเพราะ Eclipse ให้คำเตือนเกี่ยวกับการนำเข้าที่ไม่ได้ใช้ซึ่งเป็นเรื่องที่น่ารำคาญเมื่อฉันพัฒนาโค้ดเพราะฉันไม่ต้องการลบการนำเข้าจนกว่าฉันจะแน่ใจว่าฉันออกแบบคลาสเสร็จแล้ว)

ฉันไม่คิดว่าจะมีปัญหาด้านประสิทธิภาพหรืออะไรทำนองนั้นหากคุณไม่ได้ลบการนำเข้าออก

แต่อาจมีความขัดแย้งในการตั้งชื่อในบางกรณีเช่นการนำเข้าอินเทอร์เฟซรายการ

ใน Eclipse คุณสามารถใช้ทางลัด (ขึ้นอยู่กับ OS - Win: Ctrl + SHIFT + Oและ Mac :) COMMAND + SHIFT + Oเพื่อจัดระเบียบการนำเข้า Eclipse แล้วทำความสะอาดส่วนการนำเข้าลบทั้งหมดนำเข้าเก่า ฯลฯ Ctrl + SPACEหากคุณจำเป็นต้องเป็นสิ่งที่นำเข้าอีกครั้งคราสจะเพิ่มให้โดยอัตโนมัติในขณะที่คุณเสร็จสิ้นคำสั่งด้วย ดังนั้นจึงไม่จำเป็นต้องเก็บรหัสที่ไม่ได้ใช้ไว้ในชั้นเรียนของคุณ

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

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

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

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

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

จากมุมมองของคนเจ้าระเบียบการพึ่งพาใด ๆ ถือเป็น "ข้อ จำกัด " ของผลิตภัณฑ์และอาจทำให้เกิดปัญหาในการบำรุงรักษาในภายหลัง

ตัวอย่างเช่นสมมติว่าโปรแกรมของคุณใช้คลาส com.XYZObjectPool และหลังจากนั้นคุณตัดสินใจที่จะไม่ใช้ แต่จะไม่ลบการนำเข้า หากตอนนี้มีคนอื่นต้องการสร้างอินสแตนซ์ org.WVYObjectPool และอ้างถึง ObjectPool พวกเขาจะไม่ได้รับคำเตือนใด ๆ เกี่ยวกับเรื่องนี้จนกว่าจะมีปัญหาในการแคสต์หรือปัญหาการเรียกใช้

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

ไม่ว่าจะด้วยวิธีใดคุณสามารถขอให้ Eclipse ล้างสิ่งเหล่านี้ให้คุณได้

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

สิ่งนี้เกี่ยวข้องกับความชัดเจนของโปรแกรมที่มีประโยชน์สำหรับการบำรุงรักษา

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

ลองนึกถึงสถานการณ์ต่อไปนี้:

import company.billing.*;
import company.humanrerources.*;

// other imports 


class SomeClass {
      // hundreds or thousands of lines here... 
    public void veryImportantMethod() {
      Customer customer;
      Employee comployee;
      Department dept. 
      // do something with them
     }
 }

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

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

หากนี่เป็นโครงการส่วนตัวหรือสิ่งเล็ก ๆ มันไม่สำคัญ แต่สำหรับสิ่งที่ใหญ่กว่าที่นักพัฒนารายอื่นต้องใช้ (และดูแลตลอดหลายปี) นี่คือสิ่งที่ต้องมี

ไม่มีความแตกต่างอย่างแน่นอนกับประสิทธิภาพใด ๆ

FYI สิ่งนี้ทำให้ฉันรู้สึกแย่เพราะฉันไม่คิดว่าการจัดระเบียบการนำเข้าจะลบการนำเข้าที่ไม่ได้ใช้ออกไปจริงๆฉันคิดว่ามันแค่จัดเรียง

การลบการนำเข้าโดยอัตโนมัติในระหว่างการดำเนินการบันทึกทำให้ฉันรู้สึกเศร้าเมื่อตัวอย่างเช่นในระหว่างการพัฒนาหรือการทดสอบคุณมีปัญหาและแสดงความคิดเห็นเกี่ยวกับโค้ดเมื่อคุณบันทึกการนำเข้าที่ใช้โดยส่วนที่แสดงความคิดเห็นของโค้ดจะถูกลบออก บางครั้งนี่ไม่ใช่ปัญหาเนื่องจากคุณสามารถยกเลิก ( Ctrl+ Z) การเปลี่ยนแปลงได้ แต่บางครั้งก็ไม่ง่ายอย่างที่คุณอาจทำการเปลี่ยนแปลงอื่น ๆ ฉันยังมีปัญหาที่เมื่อฉันไม่ใส่ความคิดเห็นรหัส (ก่อนหน้านี้ฉันได้แสดงความคิดเห็นแล้วบันทึกไว้ดังนั้นจึงลบการนำเข้าสำหรับรหัสนั้นออก) มันจะพยายามเดาการนำเข้าที่จำเป็นโดยอัตโนมัติและหยิบรหัสที่ไม่ถูกต้อง (เช่นฉันคิดว่าฉัน มีStringUtilsคลาสและเลือกคลาสอื่นที่มีชื่อเดียวกันจากไลบรารีที่ไม่ถูกต้อง)

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

สำหรับคราสฉันใช้สิ่งนี้: window -> การตั้งค่า -> java -> ตัวแก้ไข -> บันทึกการดำเนินการ -> เลือกช่องทำเครื่องหมายเพื่อจัดระเบียบการนำเข้า (มีสิ่งที่มีประโยชน์อื่น ๆ อีกมากมายเช่นการจัดรูปแบบการทำให้ฟิลด์เป็นขั้นสุดท้ายและอื่น ๆ .). ดังนั้นเมื่อฉันบันทึกไฟล์ eclipse ของฉันจะลบการนำเข้า unessacry ให้ฉัน ในความคิดของฉันถ้าคุณไม่ต้องการบางสิ่งบางอย่างให้ลบออก (หรือปล่อยให้มันถูกลบออกโดยคราส)

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

ไม่มีผลกระทบด้านประสิทธิภาพใด ๆ แม้ว่าคุณสามารถอ่านได้เพื่อความสะอาด การลบการนำเข้าที่ไม่ได้ใช้นั้นค่อนข้างง่ายทั้งใน Eclipse และ IntelliJ IDEA

คราส

Windows / Linux -    Ctrl+ Shift+O

Mac -                        Cmd+ Shift+O

IntelliJ IDEAหรือAndroid Studio

Windows / Linux -    Ctrl+ Alt+O

Mac -                        Cmd+ Alt+O

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

และโดยวิธีการที่ eclipse คุณสามารถใช้Ctrl+ Shift+ Oเพื่อจัดระเบียบการนำเข้า แต่คุณยังสามารถกำหนดค่า "ตัวล้าง" ที่จัดการกับสิ่งเหล่านี้ (และอื่น ๆ อีกมากมาย) ทุกครั้งที่คุณบันทึกไฟล์ java

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