คนตัดไม้ควรประกาศแบบคงที่หรือไม่? โดยปกติฉันเคยเห็นคำประกาศสองประเภทสำหรับคนตัดไม้:
บันทึกบันทึกที่มีการป้องกัน = Log4JLogger ใหม่ (aClass.class);
หรือ
บันทึกล็อกแบบคงที่ส่วนตัว = Log4JLogger ใหม่ (aClass.class);
ควรใช้ตัวไหน โปรและคอนของทั้งคู่คืออะไร?
staticคือหนึ่งการอ้างอิงต่อคลาส non-static คือหนึ่งการอ้างอิงต่ออินสแตนซ์ (+ initialization) ดังนั้นในบางกรณีอย่างหลังจะส่งผลกระทบต่อหน่วยความจำอย่างมากหากคุณมีอินสแตนซ์มากมาย อย่าใช้วัตถุไม่คงที่ในวัตถุบ่อยๆ ฉันใช้เวอร์ชันคงที่เสมอ (ซึ่งควรเป็นตัวพิมพ์ใหญ่ LOG )
private static final Log logเป็นตัวพิมพ์เล็ก คนตัดไม้ไม่ใช่ค่าคงที่คนตัดไม้เป็นวัตถุสุดท้ายที่คงที่ (ที่สามารถกลายพันธุ์ได้) loggerส่วนตัวผมมักจะใช้
คำตอบ:
ข้อดีของรูปแบบไม่คงที่คือคุณสามารถประกาศในคลาสพื้นฐาน (นามธรรม) ดังต่อไปนี้โดยไม่ต้องกังวลว่าจะใช้ชื่อคลาสที่ถูกต้อง:
protected Log log = new Log4JLogger(getClass());อย่างไรก็ตามข้อเสียคือเห็นได้ชัดว่าอินสแตนซ์คนตัดไม้ใหม่ทั้งหมดจะถูกสร้างขึ้นสำหรับทุกอินสแตนซ์ของคลาส สิ่งนี้อาจไม่แพง แต่จะเพิ่มค่าใช้จ่ายที่สำคัญ หากคุณต้องการหลีกเลี่ยงสิ่งนี้คุณต้องการใช้staticแบบฟอร์มแทน แต่ข้อเสียคือคุณต้องประกาศในแต่ละคลาสและดูแลในทุกคลาสว่ามีการใช้ชื่อคลาสที่ถูกต้องระหว่างการสร้างคนตัดไม้เนื่องจากgetClass()ไม่สามารถใช้ในบริบทคงที่ อย่างไรก็ตามใน IDE โดยเฉลี่ยคุณสามารถสร้างเทมเพลตการเติมข้อความอัตโนมัติสำหรับสิ่งนี้ได้ เช่นlogger+ ctrl+space.
ในทางกลับกันหากคุณได้รับเครื่องตัดไม้จากโรงงานซึ่งอาจแคชผู้บันทึกที่สร้างอินสแตนซ์แล้วการใช้รูปแบบไม่คงที่จะไม่เพิ่มค่าใช้จ่ายมากขนาดนั้น ตัวอย่างเช่น Log4j มีLogManagerสำหรับวัตถุประสงค์นี้
protected Log log = LogManager.getLogger(getClass());abstract Log getLogger();ในคลาสนามธรรม ใช้วิธีนี้โดยส่งคืนตัวบันทึกแบบคงที่สำหรับอินสแตนซ์เฉพาะ เพิ่มprivate final static Log LOG = LogManager.getLogger(Clazz.class);ในเทมเพลตคลาส IDE ของคุณ
protected Logger log = LoggerFactory.getLogger(getClass());
ฉันเคยคิดว่าคนตัดไม้ทุกคนควรเป็นแบบคงที่ อย่างไรก็ตามบทความนี้ใน wiki.apache.orgนำเสนอข้อกังวลเกี่ยวกับหน่วยความจำที่สำคัญบางประการเกี่ยวกับการรั่วไหลของ classloader การประกาศตัวบันทึกเป็นแบบคงที่จะป้องกันไม่ให้คลาสที่ประกาศ (และตัวโหลดคลาสที่เกี่ยวข้อง) เป็นขยะที่รวบรวมในคอนเทนเนอร์ J2EE ที่ใช้คลาสโหลดเดอร์แบบแบ่งใช้ ซึ่งจะส่งผลให้เกิดข้อผิดพลาด PermGen หากคุณปรับใช้แอปพลิเคชันของคุณใหม่หลายครั้งเพียงพอ
ฉันไม่เห็นวิธีใดในการแก้ไขปัญหาการรั่วไหลของคลาสตัวโหลดนี้นอกจากการประกาศว่าตัวบันทึกเป็นแบบไม่คงที่
ความแตกต่างที่สำคัญที่สุดคือส่งผลต่อไฟล์บันทึกของคุณอย่างไร: บันทึกอยู่ในหมวดหมู่ใด
กรณีแรกของคุณมีหลายรูปแบบ:
บันทึกบันทึกที่มีการป้องกัน = Log4JLogger ใหม่ (getClass ());
ในกรณีนี้หมวดหมู่บันทึกของคุณจะบอกว่าอ็อบเจ็กต์ใดที่โค้ดที่บันทึกทำงานอยู่
ในตัวเลือกที่สองของคุณ (สแตติกส่วนตัว) ประเภทบันทึกคือคลาสที่มีรหัสการบันทึก ดังนั้นโดยปกติแล้วชั้นเรียนที่กำลังทำสิ่งที่กำลังบันทึกอยู่
ฉันขอแนะนำตัวเลือกสุดท้ายนั้นเป็นอย่างยิ่ง มีข้อดีเหล่านี้เมื่อเทียบกับโซลูชันอื่น ๆ :
นอกจากนี้ยังมีข้อเสีย:
ใช้การผกผันของการควบคุมและส่งตัวบันทึกไปยังตัวสร้าง หากคุณสร้างคนตัดไม้ในชั้นเรียนคุณจะต้องเสียเวลากับการทดสอบหน่วยของคุณ คุณกำลังเขียนแบบทดสอบหน่วยไม่ใช่เหรอ?