Скажите, где ошибка в коде Java?

Question

[Кирилл Романенко]

Вот мой кусочек кода. Вначале все правильно, программа вроде исправно запускается и выполняется, но тут вы можете увидеть новый кусок - я решил теперь еще и проверять, а вдруг игрок вообще не будет вводить свое имя? До этого программа просто шла без имени - с пустым пробелом вместо него..

Однако, программа просто игнорирует мою проверку и работает как и без нового "если-то"! Буд-то она не понимает, что я прошу проверить, нет ли на месте имени игрока пустого места. Может быть я забыл какие-то знаки? Или нужно писать не if (name=="") а что-то другое?

//Запрашиваем имя пользователя для дальнейшего использования в диалогах.
System.out.println("Привет. Как зовут тебя?");
String name=in.readLine();
//Проверяем, ввел ли пользователь свое имя?
if (name=="")
{
System.out.println("Не хочешь говорить? Подозрительно..");
System.out.print("Продавщица хмурится. Видимо, теперь она доверяет вам немного меньше.");
}
//если ввел, то далее:
else

System.out.println("Что хочешь купить, " + name + "?");
System.out.println("1) Сигареты;");
System.out.println("2) Хлеб;");
System.out.println("3) Молоко;");
System.out.println("(Укажи цифру)");
//Принимаем ответ игрока. Запоминаем как "y":
String sn=in.readLine();
int y=Integer.parseInt(sn);

Answer 1

[pi314]

//Проверяем, не ЗАБЫЛ ли пользователь ввести свое имя?
if (name == null || name.trim().isEmpty()) { ...

такая "хитрая" проверка нужна, т.к. readLine() вполне может вернуть null, и тогда мы упадем по NPE на name.trim()

В Джава сравнение строк делается так
if (name.equals("Вася"))
a name=="" сравнит объект name с другим, новым объектом типа String, со значением "" ... которые, разумеется, никогда не будут равны (вне зависимости от того, равны ли между собой ЗНАЧЕНИЯ строк), ибо это два разных объекта!

Comments

[Marat S]

наверное тогда проще
if ( !TextUtils.isEmpty( str ) ) ...

и на nullпроверит и на длину не нулевую.

[pi314]

Разумеется, проще... Но, взгляните на код. Мне кажется, что TextUtils тут еще немножко рановато ;)

[Владимир Смирнов]

А зачем проверка name != null?
Так не проще ли:
if (name == null || name.trim().isEmpty()) { ...
Если первое условие true, то сразу будет переход в блок, без проверки следующего условия. Т.е., если name == null, то name.trim().isEmpty() не будет вызван, и, соответственно, дополнительная проверка на null не нужна.

[Кирилл Романенко]

ээ... да, я не в курсе что это за утилс вообще еще 0_о

[Кирилл Романенко]

Ну, вообщем, спасибо за информацию. Пока я всю ночь просидел пытаясь понять какие скобки забыл поставить и как вывести часть кода в другой класс... Я пока использую if (name.equals("")), а там научусь чему-нить по-мощнее.

[pi314]

@bobzer Эта проверка нужна обязательно. Если name null, любая попытка выполнить name.xyz() приведет к NullPointerException. name, в свою очередь, может быть null, если пользователь в ответ на запрос введет Ctrl+D (=EndOfStream).

[Владимир Смирнов]

@pi314 Вы, видимо, не поняли мой комментарий. Зачем ДВЕ проверки на null в одном условии?

[pi314]

@bobzer А, ну это - да :) Я ж приводил общий пример для иллюстрации, так сказать, ибо автор вопроса, похоже, не совсем представляет себе разницу между объектами и значениями. Так, конечно, проще и правильнее. Именно так оно и реализовано в StringUtils. Только тогда пришлось бы еще объяснять про порядок выполнения операций и т.д., а это уже совсем другая история... короче, сейчас поправлю :)

[Lo-fi]

На будущее все же сравнение строк лучше делать вида "str".equals(var). Так не нужно проверять var на null и это для Java стандартный порядок.

Answer 2

[Marat S]

name == "" - Не правильно.
почитай доки.

вообще как то так.

if (!"".equals(name))
{
...
}

Comments

[Кирилл Романенко]

Прочитал много чего, уже исправил на if (name.equals("")). Все работает ))

Answer 3

[Максим Мосейчук]

Ко всему описанному выше добавьте фигурные скобки после else и в конце.

Comments

[Кирилл Романенко]

Можно, но там простая... ээ, фукнция, да? Так что я скобки убрал пока, чтоб искать ошибки проще было не копаясь в миллионах знаков.

Answer 4

[sgtraf]

habrahabr.ru/post/49582

В Java есть два способа сравнить объекты на равенство, == и метод equals.
== используется для примитивных типов. За его использование с объектами умные люди либо бьют по ушам либо всячески благодарят — для объектов == это исключительно сравнение по ссылке...