ตัวอย่างเช่นฉันมีคำถามดังกล่าว:

Query q = sess.createQuery("from Cat cat");
List cats = q.list();

หากฉันพยายามทำสิ่งนี้จะแสดงคำเตือนต่อไปนี้

Type safety: The expression of type List needs unchecked conversion to conform to List<Cat>


List<Cat> cats = q.list();

มีวิธีหลีกเลี่ยงหรือไม่?

ใช้@SuppressWarningsทุกที่ตามที่แนะนำเป็นวิธีที่ดีที่จะทำมัน q.list()แต่มันไม่เกี่ยวข้องกับบิตของนิ้วมือพิมพ์แต่ละครั้งที่คุณโทร

มีอีกสองเทคนิคที่ฉันแนะนำ:

เขียนผู้ช่วยนักแสดง

เพียงแค่ทำการ refactor ทั้งหมดของคุณ@SuppressWarningsในที่เดียว:

List<Cat> cats = MyHibernateUtils.listAndCast(q);

...

public static <T> List<T> listAndCast(Query q) {
    @SuppressWarnings("unchecked")
    List list = q.list();
    return list;
}

ป้องกันไม่ให้ Eclipse สร้างคำเตือนสำหรับปัญหาที่หลีกเลี่ยงไม่ได้

ใน Eclipse ไปที่ Window> Preferences> Java> Compiler> Errors / Warnings และภายใต้ Generic type ให้เลือกช่องทำเครื่องหมาย Ignore unavoidable generic type problems due to raw APIs

การดำเนินการนี้จะปิดคำเตือนที่ไม่จำเป็นสำหรับปัญหาที่คล้ายคลึงกันเช่นเดียวกับที่อธิบายไว้ข้างต้นซึ่งไม่สามารถหลีกเลี่ยงได้

ความคิดเห็นบางส่วน:

• ฉันเลือกที่จะผ่านQueryแทนที่จะเป็นq.list()เพราะวิธีนั้นวิธีการ "โกง" นี้สามารถใช้ในการโกงด้วย Hibernate เท่านั้นไม่ใช่การโกงใด ๆListโดยทั่วไป
• คุณสามารถเพิ่มวิธีการที่คล้ายกันสำหรับ.iterate()ฯลฯ

เป็นเวลานานแล้วที่คำถามถูกถาม แต่ฉันหวังว่าคำตอบของฉันอาจเป็นประโยชน์สำหรับคนอย่างฉัน

ถ้าคุณดูที่ javax.persistence เอกสาร APIJava Persistence 2.0คุณจะเห็นว่าบางส่วนวิธีการใหม่ได้รับการเพิ่มมีตั้งแต่ หนึ่งในนั้นคือการที่ผลตอบแทนcreateQuery(String, Class<T>) TypedQuery<T>คุณสามารถใช้TypedQueryเช่นเดียวกับที่คุณใช้Queryกับความแตกต่างเล็กน้อยที่การดำเนินการทั้งหมดจะปลอดภัยในขณะนี้

ดังนั้นเพียงแค่เปลี่ยนรหัสของคุณเป็น smth ดังนี้:

Query q = sess.createQuery("from Cat cat", Cat.class);
List<Cat> cats = q.list();

และคุณพร้อมแล้ว

เราใช้@SuppressWarnings("unchecked")เช่นกัน แต่ส่วนใหญ่เรามักจะพยายามใช้กับการประกาศตัวแปรเท่านั้นไม่ใช่วิธีการทั้งหมด:

public List<Cat> findAll() {
    Query q = sess.createQuery("from Cat cat");
    @SuppressWarnings("unchecked")
    List<Cat> cats = q.list();
    return cats;
}

ลองใช้TypedQueryแทนQuery. ตัวอย่างแทนสิ่งนี้: -

Query q = sess.createQuery("from Cat cat", Cat.class);
List<Cat> cats = q.list();

ใช้สิ่งนี้: -

TypedQuery<Cat> q1 = sess.createQuery("from Cat cat", Cat.class);
List<Cat> cats = q1.list();

ในรหัสของเราเราใส่คำอธิบายประกอบวิธีการโทรด้วย:

