Actually, the DBInterceptor has a major flaw... 5 points to anyone who spots it before looking down....
It keeps the Connection in the Interceptor and expects it to be the same one in after() as was created in before(). Unless you're putting it in a ThreadLocal, this is not thread safe. My suggestion (other than not managing JDBC connections like this) would be to put things like this in the ActionContext so that they're tied to the Thread. The other problem is that exceptions may cause after() to not be called, so your Connection would not be closed. > -----Original Message----- > From: Mike Cannon-Brookes [mailto:[EMAIL PROTECTED] > Sent: Saturday, September 20, 2003 11:14 PM > To: [EMAIL PROTECTED] > Subject: Re: [OS-webwork] WebWork vs. Struts > > > A good presentation - a lot of familiar content :) but some > good new things too. I like the DBInterceptor (bad pattern, > but good example ;)). > > I've added this any the link to my old presentation to the > WW2 page under > Misc: > > http://wiki.opensymphony.com/space/WebWork2 > > Cheers, > Mike > > On 21/9/03 12:54 PM, "Peter White" ([EMAIL PROTECTED]) penned > the words: > > > I stumbled across this powerpoint presentation on the web > which might > > be of > > interest: > http://www.groovesystems.com/training/java4/webwork.ppt. It's > > mostly a presentation on WebWork2 with a brief comparison > to Struts at the > > end... > > > >> -----Original Message----- > >> From: [EMAIL PROTECTED] > >> [mailto:[EMAIL PROTECTED] > On Behalf > >> Of Bob Lee > >> Sent: Saturday, September 20, 2003 5:41 PM > >> To: [EMAIL PROTECTED] > >> Subject: [OS-webwork] WebWork vs. Struts > >> > >> Does someone have a really good WebWork vs. Struts comparison > >> (specifically the benefits WebWork has over Struts). My employer > >> chose Struts, but we haven't started development yet and > I've talked > >> them into letting me put together a proposal to use > WebWork instead. > >> > >> Thanks, > >> Bob > >> > >> > >> > >> ------------------------------------------------------- > >> This sf.net email is sponsored by:ThinkGeek Welcome to > geek heaven. > >> http://thinkgeek.com/sf > >> _______________________________________________ > >> Opensymphony-webwork mailing list > >> [EMAIL PROTECTED] > >> https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork > >> > > > > > > > > > > ------------------------------------------------------- > > This sf.net email is sponsored by:ThinkGeek > > Welcome to geek heaven. > > http://thinkgeek.com/sf > > _______________________________________________ > > Opensymphony-webwork mailing list > > [EMAIL PROTECTED] > > https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork > > > > ------------------------------------------------------- > This sf.net email is sponsored by:ThinkGeek > Welcome to geek heaven. > http://thinkgeek.com/sf > _______________________________________________ > Opensymphony-webwork mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork > ------------------------------------------------------- This sf.net email is sponsored by:ThinkGeek Welcome to geek heaven. http://thinkgeek.com/sf _______________________________________________ Opensymphony-webwork mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork
