Review

Loops over a collection, only to ensure it is not empty (every subsequent iteration performs the exact same computation).

Replace the loop with a check.

• superfluous/iteration

Repetitive.java

misuse(Collection)

class Repetitive {
	public String misuse(Collection<Object> objects) {
		String value = null;
		for (Object object : objects) {
			value = objects.iterator().next().toString();
		}
		return value;
	}

}

public class OnlyOnce {
	void pattern(Collection<Object> objects) {
		if (!objects.isEmpty()) {
			objects.iterator().next();
		}
	}
}