[#GROOVY-1242] Cant have a class method returning an array of complex type - jira.codehaus.org

Details

Type: Bug
Status: Closed
Priority: Blocker
Resolution: Fixed
Affects Version/s: 1.0-JSR-5
Fix Version/s: 1.0-JSR-6
Component/s: class generator
Labels:
None
Environment:
Windows XP/SP2 + jdk1.5.0_06

Number of attachments :
0

Description

Here is a script returning a Caught: java.lang.ClassFormatError: Illegal field name "class$Person_]" in class PersonServiceImpl

class Person {
@Property String firstName
@Property int id

String toString() {"${firstName} has id ${id}"}
}

class PersonServiceImpl {
private List persons = new ArrayList()

PersonServiceImpl() { persons = [ new Person(firstName:"Guillaume", id:1), new Person(firstName:"Jochen", id:2), new Person(firstName:"Dierk", id:3), new Person(firstName:"Graeme", id:4)] }

Person[] getPersons() { return persons.toArray(new Person[persons.size()]) }

Person findPerson(int id) { return persons[id] }
}

def ps = new PersonServiceImpl()
println ps.findPerson(0)

NOTA1: Removung the Person[] getPersons() {} blocks makes the script working
NOTA2: Replacing the Person[] type by a native type (int for example) makes things going smooth.

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

blackdrag blackdrag added a comment - 14/Feb/06 9:45 AM

fixed now

Show

blackdrag blackdrag added a comment - 14/Feb/06 9:45 AM fixed now

Hide

Permalink

Guillaume ALLEON added a comment - 15/Feb/06 5:39 AM

Still not working got the following exception
Caught: java.lang.NullPointerException
at PersonServiceImpl.getPersons(JIRA_1242.groovy:25)
at JIRA_1242.run(JIRA_1242.groovy:43)
at JIRA_1242.main(JIRA_1242.groovy)

while the same construction is working for asic types.

see script

class Person {
@Property String firstName
@Property int id

String toString() {"${firstName} has id ${id}"}
}

class PersonServiceImpl {
private List persons = new ArrayList()
private List numbers = new ArrayList()

PersonServiceImpl() { persons = [ new Person(firstName:"Guillaume", id:1), new Person(firstName:"Jochen", id:2), new Person(firstName:"Dierk", id:3), new Person(firstName:"Graeme", id:4)] numbers = [1, 2, 3, 4] }

Person[] getPersons() { println persons.size() return (Person[])persons.toArray(new Person[persons.size()]) }

int[] getTruc() { return (int[])numbers.toArray(new int[numbers.size()]) }

Person findPerson(int id) { return persons[id] }

}

def ps = new PersonServiceImpl()
println ps.findPerson(0)
aa = ps.getTruc()
aa.each{println "${it}"}
bb = ps.getPersons()

Show

Guillaume ALLEON added a comment - 15/Feb/06 5:39 AM Still not working got the following exception Caught: java.lang.NullPointerException at PersonServiceImpl.getPersons(JIRA_1242.groovy:25) at JIRA_1242.run(JIRA_1242.groovy:43) at JIRA_1242.main(JIRA_1242.groovy) while the same construction is working for asic types. see script class Person { @Property String firstName @Property int id String toString() {"${firstName} has id ${id}"} } class PersonServiceImpl { private List persons = new ArrayList() private List numbers = new ArrayList() PersonServiceImpl() { persons = [ new Person(firstName:"Guillaume", id:1), new Person(firstName:"Jochen", id:2), new Person(firstName:"Dierk", id:3), new Person(firstName:"Graeme", id:4)] numbers = [1, 2, 3, 4] } Person[] getPersons() { println persons.size() return (Person[])persons.toArray(new Person[persons.size()]) } int[] getTruc() { return (int[])numbers.toArray(new int[numbers.size()]) } Person findPerson(int id) { return persons[id] } } def ps = new PersonServiceImpl() println ps.findPerson(0) aa = ps.getTruc() aa.each{println "${it}"} bb = ps.getPersons()

Hide

Permalink

blackdrag blackdrag added a comment - 15/Feb/06 6:33 AM

ok, that is 1 a new error and 2 a duplicate I know. The vargs implementation is a bit buggy ... it seems I haven't added tests for this... stupid me.

Show

blackdrag blackdrag added a comment - 15/Feb/06 6:33 AM ok, that is 1 a new error and 2 a duplicate I know. The vargs implementation is a bit buggy ... it seems I haven't added tests for this... stupid me.

Hide

Permalink

blackdrag blackdrag added a comment - 04/Mar/06 5:04 PM

as I have fixed that part of the vargs implementation it should work now

Show

blackdrag blackdrag added a comment - 04/Mar/06 5:04 PM as I have fixed that part of the vargs implementation it should work now

People

Assignee:

blackdrag blackdrag

Reporter:

Guillaume ALLEON

Vote (0)

Watch (0)

Dates

Created:

14/Feb/06 7:40 AM

Updated:

04/Mar/06 5:04 PM

Resolved:

04/Mar/06 5:04 PM