Joe Germuska wrote the following on 2/19/2005 11:56 AM:
The biggest gain I would see out of it would be a further step towards reducing dependencies on the HTTP API, which, despite your skepticism ;-) I think is a worthy goal in itself.
Oh trust me, I'm all for it reducing dependency on the HTTP API and I really like the introduction for CoR, 100%. That's why I really like the backing-bean concept of JSF. Without Struts having that, yes, I am skeptical that you'll end up being that far removed from the HTTP API.
The biggest 'ugliness' in my opinion that I don't like looking at in even my own Struts apps is the application logic stuff that gets put in my Actions. Somewhere down the line, something comes up and you end up having to call different business class methods based on ActionForm parameters or session/request parameters.
For example
someDispatchMethod(...) {(User)request.getSession().getAttribute("user");
if ( user.isSomething() ) {
//call this business method
//if you get something odd back...
//we need to add some flag into scope
reuqest.setAttribute("something",whatever);
else {
//do the usual
//call business method
//look at return and realize we need to set up the next
//page we go to with something unique bc X condition was met
//if something
//so call some other business method to get me this List and pop
request.setAttribute("neededList", list )
//else maybe return from business was fine no need to set up request
//leave
}We can say we don't like this stuff above and I agree, it's ugly. But it does come up. Sometimes I push this kind of stuff off to an intermediate class but since this class still is totally struts and HTTP dependent it doesn't help me that much. (Again I think using a regular backing bean like JSF does goes a long way to making this all much cleaner, but I'm not willing to start using JSF full time for anything yet, since Struts does the job for me.)
-- Rick
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
