¿Por qué no funciona mi comparación de cadenas?

Question

Ok, esto es estúpido, pero ¿qué sucede?

Tengo una variable String en un servlet, que toma el valor de un parámetro y en función de ese valor realizo una prueba para hacer algo, pero el if no funciona. ¿Cuál es el problema?

String action = request.getParameter("action"); 
    System.out.println("Action: " + action); 
// I put 2 ifs to be sure, but not even one is working 
    if(action.equals("something")) 
      { 
       System.out.println("hey");    
      } 
    if(action.trim() == "something") 
      { 
       System.out.println("hey"); 
      } 

En la consola, el System.out.println me demuestra que el valor de la acción es "algo"

Action: something 

Answer 1

Su segunda comparación es incorrecta. También debe utilizar en lugar de equals==, así:

if (action.trim().equals("something")) 

El operador compara == referencias de objetos (cuerdas) y en circunstancias normales iguales cadenas no tienen automáticamente la misma referencia, es decir, son diferentes objetos. (A menos que ambos sean internalized, pero normalmente no debería considerarlo)

Aparte de eso, su ejemplo funciona bien y la primera comparación es válida. Intenta arreglar la segunda comparación. Si funciona, encontraste tu problema. Si no, intente usar un depurador y vuelva a verificar todo.

PS: Al comparar las cadenas literales con objetos de cadena dinámicos, es una buena práctica para llamar al método equals en la cadena literal:

"something".equals(action) 

esa manera se puede evitar NullPointerExceptions cuando el objeto de cadena es nula.

Answer 2

El método equals compara las cadenas para la identidad del objeto y no compara el contenido. Para comparar el contenido de dos cadenas, use el método compareTo.

Answer 3

Solo una conjetura descabellada: podría ser que uno de estos "algo" contiene, por ejemplo, un carácter cirílico que se ve idéntico a su homólogo latino. En este caso, podría ser la "o".

Answer 4

Es poco probable que su segunda condición sea verdadera: está probando si el objeto de cadena creado al recortar action es el mismo objeto que la cadena literal "something". Esto solo será cierto si action se establece en ese mismo valor literal en otro lugar. Use "something".equals(action.trim()) en su lugar.

Su primera condición sería verdadera, los caracteres en la cadena action son los caracteres "something". Si no es verdad, entonces no es así. Afírmalo en una prueba, conéctalo, imprímelo o míralo en un depurador.

Si imprime una cadena para la depuración, use algo como System.out.println ("String = >" + string + "<"); para que sea obvio si hay espacios al final.

Answer 5

La comparación de cadenas en Java no puede hacerse por ==.

usted tiene que utilizar String.equals() o String.compareTo() -

Por cierto, String.compareTo() vuelve 0 mientras String.equals() vuelve true cuando dos cadenas son iguales.

Ver: http://java.sun.com/j2se/1.5.0/docs/api/java/lang/String.html#equals(java.lang.Object)

Answer 6

Mi conjetura es que tienes detrás o espacios que no muestran en el println líder.

Combina esto con el hecho de que estás usando action.trim() == 'something' significa que esta prueba tampoco funcionará.

Cambie esto a .equals("something") según lo sugerido por otros y podría funcionar.

Answer 7

Puede proteger mejor contra valores nulos cambiando las cláusulas if para que la cadena sea literal primero.

Pero ya que también parece querer proteger contra los espacios en blanco en el valor del parámetro, también se puede hacer un ajuste seguro nula del valor del parámetro utilizando StringUtils.trimToEmpty de Apache Commons Lang:

String action = StringUtils.trimToEmpty(request.getParameter("action")); 

System.out.println("Action: " + action); 

if("something".equals(action)) { 
    System.out.println("hey");    
} 

Answer 8

intente esto:

String action = request.getParameter("action"); 
System.out.println("Action: " + action); 

if(action.trim().equals("something")) 
{ 
    System.out.println("hey");    
}