Yep, it's all dependent on how well you cache. You have to be very careful with type conversion code though, as you're not directly controlling when it is called and it can easily be called many times.
> -----Original Message----- > From: Fred Lamuette [mailto:[EMAIL PROTECTED] > Sent: Wednesday, January 28, 2004 4:06 PM > To: [EMAIL PROTECTED] > Subject: RE: [OS-webwork] select tag > > > I disagree with your point of view, Jason. In some cases, to > reach a clearer code you could do database lookup in type > converters, above all when you have an efficient cache system > (e.g. Hibernate + JVM-level cache) ... Richard. > > -----Message d'origine----- > De : [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] > la part de Jason Carreira Envoye : mercredi 28 janvier 2004 > 19:52 A : [EMAIL PROTECTED] > Objet : RE: [OS-webwork] select tag > > > This is a difficult issue... Maybe we can get the Atlassian > guys to talk about what they've been doing in Confluence? > > The reason doing database lookups in type converters is bad > is because you're not in control of when it happens... It > could be that it happens 1000's of times. > > Jason > > > -----Original Message----- > > From: Dag Liodden [mailto:[EMAIL PROTECTED] > > > > Even if this example had only been about editing primitive (or > > primitive > > wrappers) properties of the Employee instance there would have been > > trouble. The simplest approach is to instanciate a > temporary instance > > of the Employee class in the action and let the interceptors set > > the values > > to that instance, but then you'll have to copy the properties > > manually > > when the action executed later. (a beanutils library will > > help in some > > cases, but what about nulls and transient instances?). > > > > The PetSoar example does not provide any pointers - it does manual > > copying and avoids this issue. Some debate on this subject would be > > very welcome! If we get a good and simple pattern for this, > it would > > be a killer feature. > > > > One last point BTW. The type conversion doco states that it's a bad > > idea to do database lookups. Can someone elaborate, please? :) > > > > Cheers, > > > > Dag > > > > > ------------------------------------------------------- > The SF.Net email is sponsored by EclipseCon 2004 > Premiere Conference on Open Tools Development and Integration > See the breadth of Eclipse activity. February 3-5 in Anaheim, > CA. http://www.eclipsecon.org/osdn > _______________________________________________ > Opensymphony-webwork mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork > > > > ------------------------------------------------------- > The SF.Net email is sponsored by EclipseCon 2004 > Premiere Conference on Open Tools Development and Integration > See the breadth of Eclipse activity. February 3-5 in Anaheim, > CA. http://www.eclipsecon.org/osdn > _______________________________________________ > Opensymphony-webwork mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork > ------------------------------------------------------- The SF.Net email is sponsored by EclipseCon 2004 Premiere Conference on Open Tools Development and Integration See the breadth of Eclipse activity. February 3-5 in Anaheim, CA. http://www.eclipsecon.org/osdn _______________________________________________ Opensymphony-webwork mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork