Идея об основном потоке и выходах по ошибкам

[beefeather]

Конечно, совсем не все, но многие методы устроены так: проверяем, делаем, еще проверяем, еще делаем.

Мне нравится стиль, при котором штатные действия записываются без отступа, а обработка сбоев - внутри if-блоков. Можно ведь и наоборот, но, мне кажется, таким образом хорошо разделяются основное русло выполнения и вспомогательная работа.

Вот так:

List collectLinks(Graph graph) {

if (graph == null) {
throw new IllegalArgumentException("graph is null");
}

Enumeration linkEnum = graph.allLinks();

if (!linkEnum.hasMoreElements()) {
return Collections.emptyList();
}

ArrayList result = new ArrayList();
while (linkEnum.hasMoreElements()) {
Link nextLink = linkEnum.nextElement();

if (!nextLink.isResolved()) {
continue;
}

result.add(nextLink);
}

return result;
}

А еще внутри циклов тоже.