Dieter Quickfend wrote:Alright. I think you're gonna have to go back to the drawing board for your application.
You've got no separation of concerns in your servlet, which means spaghetti code. One giant method in your servlet, one small setback for the human mind.
You're using instance variables in your servlet, which means none of it's thread safe.
It's time to learn object oriented programming.
And it's also time to learn about the scopes.
Also, don't use DAO's as value objects.