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

Reply via email to