Должен проверить метод возвращаемое значение

Question

Должен проверить метод возвращаемое значение

1

Почему в этом случае я получаю сообщение "Должен проверить возвращаемое значение метода"?

if(!qty.contains("@"))
      mgm.addOrder(date, Integer.parseInt(qty));
else
{
      qty.replace("@", "");
      mgm.addOrder(date, Integer.parseInt(qty));
      mgm.startNew();
}

Я получаю это сообщение: qty.replace("@", "");

Оператор if проверял, есть ли @, что еще мне нужно, чтобы сделать его идеальным?

ArthurV 27 март 2015, в 23:07

Источник

0

qty.replace("@", "") самом деле не меняет qty . qty = qty.replace("@", ""); если вы ничего не делаете с результатом, а не, например, qty = qty.replace("@", ""); ,
Andy Turner 27 март 2015, в 21:23
0

@AndyTurner опубликовать ответ с этим объяснением и тем, как должен выглядеть код.
Luiggi Mendoza 27 март 2015, в 21:24

Теги:

java

2 ответа

4

qty.replace("@", "") фактически не изменяет qty. Это не-op, если вы ничего не делаете с результатом.

Вы должны сделать что-то вроде:

qty = qty.replace("@", "");

Если вы этого не сделаете, следующее выражение будет NumberFormatException, потому что qty все еще имеет знак @:

Integer.parseInt(qty)

Andy Turner 27 март 2015, в 18:44

0

Большое спасибо вам обоим!
ArthurV 27 март 2015, в 21:29

Ещё вопросы

qty.replace("@", "") самом деле не меняет qty . qty = qty.replace("@", ""); если вы ничего не делаете с результатом, а не, например, qty = qty.replace("@", ""); ,
@AndyTurner опубликовать ответ с этим объяснением и тем, как должен выглядеть код.

MadProgrammer · Accepted Answer · 2015-03-27T20-01-00.000Z

String#replace возвращает новую String представляющую результаты операции replace. Вы должны вернуть результаты в String, иначе они будут потеряны

qty = qty.replace("@", "");