Точка стиля с Try/уловом

Question

мне было интересно, если этот подход был правильным:

public ITask getState() 
{ 
    statePredicate[Some predicate definition]; 
    ITask nextRunnable = null; 
try { 
    nextRunnable = Iterables.find((Iterable)queue, statePredicate); 
} 
    catch (NoSuchElementException e) 
    {} 
return nextRunnable; 
} 

Точки, на которых мне интересно, являются:

• следует сказуемое кэшируются в качестве члена класса?
• Я ничего не делаю с уловом, я даже не регистрирую его, потому что это normal для моего приложения, чтобы ничего не находить.
• t возврат null, потому что я делаю окончательное возвращение.

Благодарим за внимание! -

Answer 1

1) Если предикат всегда один и тот же, я бы сделал его членом класса static final.

2) Существует также версия Iterables.find, в которой вы можете указать значение по умолчанию (при условии, что вы используете Google Guava). Тогда вам не нужно иметь дело с NoSuchElementException.

3) Есть ли причина для отливки queue - Iterable? Если это не обязательно, то не бросайте.

class MyClass { 
    private static final Predicate STATE_PREDICATE = new Predicate<ITask>() { 
     @Override 
     public boolean apply(ITask input) { 
      // ... your code here 
     } 
    }; 

    public ITask getState() { 
     return Iterables.find(queue, STATE_PREDICATE, null); 
    } 
} 

Answer 2

Если исключение действительно является обычным случаем в вашем подходе, вы должны поместить хотя бы комментарий в область улова, чтобы дать понять всем, кто читает код, который был преднамеренным, а не ошибкой. По-моему, возвращение Null - это нечто иное, но это не случайно.