Yup, sounds like I misjudged the appetite for a change like this. That's ok,
half the benefit of digging into this was getting to know the insides of my new
set of tools (Stripes).
I'll shelve my work on this for a while until I've got more time under my belt
with the framework, maybe I'll come to love the stateful bean paradigm too.
I also did some work on Rest/Http verb binding to handlers. Is this something
folks have more interest in? I sent a patch to the dev list but didn't get much
feedback. Here's what that looks like. This was a much smaller change :)
@UrlBinding("/RestAction")
public class RestActionBean implements ActionBean {
@RestBinding("PUT")
public Resolution myPutHandler() {
log.info("Got a Put");
return null;
}
}
A PUT request to "/yourapp/RestAction", will be routed to the "myPutHandler"
method.
Evan
On Oct 7, 2010, at 7:02 PM, Ross Sargant wrote:
> Fair enough, but I think that is largely a matter of personal preference.
>
> If you split a complex action bean whose events currently share bean
> properties into multiple beans, then you'll probably wind up with
> duplication across the beans. Possibly even more then you would having to
> duplicate event names in "on={...}" lists or otherwise. That starts to get
> really bad if it forces you into duplicating validation rules.
>
> Still, I agree that having a single large action bean whose events handle
> large sets of independent properties probably isn't the greatest thing
> maintenance wise .
>
> I just find the on={} construct a little awkward to work with when you are in
> the "grey" area and there is some significant (but not total) cross-over
> between the events . It really isn't a big deal though.
>
> Changing stripes to event method parameter binding feels like a real paradigm
> shift. Nothing wrong with that if there are good reasons to do it. I'm just
> not sure what they are and it doesn't sound like I'm the only one..
>
>
>
>
>
> 2010/10/7 Grzegorz Krugły <g...@karko.net>
> Just my 2 cents - I think that if You have many action fields that are
> not used in some of the events, Your action is too big and should be split.
>
> Solutions to the problem involving additional annotations and other ways
> of duplicating field names in another configuration structures seem ugly
> to me -- they just add more complexity to a complex action bean that
> should be simplified instead so it remains maintainable.
>
>
> ------------------------------------------------------------------------------
> Beautiful is writing same markup. Internet Explorer 9 supports
> standards for HTML5, CSS3, SVG 1.1, ECMAScript5, and DOM L2 & L3.
> Spend less time writing and rewriting code and more time creating great
> experiences on the web. Be a part of the beta today.
> http://p.sf.net/sfu/beautyoftheweb
> _______________________________________________
> Stripes-users mailing list
> Stripes-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/stripes-users
>
>
>
> --
> Ross Sargant
> Software Engineer
> p: 954-623-6015 x2108
> email: rsarg...@tvrc.com
>
> TVR Communications LLC
> 541 S. State Road 7,Suite 5,Margate, Florida,33068
>
> http://www.tvrc.com
>
> ------------------------------------------------------------------------------
> Beautiful is writing same markup. Internet Explorer 9 supports
> standards for HTML5, CSS3, SVG 1.1, ECMAScript5, and DOM L2 & L3.
> Spend less time writing and rewriting code and more time creating great
> experiences on the web. Be a part of the beta today.
> http://p.sf.net/sfu/beautyoftheweb_______________________________________________
> Stripes-users mailing list
> Stripes-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/stripes-users
------------------------------------------------------------------------------
Beautiful is writing same markup. Internet Explorer 9 supports
standards for HTML5, CSS3, SVG 1.1, ECMAScript5, and DOM L2 & L3.
Spend less time writing and rewriting code and more time creating great
experiences on the web. Be a part of the beta today.
http://p.sf.net/sfu/beautyoftheweb
_______________________________________________
Stripes-users mailing list
Stripes-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/stripes-users
