Java Collections.sort - 帮助我删除未经检查的警告

Question

List<Question> questions = new ArrayList<Question>();
questions.addAll(getAllQuestions()); //returns a set of Questions
Collections.sort(questions, new BeanComparator("questionId")); //org.apache.commons.beanutils.BeanComparator

在 Java 1.5 下，除了 'new BeanComparator("questionId")' 生成未经检查的警告之外，上述工作正常。我不喜欢警告。有没有办法为 BeanComparator 提供类型，或者我必须使用 @SuppressWarnings("unchecked") ？

Answer 1

选项有：

• 更改 BeanComparator 以实现 Comparator。这不是一个真正的选择，因为它是一个众所周知的外部库类。人们不会让你这样做。
• 按照上面的方法分叉并修改 BeanComparator，给它一个不同的 FQN。
• 使用实现 Comparator 的类包装现有的 BeanComparator。
• 将问题类型更改为List。
• 添加抑制警告注释。

Answer 2

自 < code>BeanComparator 不是通用的，您只需抑制即可。

更新：实际上，如果它足够困扰您，您可以分叉代码库以使其通用，因为它是开源的。

Answer 3

除非在 Apache Commons Beanutils 中添加新的泛型类，否则我发现最好的方法是将 BeanComparator 包装在我的“bean 工具箱”中的新方法中：

/**
 * Wrapping of Apache communs BeanComparator. Create a comparator which compares two beans by the specified bean
 * property. Property expression can use Apache's nested, indexed, combinated, mapped syntax. @see <a
 * href="http://commons.apache.org/beanutils/api/org/apache/commons/beanutils/BeanComparator.html">Apache's Bean
 * Comparator</a> for more details.
 * @param <T> generic type
 * @param propertyExpression propertyExpression
 * @return the comparator
 */
@SuppressWarnings("unchecked")
public static <T> Comparator<T> createPropertyComparator(final String propertyExpression) {
    return new BeanComparator(propertyExpression);
}

Answer 4

创建一个通用包装类：

public class GenericBeanComparator<T> implements Comparator<T> {
  private final BeanComparator myBeanComparator;

  public GenericBeanComparator(String property) {
    myBeanComparator = new BeanComparator(property);
  }

  public int compare(T o1, T o2) {
    return myBeanComparator.compare(o1, o2);
  }
}

像这样使用它：

List<Question> questions = new ArrayList<Question>();
questions.addAll(getAllQuestions()); //returns a set of Questions
Collections.sort(questions, new GenericBeanComparator<Question>("questionId"));

Answer 5

是的，你应该使用@SuppressWarnings(“unchecked”)。在这种情况下，没有理由认为不使用泛型的比较器会导致问题。

Answer 6

您随时可以改用 Google Collections。

他们支持泛型。

Answer 7

消除警告的唯一方法是更改​​ BeanComparator 的代码，但即使可以，除非您将其制作为能够理解您的特定类型的特定包装器，否则该概念也不会很好地工作。该类通过反射对任何对象进行操作，该对象可能具有或不具有该方法。它本质上不是类型安全的。

解决警告的最简单方法是实现您自己的比较器：

 public class QuestionComparator extends Comparator<Question> {
      private BeanComparator peer = new BeanComparator("questionId");

      public int compare(Question o1, Question o2) {
             return peer.compare(o1, o2);
      }
 }

如果重要的话，您还可以实现 equals，并像这样调用 BeanComparator equals 方法：

   public boolean equals(Object o) {
       //boiler plate code here to ensure o is an instance of Question and not null
       return ((QuestionComparator) o).peer.equals(peer);
   }

Answer 8

BeanComparator 是一个非常小的类。获取源代码并进行如下修改：


公共类 BeanComparator实现比较器，可序列化{


并像这样修改您的调用：


Collections.sort(yourCollection, new BeanComparator(yourProperty));


瞧，警告消失了。