A quick look at the code... you use open_! That's considered very bad practice. Any time you use open_!, you are saying "I know this thing contains a value." It's like not doing null testing when the method you are calling is expected to return a null under certain circumstance. Put another way, the "!" is in the name for a reason. It's dangerous... think before using.
On Thu, Oct 1, 2009 at 2:14 AM, rintcius <[email protected]> wrote: > > Hi, > > I have started integrating Lift in a Scala + Spring example project > (see http://code.google.com/p/scala-spring). This first integration > just has a single template and snippet for now but should give an idea > how Lift can be integrated. > Before I go further with this I would like some Lift experts to review > the code I have written so far. > Anybody? The changeset that adds lift integration is in > http://code.google.com/p/scala-spring/source/detail?r=18 > > Thanks, Rintcius > > > > -- Lift, the simply functional web framework http://liftweb.net Beginning Scala http://www.apress.com/book/view/1430219890 Follow me: http://twitter.com/dpp Surf the harmonics --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Lift" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~----------~----~----~----~------~----~------~--~---
