Skip to content

Commit 741f3bb

Browse files
Replace for loop with enhanced for loop (assertj#2501)
1 parent aa738bf commit 741f3bb

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/main/java/org/assertj/core/api/ClassBasedNavigableIterableAssert.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,7 @@ private <V> ELEMENT_ASSERT buildAssert(V value, String description, Class<?> cla
4949
try {
5050
Constructor<?>[] declaredConstructors = assertClass.getDeclaredConstructors();
5151
// find a matching Assert constructor for E or one of its subclass.
52-
for (int i = 0; i < declaredConstructors.length; i++) {
53-
Constructor<?> constructor = declaredConstructors[i];
52+
for (Constructor<?> constructor : declaredConstructors) {
5453
if (constructor.getParameterTypes().length == 1 && constructor.getParameterTypes()[0].isAssignableFrom(clazz)) {
5554
@SuppressWarnings("unchecked")
5655
ELEMENT_ASSERT newAssert = (ELEMENT_ASSERT) constructor.newInstance(value);

src/main/java/org/assertj/core/presentation/StandardRepresentation.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -323,8 +323,8 @@ protected boolean hasCustomFormatterFor(Object object) {
323323
*/
324324
// not static so that it can be overridden
325325
protected boolean hasAlreadyAnUnambiguousToStringOf(Object obj) {
326-
for (int i = 0; i < TYPE_WITH_UNAMBIGUOUS_REPRESENTATION.length; i++) {
327-
if (TYPE_WITH_UNAMBIGUOUS_REPRESENTATION[i].isInstance(obj)) return true;
326+
for (Class<?> aClass : TYPE_WITH_UNAMBIGUOUS_REPRESENTATION) {
327+
if (aClass.isInstance(obj)) return true;
328328
}
329329
return false;
330330
}

0 commit comments

Comments
 (0)