@SuppressWarnings ("ไม่เลือก")

ฉันรู้ว่าดูเหมือนเป็นการแฮ็ก แต่ผู้พัฒนาร่วมตรวจสอบเมื่อเร็ว ๆ นี้และพบว่านั่นคือทั้งหมดที่เราทำได้

เห็นได้ชัดว่า Query.list () วิธีการใน Hibernate API ไม่ได้พิมพ์ปลอดภัย "โดยการออกแบบ" และมีแผนการที่จะเปลี่ยน

ฉันเชื่อว่าวิธีที่ง่ายที่สุดในการหลีกเลี่ยงคำเตือนของคอมไพเลอร์คือการเพิ่ม @SuppressWarnings ("unchecked") คำอธิบายประกอบนี้สามารถวางไว้ที่ระดับวิธีการหรือหากอยู่ในเมธอดก่อนการประกาศตัวแปร

ในกรณีที่คุณมีเมธอดที่สรุป Query.list () และส่งคืน List (หรือ Collection) คุณจะได้รับคำเตือนด้วย แต่สิ่งนี้ถูกระงับโดยใช้ @SuppressWarnings ("rawtypes")

วิธี listAndCast (Query) ที่ Matt Quail เสนอมีความยืดหยุ่นน้อยกว่า Query.list () ในขณะที่ฉันสามารถทำได้:

Query q = sess.createQuery("from Cat cat");
ArrayList cats = q.list();

หากฉันลองใช้รหัสด้านล่าง:

Query q = sess.createQuery("from Cat cat");
ArrayList<Cat> cats = MyHibernateUtils.listAndCast(q);

ฉันจะได้รับข้อผิดพลาดในการคอมไพล์: Type mismatch: can't convert from List to ArrayList

ไม่ใช่การกำกับดูแลหรือผิดพลาด คำเตือนดังกล่าวสะท้อนถึงปัญหาพื้นฐานที่แท้จริง - ไม่มีวิธีใดที่คอมไพเลอร์ java สามารถมั่นใจได้ว่าคลาส hibernate จะทำงานได้อย่างถูกต้องและรายการที่ส่งคืนจะมีเฉพาะ Cats เท่านั้น ข้อเสนอแนะใด ๆ ที่นี่ดี

ไม่ได้ แต่คุณสามารถแยกออกเป็นวิธีการสืบค้นเฉพาะและระงับคำเตือนด้วย@SuppressWarnings("unchecked")คำอธิบายประกอบ

ตอนนี้ Hibernate เวอร์ชันใหม่กว่ารองรับQuery<T>อ็อบเจ็กต์ประเภทที่ปลอดภัยแล้วดังนั้นคุณจึงไม่ต้องใช้@SuppressWarningsหรือใช้แฮ็กเพื่อทำให้คำเตือนของคอมไพเลอร์หายไปอีกต่อไป ในAPI เซสชัน , Session.createQueryตอนนี้จะกลับมาเป็นประเภทปลอดภัยQuery<T>วัตถุ คุณสามารถใช้วิธีนี้:

Query<Cat> query = session.createQuery("FROM Cat", Cat.class);
List<Cat> cats = query.list();

คุณยังสามารถใช้เมื่อผลการค้นหาไม่ส่งคืน Cat:

public Integer count() {
    Query<Integer> query = sessionFactory.getCurrentSession().createQuery("SELECT COUNT(id) FROM Cat", Integer.class);
    return query.getSingleResult();
}

หรือเมื่อทำการเลือกบางส่วน:

public List<Object[]> String getName() {
    Query<Object[]> query = sessionFactory.getCurrentSession().createQuery("SELECT id, name FROM Cat", Object[].class);
    return query.list();
}

เรามีปัญหาเดียวกัน แต่มันไม่ใช่เรื่องใหญ่สำหรับเราเพราะเราต้องแก้ปัญหาสำคัญอื่น ๆ ด้วย Hibernate Query and Session

โดยเฉพาะ:

1. ควบคุมว่าเมื่อใดที่สามารถทำธุรกรรมได้ (เราต้องการนับจำนวนครั้งที่ tx "เริ่มต้น" และคอมมิตก็ต่อเมื่อ tx "สิ้นสุด" ตามจำนวนครั้งที่เริ่มต้นเท่านั้นมีประโยชน์สำหรับโค้ดที่ไม่รู้ว่าจำเป็นต้องเริ่มทำธุรกรรมตอนนี้หรือไม่ รหัสใด ๆ ที่ต้องการ tx เพียงแค่ "เริ่มต้น" และสิ้นสุดเมื่อเสร็จสิ้น)
2. การรวบรวมเมตริกประสิทธิภาพ
3. ชะลอการเริ่มต้นธุรกรรมจนกว่าจะทราบว่าจะมีบางอย่างเกิดขึ้นจริง
4. พฤติกรรมที่อ่อนโยนมากขึ้นสำหรับ query.uniqueResult ()

ดังนั้นสำหรับเราเรามี:

1. สร้างอินเทอร์เฟซ (AmplafiQuery) ที่ขยายคิวรี
2. สร้างคลาส (AmplafiQueryImpl) ที่ขยาย AmplafiQuery และห่อ org.hibernate.Query
3. สร้าง Txmanager ที่ส่งคืน Tx
4. Tx มีเมธอด createQuery ต่างๆและส่งคืน AmplafiQueryImpl

และประการสุดท้าย

AmplafiQuery มี "asList ()" ซึ่งเป็น Query.list () เวอร์ชันทั่วไปที่เปิดใช้งาน AmplafiQuery มี "unique ()" ซึ่งเป็นเวอร์ชันที่เปิดใช้งานทั่วไปของ Query.uniqueResult () (และเพียงแค่บันทึกปัญหาแทนที่จะโยน ข้อยกเว้น)

นี่เป็นงานที่ต้องทำเพื่อหลีกเลี่ยง @SuppressWarnings อย่างไรก็ตามอย่างที่ฉันพูด (และระบุไว้) มีอีกมากมายที่ดีกว่า! เหตุผลในการทำงานห่อ

ฉันรู้ว่าสิ่งนี้เก่ากว่า แต่มี 2 ประเด็นที่ควรทราบ ณ วันนี้ใน Matt Quails Answer

จุดที่ 1

นี้

List<Cat> cats = Collections.checkedList(Cat.class, q.list());

ควรเป็นสิ่งนี้

List<Cat> cats = Collections.checkedList(q.list(), Cat.class);

จุดที่ 2

จากนี้

List list = q.list();

สำหรับสิ่งนี้

List<T> list = q.list();

จะลดคำเตือนอื่น ๆ ลงอย่างเห็นได้ชัดในเครื่องหมายแท็กตอบกลับเดิมถูกเบราว์เซอร์ปล้น

ลองสิ่งนี้:

Query q = sess.createQuery("from Cat cat");
List<?> results = q.list();
for (Object obj : results) {
    Cat cat = (Cat) obj;
}

ทางออกที่ดีในการหลีกเลี่ยงคำเตือนด้านความปลอดภัยด้วยการสืบค้นแบบจำศีลคือการใช้เครื่องมือเช่นTorpedoQueryเพื่อช่วยคุณสร้าง hql ประเภทที่ปลอดภัย

Cat cat = from(Cat.class);
org.torpedoquery.jpa.Query<Entity> select = select(cat);
List<Cat> cats = select.list(entityManager);

TypedQuery<EntityName> createQuery = entityManager.createQuery("from EntityName", EntityName.class);
List<EntityName> resultList = createQuery.getResultList();

หากคุณไม่ต้องการใช้ @SuppressWarnings ("unchecked") คุณสามารถทำสิ่งต่อไปนี้ได้

   Query q = sess.createQuery("from Cat cat");
   List<?> results =(List<?>) q.list();
   List<Cat> cats = new ArrayList<Cat>();
   for(Object result:results) {
       Cat cat = (Cat) result;
       cats.add(cat);
    }

FYI - ฉันสร้างวิธีการใช้งานที่ทำเพื่อฉันดังนั้นมันจึงไม่ทิ้งโค้ดของฉันและฉันไม่ต้องใช้ @SupressWarning