Member 13949263 Ответов: 2

Неправильное сообщение java при добавлении нового объекта

Код работает хорошо, но проблема та же, даже если вид птицы уже есть в списке, я получаю сообщение о том, что в список добавлена новая птица. Когда я меняю (найденный) на (! Вместо этого я получаю второе сообщение о том, что птица уже есть в списке, хотя ее и нет в списке.

Что я уже пробовал:

общественного недействительными addBird(стринг-арт, поле ){
boolean found = false;
for (int i = 0; i <birdlList.size(); i++){

если(birdList.содержит(ст.))
нашли = истина;
перерыв;
}

если (!найдено) {
System.out.println("птица уже есть в списке");

} еще {
birdlList.добавить(новый птица(искусства, расположение));
System.out.println("в список добавлена новая птица");
}
}

2 Ответов

Рейтинг:
0

Bryian Tan

Глядя на код, я думаю, что проблема заключается в этом блоке. Если вы правильно отформатируете код, то увидите, что цикл запускается только один раз и сразу же завершается

for (int i = 0; i < birdlList.size(); i++) {

  if (birdList.contains(art))
   found = true;
  break;
 }


Я думаю, что это то, что вы хотите в коде. если условие истинно, установите found = true и затем выйдите из цикла.
for (int i = 0; i <birdlList.size(); i++)
{
    if(birdList.contains(art)) {
        found = true;
        break;
    }
}

Рейтинг:
0

Patrice T

1) Java и JavaScript-это 2 разных зверя, вам нужно сделать свой ум и выбрать 1 из них.
2) Ваш код странный:

for (int i = 0; i <birdlList.size(); i++){
    if(birdList.contains(art))
        found = true;
    break;
}

вы делаете цикл, который будет выполняться только один раз, и вы не используете счетчик? вы можете упростить до:
for (int i = 0; i <birdlList.size(); i++){
    if(birdList.contains(art))
        found = true;
    break;
}

3)
Цитата:
Код работает хорошо но проблема та же самая,

Перевод: код не работает.
Используйте отладчик, чтобы наблюдать за выполнением кода и находить, где он идет не так.
Можно заподозрить birdList.contains(art) ошибаться.

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш cpde, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.
Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]
http://docs.oracle.com/javase/7/docs/technotes/tools/windows/jdb.html[^]
https://www.jetbrains.com/idea/help/debugging-your-first-java-application.html[^]
Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.