12

使用自定义比较器,每次实例化它而不是将其创建为常量(使用匿名类)并使用单个实例有什么好处?我一直认为每次都创建一个新实例没有任何优势,并且总是采用选项#2(静态最终字段中的单个实例)的方式。

public class SomeClass {

  //First option:
  private static class SomeCustomComparator implements Comparator<SomeObject> {
    public int compare(SomeObject o1, SomeObject o2) {
      /*implementation*/
    }
  }

  //Second option:
  private static final Comparator<SomeObject> CUSTOM_COMPARATOR = new Comparator<SomeObject> {
    public int compare(SomeObject o1, SomeObject o2) {
      /*implementation*/
    }
  };

  public void doSomething() {
    //are there any advantages to one over the other?
    SortedSet<SomeObject> s1 = new TreeSet<>(CUSTOM_COMPARATOR);

    SortedSet<SomeObject> s2 = new TreeSet<>(new SomeCustomComparator());
  }
}

这里的假设是比较器中不需要保存任何状态。

如果 doSomething() 被多次调用怎么办?如果从多个线程调用 doSomething() 怎么办?如果 CUSTOM_COMPARATOR 被拉到一个公共类中并公开而不是私有怎么办?

4

1 回答 1

19

如果比较器没有状态(大多数都没有),那么创建一个实例并在任何地方使用它是绝对可以的。不要仅仅为了它而创建额外的对象。

于 2012-10-25T17:50:08.850 回答