As Paul said, you must avoid instance variables in Struts 1. 

I once had a HUGE project that was written with Struts 1. Along with terrible 
code I saw instance variables everywhere. I think over 650 action classes with 
instance variables so I just re-wrote a few lines in the latest Struts 1 source 
code and initialized a new action for every request.  It shot up the server 
memory by a bunch but who cares, memory is cheap compared to me re-writing all 
those classes.

Regards,
Eric




-----Original Message-----
From: Paul Benedict [mailto:pbened...@apache.org] 
Sent: Tuesday, November 04, 2014 4:16 PM
To: Struts Users Mailing List
Subject: Re: Struts 1 Thread safe action classes

To be thread safe, your services should never store user data in instance 
variables. That's all there is to it. Keep everything local.


Cheers,
Paul

On Tue, Nov 4, 2014 at 3:14 PM, Sekar, Sowmya <sse...@ucsd.edu> wrote:

> How do I accomplish that?
>
> -----Original Message-----
> From: Paul Benedict [mailto:pbened...@apache.org]
> Sent: Tuesday, November 04, 2014 1:13 PM
> To: Struts Users Mailing List
> Subject: Re: Struts 1 Thread safe action classes
>
> Your service classes should be thread-safe to begin with. You don't 
> need multiple instances of a business service.
>
>
> Cheers,
> Paul
>
> On Tue, Nov 4, 2014 at 3:08 PM, Sekar, Sowmya <sse...@ucsd.edu> wrote:
>
> > Hi,
> >
> > To invoke service classes from each method in the action classes, 
> > are there any other options to make it thread safe other than 
> > initializing them as local variables?
> >
> > Thanks, Sowmya
> >
>

Reply via email to