Problem with else statement

Question

There is an if statement in this servlet which basically checks for null values, that works grand, however, when I added in an else statement (for the try and catch functions), the servlet let won't run. Is it not possible to embed Try and Catch functions with an else loop?
package client;
import java.io.*;
import javax.servlet.*;
import javax.servlet.http.*;
import java.sql.*;
import business.Client;
import data.ClientDB;
import util.MarshPool;
public class AddClientServlet extends HttpServlet{
    private MarshPool connectionPool;
    public void init() throws ServletException{
        connectionPool = MarshPool.getInstance();
    public void destroy() {
        connectionPool.destroy();
    public void doPost(HttpServletRequest request,
                      HttpServletResponse response)
                      throws IOException, ServletException{
        Connection connection = connectionPool.getConnection();
        String clientName = request.getParameter("clientName");
        String address1 = request.getParameter("address1");
        String address2 = request.getParameter("address2");
        String address3 = request.getParameter("address3");
        String postCode = request.getParameter("postCode");
        String county = request.getParameter("county");
        String country = request.getParameter("country");
        String since = request.getParameter("since");
        String category = request.getParameter("category");
        String units = request.getParameter("units");
        String company = request.getParameter("company");
        String ae = request.getParameter("ae");
          String phoneNo = request.getParameter("phoneNo");
        String faxNo = request.getParameter("faxNo");
        String mobileNo = request.getParameter("mobileNo");
        String emailAddress = request.getParameter("emailAddress");
        String comments = request.getParameter("comments");
        int clientNo = 0;
        int active = 0;
        String message = "";
        HttpSession session = request.getSession();
        Client client = new Client (clientNo, clientName, address1, address2, address3, postCode, county, country, company, units, category, ae, emailAddress, phoneNo, faxNo, mobileNo, comments, since, active);
        session.setAttribute("client", client);
          if ((clientName.length()==0) || (address1.length()==0) ||(category.length()==0) || (units.length()==0) ||(company.length()==0)||(ae.length()==0) ||(country.length()==0) ||(since.length()==0)){
             message = "Please enter values in all the required fields";
             session.setAttribute("message", message);
             RequestDispatcher dispatcher =
                 getServletContext().getRequestDispatcher(
                    "/client/missingfield.jsp");
             dispatcher.forward(request, response);
        else{     
             try{
                ClientDB.addRecord(connection, client);
                    message = "Client entered successfully";
                     session.setAttribute("message", message);
                RequestDispatcher dispatcher =
                        getServletContext().getRequestDispatcher(
                            "/client/added_client.jsp");
                dispatcher.forward(request, response);
             catch(SQLException sqle){
                 message = "EmailServlet SQLException: " + sqle;
                 session.setAttribute("message", message);
                 RequestDispatcher dispatcher =
                     getServletContext().getRequestDispatcher(
                         "/eurosys/");
                 dispatcher.forward(request, response);
        connectionPool.freeConnection(connection);
}

Answer 1

If what you mean is that you used to have:
  if ( clientName == null // ... ... B4 checking their length, then yeah, put that back in. You can't check the length on a null Object.
But why would you remove that? I think you should post the code the way it existed B4 you altered it and let's see the diff.