有人建议我不要在这里使用反射...为什么不呢？

Question

该代码用于对列表进行排序。该列表可以包含数千个元素，但少于 10k。

protected <E> int compareFields(E o1, E o2, String fieldName){
    try { 
        Comparable o1Data = (Comparable) o1.getClass().getMethod(fieldName).invoke(o1);
        Comparable o2Data = (Comparable) o2.getClass().getMethod(fieldName).invoke(o2);
        return o1Data == null ? o2Data == null ? 0 : 1 :
               o2Data == null ? -1 : o1Data.compareTo(o2Data);
    } catch(Exception e) {
        throw new RuntimeException(e);
    }
}

我被建议

“请不要将反射用于此类事情！！ 要么为该方法提供合适的比较器，要么为该方法提供提取相关属性的方法（可能以原始类型不支持的方式计算），或两者兼而有之。”

兼而有之。”最好提供一个更好的方法

示例 。 : 我有很多带有数据表的屏幕。每个都是从列表构建的。每个数据表都需要可按其 6 列中的每一列进行排序。这些列是日期或字符串。

Answer 1

在没有上下文的情况下很难给出一个好的例子，所以现在这里列出了为什么它不是最好的想法：

1. 提供的字段不能保证可比较（不确定为什么这里的代码需要捕获该异常并重新命名它） 。
2. 如果提供的对象类型不打算以这种方式进行比较怎么办？ （这是一个过于通用的方法名称，无法知道它应该如何使用）。
3. 它不是强类型的。以字符串形式提供字段名称意味着每当属性名称发生更改时，您都必须更改代码，并且很难找到需要进行这些更改的位置。
4. 反射可能比以强类型方式实现要慢。

Answer 2

其他答案很好地描述了为什么不建议使用反射。我想添加一个使用更传统解决方案的示例。

您不应指定用于比较两个对象的字段，而应采用 Comparator 实例作为参数。这样使用此方法的客户端可以指定如何比较两个对象。

protected <E> int compareFields(E o1, E o2, Comparator<E> comparator) {
    return comparator.compare(o1, o2);
}

对此函数的调用示例如下所示：

MyClass a = ...;
MyClass b = ...;
Comparator<MyClass> intFieldComparator = new Comparator<MyClass> {
    public int compare(MyClass o1, MyClass o2) {
        int field1 = o1.getIntField();
        int field2 = o2.getIntField();

        return field2 - field1;
    }
};

compareFields(a, b, intFieldComparator);

如果要使用多个字段比较对象，则可以定义不同的比较器。

Answer 3

此处使用反射可能会慢得多，因为您通过使用 getClass、getMethod 和 invoke 向每次比较添加堆栈帧数，而不是使用对象的本机比较方法。

理想情况下，您应该编写方法以避免在签名中使用object。 “合适的比较器”至少会强烈绑定到对象的类型（您假设它们是相同的）。如果您必须进行动态字段比较（如图所示），那么至少可以将反射封装在该比较器中。

但是，如果您要调用此数千次，最好将比较器预先绑定到您要排序的字段。这样，您只需预先调用 getMethod 一次，而不是为每个单独的比较调用一次。