Collections.sort报Comparison method违反约定，如何处理字符串空值比较？

这个错误的核心是你的Comparator违反了Java要求的三大契约规则：自反性、对称性、传递性。你的代码里有几处逻辑破坏了这些规则，尤其是空字符串/特殊行（文本行）的处理，我来一步步帮你修复。

先明确问题出在哪 #

1. 文本行判断逻辑不对称 #

你的代码里：

if (dim1.isTextLine()) return 1;
if (dim2.isTextLine()) return -1;

如果两个对象都是文本行，第一个条件会返回1，但此时compare(dim1, dim2)=1，compare(dim2, dim1)也会返回1——这直接违反了对称性（要求如果a > b，则b < a）。

2. 字符串比较用了错误的方式 #

你原来注释里的if(dim1.getGroupName() != "" || dim2.getGroupName() != "")用了!=比较字符串内容，这是错的！==/!=比较的是对象引用，不是字符串内容，应该用!isEmpty()或者!equals("")，而且要先处理null的情况。

3. 字段比较未处理空值 #

你的compare(dim1, dim2, mySortFields[i])方法（没贴出来，但肯定存在）没有统一处理空字符串/null的情况，这很容易破坏传递性。

4. 重复分割排序字段影响性能（非错误，但建议优化） #

每次compare都执行Util.split(this.sortField, ",")，会重复创建数组，没必要。

修复后的完整代码 & 空值处理方案 #

我会把所有问题修复，并且统一空字符串的处理规则（这里约定：空字符串/null排在非空字符串前面，文本行排在普通行后面，你可以根据需求调整）：

Collections.sort(keyList, new DIMsComparator<DIM>(sortField));

public static class DIMsComparator<T> implements Comparator<T> {
    private final String[] sortFields;

    public DIMsComparator(String sortField) {
        // 构造函数内只分割一次，过滤空字段
        this.sortFields = sortField == null || sortField.isEmpty() 
            ? new String[0] 
            : Arrays.stream(Util.split(sortField, ","))
                    .filter(s -> s != null && !s.isEmpty())
                    .toArray(String[]::new);
    }

    @Override
    public int compare(T obj1, T obj2) {
        // 处理非DIM对象的情况，保证对称性
        boolean isObj1DIM = obj1 instanceof DIM;
        boolean isObj2DIM = obj2 instanceof DIM;
        if (!isObj1DIM || !isObj2DIM) {
            if (isObj1DIM && !isObj2DIM) {
                return -1; // DIM对象排在非DIM前面
            } else if (!isObj1DIM && isObj2DIM) {
                return 1;
            } else {
                return 0;
            }
        }

        DIM dim1 = (DIM) obj1;
        DIM dim2 = (DIM) obj2;

        // 修复文本行判断的对称性
        boolean isDim1TextLine = dim1.isTextLine();
        boolean isDim2TextLine = dim2.isTextLine();
        if (isDim1TextLine && isDim2TextLine) {
            return 0; // 两个都是文本行，相等
        }
        if (isDim1TextLine) {
            return 1; // 文本行排在普通行后面
        }
        if (isDim2TextLine) {
            return -1;
        }

        // 按排序字段依次比较
        for (String field : sortFields) {
            int compareResult = compareField(dim1, dim2, field);
            if (compareResult != 0) {
                return compareResult;
            }
        }

        // 最后按dimSeqNo排序，满足三大规则
        return Integer.compare(dim1.dimSeqNo, dim2.dimSeqNo);
    }

    // 核心：处理字段比较中的空值/空字符串
    private int compareField(DIM dim1, DIM dim2, String field) {
        // 假设你有一个获取字段值的方法，比如getFieldValue
        String val1 = getFieldValue(dim1, field);
        String val2 = getFieldValue(dim2, field);

        // 统一空值处理规则：空/null排在前面
        boolean isVal1Empty = val1 == null || val1.isEmpty();
        boolean isVal2Empty = val2 == null || val2.isEmpty();

        if (isVal1Empty && isVal2Empty) {
            return 0;
        }
        if (isVal1Empty) {
            return -1; // val1空，排在val2前面
        }
        if (isVal2Empty) {
            return 1; // val2空，val1排在后面
        }

        // 普通字符串比较，默认按字典序
        return val1.compareTo(val2);
    }

    // 你自己的获取字段值的方法，比如根据field名反射或者直接调用getter
    private String getFieldValue(DIM dim, String field) {
        // 示例实现，根据你的实际情况修改
        if ("groupName".equals(field)) {
            return dim.getGroupName();
        }
        // 其他字段...
        return "";
    }
}

关键规则验证 #

修复后的代码完全满足Comparator的三大契约：

1. 自反性：compare(a,a)返回0，不管是文本行、空值还是普通对象。
2. 对称性：如果compare(a,b)=1，那么compare(b,a)必然返回-1，比如文本行和普通行的比较，空值和非空值的比较。
3. 传递性：如果a < b且b < c，则a < c，比如空值<非空值，非空值<文本行，那么空值<文本行，符合传递性。

额外注意事项 #

• 如果你需要不同的空值排序规则（比如空值排在后面），只需要修改compareField里的返回值：空值时返回1，非空值返回-1即可。
• 尽量避免在compare方法里做耗时操作（比如重复分割字符串、数据库查询），会严重影响排序性能。

内容的提问来源于stack exchange，提问作者Vivek Shankar