Please consider using a service like pastebin or whatever so the line numbers actually make sense when we look at it. I have no idea which line 92 is.
This code: List loadDomainType = DomainDAO.loadDomainType(); dtList=loadDomainType; confuses me; couldn't you just say: dtList = DomainDAO.loadDomainType(); This code: public String addupDomain(){ String result = ERROR; if (listbean != null) { try { if (DomainDAO.addupDomain(listbean)) { result = SUCCESS; } else { result=ERROR; } } catch (Exception e) { result = null; listbean = null; } } return result; } seems weird to me--do you *really* want to return a null result?! Personally, I find the method a little hard to read anyway--too much work to get to its essence. In any case, without being able to line up your line numbers, it's tough to help. Dave --------------------------------------------------------------------- To unsubscribe, e-mail: user-unsubscr...@struts.apache.org For additional commands, e-mail: user-h...@struts.apache.org