[GROOVY-886] each(){} throws NPE when called on a Java Object[], that contains String[]'s with null entries

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0-JSR-1
• Fix Version/s: 1.0-JSR-3
• Assignee: Kim, Pilho
• Reporter: Dierk König
• Created: 14/Jun/05 05:18
• Updated: 16/Jul/05 15:21
• Resolved: 16/Jul/05 15:21

Description

Say you have a Java Class like

public class SomeClass {
public Object[] anArrayOfStringArraysWorkaround(){
return new Object[]{ new String[]{"whatever",null}};
}
}

then the following Groovy code fails with NPE

public class MultiDimArraysTest extends GroovyTestCase {
void testCallTwoDimStringArrayWorkaroundWithNull(){
def someArrayOfStringArrays = new SomeClass().anArrayOfStringArraysWorkaround()
assert 1 == someArrayOfStringArrays.size()
assert "whatever" == someArrayOfStringArrays[0][0]
someArrayOfStringArrays.each()

{ assert it}

// throws NPE !!
}
}

while the iteration with "for" works:

public class MultiDimArraysTest extends GroovyTestCase {
void testCallTwoDimStringArrayWorkaround(){
def someArrayOfStringArrays = new SomeClass().anArrayOfStringArraysWorkaround()
assert 1 == someArrayOfStringArrays.size()
assert "whatever" == someArrayOfStringArrays[0][0]
for (i in 0..<someArrayOfStringArrays.size())

{ assert someArrayOfStringArrays[i] }

}
}

The above TestCase will be committed.

Issue Links

• duplicates: GROOVY-584 Exception passing array into Closure
• is depended upon by: GROOVY-764 closures
• is duplicated by: GROOVY-888 Object[] vs int[]