komalbansal1991 / hibernate-generic-dao

Automatically exported from code.google.com/p/hibernate-generic-dao
0 stars 0 forks source link

Possible conflict between GenericDAO and GenericDAOImpl #92

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
NOTE: If you have a question rather than a bug report or feature request,
please post at http://groups.google.com/group/java-generic-dao .

What steps will reproduce the problem?
1. Just create a simple Dao interface extending GenericDAO and a DAO impl
2.
3.

What is the expected output? What do you see instead?
find(Serializable) in GenericDAOImpl clashes with find(ID) in GenericDAO.

What version of the product are you using? On what operating system?
Generic DAO version 1.0 and 1.1

Please provide any additional information below.

I was able to resolve/work around this issue by using my own GenericDAOImpl 
that has a much better method signature for find() methods and other methods.  
I simply made the customized GenericDAOImpl find(..) etc methods to use the ID 
generic type instead of the Serializable.

Regards,
Jeff

Original issue reported on code.google.com by jeff.l...@designhaiku.com on 25 Oct 2011 at 2:47

Attachments:

GoogleCodeExporter commented 9 years ago
It does seem like this is a problem. I don't remember if there was a reason why 
I originally made the implementation the way I did or not. What sort of error 
did this create for you? Did this create a compile-time error or and IDE 
warning or what?

We should probably try changing it like you suggest and see if it causes 
problems for anybody. I don't think it will.

Original comment by dwolvert on 3 Feb 2012 at 6:27

GoogleCodeExporter commented 9 years ago

Original comment by dwolvert on 4 Feb 2012 at 4:39

GoogleCodeExporter commented 9 years ago

Original comment by dwolvert on 4 Feb 2012 at 4:40

GoogleCodeExporter commented 9 years ago

Original comment by dwolvert on 15 Feb 2013 at 1:55

GoogleCodeExporter commented 9 years ago

Original comment by dwolvert on 29 Apr 2013 at 2:36