• Post Reply
  • Bookmark Topic Watch Topic
  • New Topic

reflection and overloading, java.lang.IllegalArgumentException: wrong number of arguments

 
Alessandro Ilardo
Ranch Hand
Posts: 218
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
I am testing an ejb with a client which uses the Junit framework.
On the ejb I have two public methods

public User getUser(final String email, final String domain)
public User getUser(final Long id)

the client code is something like this


called by a junit test


The test doesn't complete correctly and I get this exception
834 [main] FATAL eu.virtualLab.security.user.test.util.ServiceLocator - java.lang.IllegalArgumentException: wrong number of arguments
835 [main] FATAL eu.virtualLab.security.user.test.junit.JunitUserActions - [testGetUser] wrong number of arguments

Could you please give me some help about it?
Thanks in advance
 
Campbell Ritchie
Sheriff
Posts: 48377
56
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Don't know, sorry, but that looks too difficult for us beginners. Moving.
 
Steve Luke
Bartender
Posts: 4181
21
IntelliJ IDE Java Python
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
It looks to me like you have to find a better way of caching methods. You have 2 methods with different numbers of parameters. Whichever is the first one that is searched for will be cached and returned for any time another method with the same method name is called. This will cause problems when you would have overloaded methods with different single parameters as well (for example a getUser(Long) and a getUser(Integer)).

I would suggest putting more information in the cache's key, for example the method name and the list of parameter types. For example this would be a utility method that could help you generate a more useful key:


You could call it in your getRemoteMethod() method like:

 
Alessandro Ilardo
Ranch Hand
Posts: 218
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Thank you, you're right
 
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic