Detectar NULL y Cadena Vacía en un String

[Isdiar]

Buenas tardes,

tengo un formulario dentro de una web en JSP, y los datos del formulario los paso a un objeto "Usuarios". Dicho objeto, lo mando a un método de Java que se encarga de montar la sentencia SQL y de mandarlo a la BD.

Dicho método es el siguiente:

public boolean addUsuario(Usuarios usuario) { // Correcto

String comandoSql;


if(usuario.getNombre()==null || usuario.getApellidos()==null)
return(false);
else if(usuario.getNombre()=="" || usuario.getApellidos()=="")
return(false);
else {
comandoSql = "INSERT INTO usuarios (nombre, apellidos, dni, telefonoParticular, telefonoMovil, correoE, fechaNacimiento, observaciones, contactoFamilia1, telefonoTrabajo, domicilio, codigoPostal, localidad, provincia, pais, contactoFamilia2, contactoFamilia3) VALUES (";
comandoSql += "'" +usuario.getNombre()+ "', ";
comandoSql += "'" +usuario.getApellidos()+ "', ";
comandoSql += "'" +usuario.getDni()+ "', ";
comandoSql += "'" +usuario.getTelefonoParticular()+ "', ";
comandoSql += "'" +usuario.getTelefonoMovil()+ "', ";
comandoSql += "'" +usuario.getCorreoE()+ "', ";
comandoSql += "'" +new Fecha(usuario.getFechaNacimientoString()).getStrin gAnnoMesDia()+ "', ";
comandoSql += "'" +usuario.getObservaciones()+ "', ";
comandoSql += "'" +usuario.getContactoFamilia1()+ "', ";
comandoSql += "'" +usuario.getTelefonoTrabajo()+ "', ";
comandoSql += "'" +usuario.getDomicilio()+ "', ";
comandoSql += "'" +usuario.getCodigoPostal()+ "', ";
comandoSql += "'" +usuario.getLocalidad()+ "', ";
comandoSql += "'" +usuario.getProvincia()+ "', ";
comandoSql += "'" +usuario.getPais()+ "', ";
comandoSql += "'" +usuario.getContactoFamilia2()+ "', ";
comandoSql += "'" +usuario.getContactoFamilia3()+ "'";
comandoSql += ") ";

if (this.realizarAccion(comandoSql) != true)
return(false);
else
return(true);
}

} // END_addUsuario

El problema viene cuando en el formulario JSP no introduzco nombre, pues en teoría, el método "boolean addUsuario(Usuarios)" debería hecharme para atrás dicho usuario, pues es una cadena vacía lo que tiene, pero me lo acepta y lo mete en la BD. ¿Por qué?

Un saludo! Y gracias!

[venkman]

Un error clásico.

La comparación de contenido de cadenas debes hacerla con el método equals:

Código:

... usuario.getNombre().equals("") ...

[Xinef]

Iniciado por venkman

La comparación de contenido de cadenas debes hacerla con el método equals:

Código:

... usuario.getNombre().equals("") ...

Y siempre es recomendable hacer la comparación desde la cadena:

Código PHP:

     "".equals(usuario.getNombre()); 
   

Así te ahorras comparar por nulo... y bastantes NullPointerException.... :)

[Cimballi]

Y puedes usar esta funcion :

http://commons.apache.org/lang/api-release/org/apache/commons/lang/StringUtils.html#isEmpty(java.lang.String)

Cimballi

[cambito]

usuario.getNombre().equals(null) .... etc

[Cimballi]

Cambito,

No es posible de probar si una propriedad esta null de esta forma !
Si tu haces : usuario.getNombre().equals(null) y si nombre esta null, entonces va a ocurrir una NullPointerException...

Cimballi

[Isdiar]

Gracias a todos, tema cerrado.

Como mejor funciona, comprobado en mi aplicación: "".equals(usuario.getNombre());

Hay que ver lo que estoy aprendiendo con vosotros... jejeje Gracias, en serio, a todos los foreros!! Un abrazo!!