Java - Ayuda por favor!

Es que es increible, he estado una hora intentando que el siguiente código funcione pero no lo hace... De verdad debe ser un error muy estúpido, pero no me doy cuenta.-

http://puu.sh/lk1tY/57d8907b81.png

Los atributos pY y pX siguen siendo cero, el operador (a == "@") nunca es verdadero. ¿Por qué????

Cuando utilizas el operador == para comparar cadenas (String), estás comparando sus referencias (posiciones de memoria); y no los valores que contienen esas posiciones de memoria (que es lo que necesitas hacer en tu programa).

Para comparar los valores necesitas utilizar el método equals().

La condición que necesitas evaluar en la sentencia if, quedaría de esta manera:


También podría escribirse de esta manera:


Pero, en este caso, es preferible la primera forma, porque "@" es un literal y por lo tanto no existe la posibilidad de que tome el valor null durante la ejecución de tu programa. En cambio, en la segunda forma, si por algún motivo se llega a la situación en que a = null, tu programa ocasionaría una excepcion NullPointerException al hacer la comparación.

De cualquiera de las 2 formas puede ocasionar un nullpointer.
Debe de controlarlo al implementar el metodo equals.

Un saludo.

Tienes razón Xampy, lo mejor sería controlar esa excepción. En especial para luego localizar la parte del código donde la variable llega a hacer referencia a null.

No obstante, en este caso, la primera forma de uso del método equals() no ocasiona que se lance la excepción: