Здесь:
if (ctrlstr=="END")
вы сравниваете строки, используя ==
, который всегда проверяет идентичность ссылки. Вместо этого вы должны использовать equals
:
if (ctrlstr.equals("END"))
или, возможно, (если вы хотите получить false
результат, когда ctrlstr
равен нулю)
if ("END".equals(ctrlstr))
Несколько дополнительных заметок:
- Поймать
Exception
обычно плохая идея - вы должны поймать более конкретные исключения
- Поймать исключение и продолжить в любом случае после печати это обычно плохая идея
- Ваш код будет более читабельным, если вы сделаете соответствующий отступ
- Ваш код будет более читабельным, если вы всегда используете фигурные скобки для
if
операторов и т. Д.