Question

我想知道这种方法是否正确：

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

我想知道的是：

谓词是否应该作为类的成员进行缓存？
我对捕获没有任何作用，我甚至没有记录它，因为它是正常，我的应用程序找不到任何东西。
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

如果异常真的是你的方法中的常见情况，那么你应该至少对catch区域发表评论，以便为每个阅读代码的人明确说明它是故意的而不是错误的。在我看来，返回Null是不同的，但有些情况是不可避免的。

try / catch的风格点

2 个答案: