Каков наилучший способ получить минимальное и максимальное значение из списка Comparables, которые содержат нулевые значения?

Я думаю о чем-то вроде этого:

public static <T extends Comparable<T>> T minOf(T...ts){        
    SortedSet<T> set = new TreeSet<T>(Arrays.asList(ts));
    return set.first();
}

public static <T extends Comparable<T>> T maxOf(T...ts){
    SortedSet<T> set = new TreeSet<T>(Arrays.asList(ts));
    return set.last();
}

Но не является нулевым, что тоже нужно.

Знаете ли вы, что лучше решить эту проблему?

EDIT:

После комментариев я также попробовал min():

public static <T extends Comparable<T>> T minOf(T...ts){        
    return Collections.min(Arrays.asList(ts), new Comparator<T>(){

        public int compare(T o1, T o2) {
            if(o1!=null && o2!=null){
                return o1.compareTo(o2);
            }else if(o1!=null){
                return 1;
            }else{
                return -1;  
            }
        }});
}

Что вы думаете об этом?

Ответы

Ответ 1

Что случилось с Collections.max?

И почему вы заботитесь о нулевой безопасности? Вы уверены, что хотите разрешить nulls в своей коллекции?

Ответ 2

Если вам действительно нужно исключить "нуль" из результата, и вы не можете помешать ему находиться в вашем массиве, то, возможно, вам нужно просто перебирать массив с помощью простого цикла и отслеживать "мин", и "max" в отдельных переменных. Вы все равно можете использовать метод compare() для каждого объекта, чтобы сравнить его с текущими значениями "min" и "max". Таким образом, вы можете добавить свой собственный код для проверки нулей и игнорирования их.

EDIT: вот какой код, чтобы проиллюстрировать то, о чем я говорю. К сожалению, есть краевой случай, который вам нужно рассмотреть - что, если все аргументы, переданные в, равны нулю? Что возвращает ваш метод?

public static <T extends Comparable<T>> T minOf(T...ts){
    T min = null;
    for (T t : ts) {
        if (t != null && (min == null || t.compareTo(min) < 0)) {
            min = t;
        }
    }
    return min;
}

public static <T extends Comparable<T>> T maxOf(T...ts){
    T max = null;
    for (T t : ts) {
        if (t != null && (max == null || t.compareTo(max) > 0)) {
            max = t;
        }
    }
    return max;
}

Ответ 3

Вы не должны использовать Comparable для принятия значения null, поскольку он нарушает контракт интерфейса.

От https://docs.oracle.com/javase/7/docs/api/java/lang/Comparable.html:

Обратите внимание, что null не является экземпляром какого-либо класса, а e.compareTo(null) должен вызывать исключение NullPointerException, даже если e.equals(null) возвращает false.

Вместо этого вы должны создать новый интерфейс, например. ComparableNull.

См. также: