Yea, I can see how that would work. You're just saying to put the
"black box" in the ListView, right? Your DataProvider interface is a
bit too simple (doesn't allow ordering, and EJB and Hibernate2 can't
wrap an object in a model with only the object itself for reference),
but I think it could work nicely with some fleshing out.

Returning an Iterator is very nice. If all you need to do is iterate,
there's no reason to require a List. But because I'm implementing a
List, there's no way to incorporate that. However, returning a list
WILL use the cache, just not by default. You have to set useQueryCache
to true.

-Phil

On 7/29/05, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
> The difference is my segment of code is the entire implementation, you are
> implementing a few abstract methods on a surface of a black box.
> 
> -Igor
> 
> 
> > -----Original Message-----
> > From: [EMAIL PROTECTED]
> > [mailto:[EMAIL PROTECTED] On Behalf Of
> > Phil Kulak
> > Sent: Friday, July 29, 2005 5:03 PM
> > To: wicket-user@lists.sourceforge.net
> > Subject: Re: [Wicket-user] lists
> >
> > On 7/29/05, Christian Essl <[EMAIL PROTECTED]> wrote:
> > > Igor needs an eight-liner where you need a helper-class which is
> > > complicated enough to put in an extra contribution-package.
> > > OrderedPageList adds some indirection to get something
> > which is less
> > > efficient and more work to use.
> >
> > My "eight-liner" lookes almost exactly the same, (except for
> > a few more lines of course):
> >
> > new QueryList() {
> >   protected List getItems(int start, int max, String orderBy) {
> >     session.createQuery("FROM Entity e" + orderBy)
> >       .setFirstResult(start).setMaxResults(max).list();
> >   }
> >
> >   protected int getCount() {
> >     return (Integer) session
> >       .createQuery("SELECT COUNT(*) FROM Entity e")
> >       .uniqueResult();
> >   }
> > }
> >
> > Or how about the same thing as a one-liner?
> >
> > new HibernateQueryList("FROM Entity e", dao);
> >
> > > Sure you have a List, with all those nice methods
> > unfortunately many
> > > are not supported and many are not recommended to use. To me
> > > OrderedPageList is not a List it is a DataProvider which happens to
> > > implement List because of ListView. Of course the user must
> > know and
> > > this is exactly what the user does with one spot when using
> > Igors interface.
> >
> > To me a list IS a data provider. Sometimes the data is in an
> > array, sometimes linked objects, sometimes a database... But
> > it's all good.
> > Like Igor said, this is all just in the contrib, so we can
> > just use what we're comfortable with. I'm just happy that
> > people are taking this stuff so seriosly. I think that really
> > shows that Wicket is off to a great start.
> >
> >
> > -------------------------------------------------------
> > SF.Net email is sponsored by: Discover Easy Linux Migration
> > Strategies from IBM. Find simple to follow Roadmaps,
> > straightforward articles, informative Webcasts and more! Get
> > everything you need to get up to speed, fast.
> > http://ads.osdn.com/?ad_idt77&alloc_id492&op=ick
> > _______________________________________________
> > Wicket-user mailing list
> > Wicket-user@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/wicket-user
> >
> >
> >
> 
> 
> 
> 
> -------------------------------------------------------
> SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
> from IBM. Find simple to follow Roadmaps, straightforward articles,
> informative Webcasts and more! Get everything you need to get up to
> speed, fast. http://ads.osdn.com/?ad_idt77&alloc_id492&opclick
> _______________________________________________
> Wicket-user mailing list
> Wicket-user@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/wicket-user
>


-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_idt77&alloc_id492&op=click
_______________________________________________
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

Reply via email to