Te recomiendo que pongas un system.out.println o algo pareceido para ver si entra en el if o no, en el get string para ver el valor y si realmente es 0 como pensamos, etc. para ir identificando el error. A veces nos obcecamos en creer que el error está en un sitio y puede estar en otro.
Por otro lado, he visto algún error de programación básico, así que te pongo aquí tu código al que le he quitado alguna cosa, incluyendo dos system.out.println de ejemplo:

-Lo primero, el return sobra, cuando una función void llega al final, se termina, el compilador ya lo sabe y poner un return no es necesario a no ser que quieras rompet la ejecución en otro sitio, cosa que no es este caso.
-Los else vacíos son innecesarios, si no vas a hacer nada, no lo pongas.
-No pongas tanto salto de línea, solo cuando quieras dividir dos bloques de código de algo grande. Lo único que consigues es que se vea peor.
-Si solo vas a usar una vez una variable, para eso no la hagas, consumes más recursos de los necesarios.
-Si un import no lo vas a usar, no lo comentes, elimínalo. No aporta nada.
-Te he puesto los { en una nueva línea, está bien de ambas maneras, como lo tenías o como lo puse yo, simplemente es por costumbre mía.