タイプunsafety：FooからBarへのチェックされていないキャスト。 JPAのもの

Question

私はopenJPA（ちょっと古いバージョン1.2）を使用していて、私がquery.getResultList()をキャストするたびに警告を受け取ります。私はその特定の警告を取り除くことができますが、私は次の例のようにリストを返す必要があるとき、私は同じことを行うことができないので、私は前の行に@SuppressWarnings("unchecked")を使用します。

// @SuppressWarnings("unchecked") I DON'T WANT TO DO THIS HERE 
public <T> List<T> findByQuery(String queryString) { 
    myEntityManager.createQuery(queryString); 
    @SuppressWarnings("unchecked") // I WANT TO DO IT HERE 
    return (List<T>)query.getResultList(); 
} 

ここで私は何ができる：

... 
    @SuppressWarnings("unchecked") 
    List<T> result = query.getResultList(); 
    return result; 
} 

ですが、sonarはリストを変数に格納してはいけませんが、すぐにそれを返すべきだと不平を言います。ここで私の最高のオプションは何ですか？

Answer 1

query.getResultList()返信List<Object>です。あなたはそれをList<T>にキャストすることを望んでいませんが、javaはそれが可能かどうかを定義することはできません - タイプの消去のために実行時に違いはありません。そのため、@SuppressWarnings("unchecked")を使用する必要があります。

これを取り除くには、単純な解決策が1つあります。すべてのオブジェクトを新しい関数に必要な型にキャストします。これを行うには、あなたはそれがあるに今何クラス持っている - そうします。javaの古いバージョンでは

Javaで

public <T> List<T> findByQuery(String queryString, Class<T> clazz) { 
    myEntityManager.createQuery(queryString); 
    return query 
      .getResultList() 
      .stream() 
      .map(result -> clazz.cast(result)) 
      .collect(Collectors.toList()); 
} 

あなただけの新しいList<T>

にキャストした結果を置く単純なループにマップ機能を変更します