Need help trying to figure this problem out. Username validate on servlet.

Question

Hello.
Here is my dilema. Almost everything works but the username check servlet.
I have form that gathers information that sends to a bean to validate the forms fields. If the fields check out ok it than forwards to Vusername servlet which checks the username in the database to make sure there is not one of the same username in there. If there is the servlet redirects to a retry page notifying the user that the username is already taken. If successfull the servlet forwards to another servlet which processes all the registration information into the database.
As i said everything works fine but the Vusername servlet. I dont think the problem is with my sql query which checks for an exsiting username with the one that was entered into the form.
I think the problem is with my if statements. I think they are currently checking if the sql query has ran. If true do this if false do that. Which is not right.
I need the if statments to check against the result of the sql query being true or false and than redirect depending on those variables.
Anyway any insight would be greatly appreciated.
Here is a copy of the Vusername servlet.
Thanks.
// Andy Janow
// 03/01/05
// Make sure the username is available.
import java.io.*;
import java.net.*;
import javax.servlet.*;
import javax.servlet.http.*;
import java.util.*;
import java.sql.*;
import Sql;
public class Vusername extends Sql
public void doPost( HttpServletRequest req, HttpServletResponse res )
throws ServletException, IOException
     String user;
     String pass1;
     String pass2;
     String email;
     String sError = "/Error.jsp";
     String Failure = "You need to retry";
     String fUsername = "/usernameError.jsp";
     String fUserOb = "Username is in use already";
     // Get the username from the bean.
  user = req.getParameter("userName");
  pass1 = req.getParameter("password1");
  pass2 = req.getParameter("password2");
  email = req.getParameter("emailAddress");
  // Creating the string to check available username.  
  boolean isTaken = isUserNameTaken(user);
  // Creating the forward on error due to unavailable username selection.
   if (isTaken = true);
    req.setAttribute("fUserOb",fUserOb);
       RequestDispatcher rd = getServletContext().getRequestDispatcher(fUsername);
       rd.forward(req, res);
  // Creating the forward on Error page.
   if (isTaken = false);
    req.setAttribute("Failure", Failure);
       RequestDispatcher rd = getServletContext().getRequestDispatcher(sError);
       rd.include(req, res);
// Sql QUERY for username.
public boolean isUserNameTaken(String username)
    boolean isTaken = false;
    PreparedStatement ps = null;
    ResultSet rs = null;
    try
        ps = connection.prepareStatement("SELECT userName FROM Registration WHERE userName = ?");
        ps.setString(1,username);
        rs = ps.executeQuery();
        isTaken = rs.next(); // returns true if there is a row returned
    catch (SQLException e)
        e.printStackTrace();
    finally
    { // ALWAYS clean up your resources
        try
            if (rs != null) rs.close();
        } catch (SQLException e) { /* close quietly */ }
        try
            if (ps != null) ps.close();
        } catch (SQLException e) { /* close quietly */ }
    return isTaken;
public void destroy()
  try
   connection.close(); // Closing the db connection (cleanup).
  catch( Exception e )
   System.err.println( "Problem closing the database" );
}

Answer 1

Thanks for the repli.
I forgot to come back here and fix the topic to issue solved.
In the past hour sience i posted i was able to work through and fix the problem.
Was with my if statment... Just an error of typing to fast i suppose and not really going through my code very thouroughly.
Think how i had the code posted i was using the
if statment to check if the sql statement was true , than another if statment to check if was false..
Thing is, is that i already was checking if the username was in the database or not in the actual sql statment block by assinging weather the it was (true) or it was not (false) to the resultset variable.
So in my if statments higher up in the code i just needed to reformat them correctly, from if true do this, and if false do this... To if some variable is present do this, else do this.
And now it works great.
Just in case anyone was wondering.
But thanks for the repli and interest :)