ок. то что не нравицо мне:
1. prepare, addString, execute. много лишних-ненужных вызовов. достаточно одного.
2. обработка эксепшнов. на..я? кому интересен SQLException, поднимите руки. в подавляющем большинстве случаев это лишние строчки.
3. finally - аналогично. зачем это? чтобы писать в каждом месте? на.уй, на.уй.
1) Видел я вызовы одной строкой (prepare("bla.bla").addString("dsads").execute()). Иногда даже сам делал. Фтопку. Это уже за гранью нормальной лени.
2) Да, возможно, есть такой момент. Мне лично он не нужен. Потом, возможно, переделаю.
3) Не понимаешь, зачем нужен finally? В finalizer'е пишешь нужный код??? Фтопку тебя :)

Сократить тре вызова можно до двух.
Путем prepare("ddsds").execute(userId). Этот подход тоже фтопку.

Кстати, по поводу Exception'ов, может товарищ не в курсе. В EJB нормальной практикой является вызов исключения EJBException внутри методов EJB бина. Чтобы не было мучительно больно, других Exception'ов наружу лучше не выдавать.

мда, что и требовалосьдоказать :)
иди открой уже Spring и не гадай.
Нихачу. В следующем проекте.