Google Collections Suppliers and Find

Question

Я ищу метод Google Collections, который возвращает первый результат последовательности поставщиков, который не возвращает null.

Я искал использование Iterables.find(), но в своем Predicate мне пришлось бы позвонить моему поставщику, чтобы сравнить результат с нулем, а затем снова позвонить ему, как только метод find вернет поставщика.

Answer 1

Учитывая ваш комментарий Успокоить ответ Шторма (желание не называть Supplier.get() дважды) , то как:

private static final Function<Supplier<X>, X> SUPPLY = new Function<....>() { 
    public X apply(Supplier<X> in) { 
     // If you will never have a null Supplier, you can skip the test; 
     // otherwise, null Supplier will be treated same as one that returns null 
     // from get(), i.e. skipped 
     return (in == null) ? null : in.get(); 
    } 
} 

затем

Iterable<Supplier<X>> suppliers = ... wherever this comes from ... 

Iterable<X> supplied = Iterables.transform(suppliers, SUPPLY); 

X first = Iterables.find(supplied, Predicates.notNull()); 

отмечают, что Ite который исходит из Iterables.transform(), оценивается лениво, поэтому, как Iterables.find() петли над ним, вы оцениваете только первый, не null, - возвратный, и только один раз.

Answer 2

Что в этом плохого?

List<Supplier> supplierList = //somehow get the list 
Supplier s = Iterables.find(supplierList, new Predicate<Supplier>(){ 
    boolean apply(Supplier supplier) { 
     return supplier.isSomeMethodCall() == null; 
    } 
    boolean equals(Object o) { 
     return false; 
    } 
}); 

Вы пытаетесь сохранить некоторые строки? Единственная оптимизация, о которой я могу думать, - это статический импорт находки, чтобы вы могли избавиться от «Итераторов». Кроме того, предикат является анонимным внутренним классом, если вам это нужно в более чем одном месте вы можете создать класс, и это будет выглядеть так,

List<Supplier> supplierList = //somehow get the list 
Supplier s = find(supplierList, new SupplierPredicateFinder()); 

Где SupplierPredicateFinder другой класс.

ОБНОВЛЕНИЕ: В этом случае поиск является неправильным методом. Вам действительно нужна такая настраиваемая функция, которая может вернуть два значения. Если вы используете коллекцию commons, вы можете использовать DefaultMapEntry или просто можете вернуть объект [2] или Map.Entry.

public static DefaultMapEntry getSupplier(List<Supplier> list) { 
    for(Supplier s : list) { 
     Object heavyObject = s.invokeCostlyMethod(); 
     if(heavyObject != null) { 
      return new DefaultMapEntry(s, heavyObject); 
     } 
    } 
} 

Заменить DefaultMapEntry со списком размером 2 или HashMap размера 1 или массив длиной 2 :)

Answer 3

Вы спрашивали, как это сделать с помощью Google Collections, но вот как вы это сделаете, не используя Google Collections. Сравните это с ответом Коуэна (что является хорошим ответом) - что легче понять?

private static Thing findThing(List<Supplier<Thing>> thingSuppliers) { 
    for (Supplier<Thing> supplier : thingSuppliers) { 
    Thing thing = supplier.get(); 
    if (thing != null) { 
     return thing; 
    } 
    } 
    // throw exception or return null 
} 

Вместо комментария - если это произошло по вине вызывающему вашего класса, бросить IllegalArgumentException или IllegalStateException в зависимости от обстоятельств; если этого не должно было быть, используйте AssertionError; если это нормально, ваш код, который вызывает это ожидание, должен проверяться, вы можете вернуть null.