Блог

При попытке написать следующий код выдает 3 ошибки. Синтаксическая ошибка в токене ‘else’, вставьте ‘выражение assignmentoperator’ для завершения присваивания и не удается преобразовать из строки в логическое значение. Это для поля со списком, и когда я выбираю из поля со списком, я хочу удалить другие сайты из списка. ‘site’ — это переменная в main и c.site — это параметр в другом агенте. Может кто-нибудь объяснить, что делать.

 if ( site ) {
    if ( c.site.equals( "x" ) ) {
        cavernIterator.remove();
        continue;
    }
}
else {
    if ( c.site.equals( "y" ) ) {
    cavernIterator.remove();
    continue;
    }
}
else {
    if ( c.site.equals( "z" ) ) {
        cavernIterator.remove();
        continue;
        }
    }
else {
    ( c.site.equals( "a" ) ) {
    cavernIterator.remove();
    continue;
    }
}
  

У вас не может быть нескольких else блоков для одного и того же if . Каждый else блок должен быть присоединен к своему if :

 if(a) {
    ...
} else {
    if (b) {
        ...
    }
    else {
        if (c) {
            ...
        }
        else {
            ...
        }
    }
}
  

Или, сокращенно для всего этого, вы можете иметь блок с одним оператором внутри else , чтобы он выглядел так:

 if (a) {
    ...
}
else if (b) {
    ...
}
else if (c) {
    ...
}
else {
    ...
}
  

Этот второй вариант почти такой же, как и первый, поскольку он использует преимущество отсутствия необходимости заключать блок оператора в фигурные скобки, если ваш блок оператора содержит только один оператор, например:

 if (myCondition)
    System.out.println("My condition passed");
else
    System.out.println("My condition did not pass");
  

Остальное, если должно быть написано таким образом :

 if (...) {
    // Code here
} else if (...) {
    // Code here
} else {
    // Code here
}
  

Также ваши случаи кажутся похожими

 String[] sites = {"x","y","z","a"};
boolean contains = Arrays.stream(sites).anyMatch(c.site::equals);
if (contains) {
    cavernIterator.remove();
    continue;
}
  

РЕДАКТИРОВАТЬ: Учитывая, что сайт является строкой

 String site = "x"; // Default value provided
String[] sites = {"x","y","z","a"};
boolean condition = site.equals(c.site) amp;amp; Arrays.stream(sites).anyMatch(c.site::equals);
if (condition) {
    cavernIterator.remove();
    continue;
}
  

     if ( site ) {
        if ( c.site.equals( "x" ) ) {
        }
   }
  

равно

     if(site amp;amp; c.site.equals("x")){
    }
  

И ваше последнее еще перепутано.

Вероятно, посмотрите некоторые руководства по Java, потому что вы пишете наименее эффективный способ. Короче говоря, это ваш код:

 if (site amp;amp; (c.site.equals("x") || c.site.equals("y") || c.site.equals("z") || c.site.equals("a"))) {
    cavernIterator.remove();
}