Почему мой while / switch выполняет один случай дважды?

Question

Почему мой while / switch выполняет один случай дважды?

1

По какой-то причине, если один из трех возможных сценариев моего переключателя stmt запущен, он выполняется дважды. Три возможности: Gone, Unlocked, Locked. Если "Заблокировано" запущено, он запускает как "Заблокированные", так и "Разблокированные" пути.

Мой код ниже, и поведение приведено в скриншоте. Я не понимаю, как коммутатор оценивает значение? (Т.е. это "содержит", а не "равно" проверку?)

            while (graveData.next()) {

                switch(graveData.getString("Status")) {
                case "Gone":
                    new FancyMessage("ID").color(GRAY)
                    .then("" + graveData.getInt("ID")).color(AQUA)
                    .then(" ").color(GRAY)
                    .then(graveData.getString("Status"))
                    .color(GREEN).then(", Owner ").color(GRAY)
                    .then(graveData.getString("PlayerName"))
                    .color(YELLOW).then(", Looter ").color(GRAY)
                    .then(graveData.getString("LooterName"))
                    .color(YELLOW).then(", ").color(GRAY)
                    .then("Loc").color(GRAY)
                    .tooltip(graveData.getString("Location"))
                    .send(commandSender);
                    break;
                case "Locked":
                    new FancyMessage("ID").color(GRAY)
                    .then("" + graveData.getInt("ID")).color(AQUA)
                    .then(" ").color(GRAY)
                    .then(graveData.getString("Status"))
                    .color(YELLOW).then(", Owner ").color(GRAY)
                    .then(graveData.getString("PlayerName"))
                    .color(YELLOW).then(", ").color(GRAY)
                    .then("Loc").color(GRAY)
                    .tooltip(graveData.getString("Location"))
                    .send(commandSender);
                case "Unlocked":
                    new FancyMessage("ID").color(GRAY)
                    .then("" + graveData.getInt("ID")).color(AQUA)
                    .then(" ").color(GRAY)
                    .then(graveData.getString("Status"))
                    .color(RED).then(", Owner ").color(GRAY)
                    .then(graveData.getString("PlayerName"))
                    .color(YELLOW).then(", ").color(GRAY)
                    .then("Loc").color(GRAY)
                    .tooltip(graveData.getString("Location"))
                    .send(commandSender);
                    break;
                }

runelynx 12 апр. 2015, в 15:45

Источник

2

Там нет break; для "Запертого" случая.
Luiggi Mendoza 12 апр. 2015, в 13:44
3

Не связано, но, пожалуйста, рассмотрите рефакторинг или три.
Dave Newton 12 апр. 2015, в 13:47
0

Каково ваше ожидаемое поведение?
Platinum Azure 12 апр. 2015, в 13:48
0

@DaveNewton - Что ты имеешь в виду?
runelynx 12 апр. 2015, в 13:53
1

@runelynx Я имею в виду, что у вас очень похожий код в трех местах, о нем очень трудно читать и думать, и его будет сложно поддерживать. Это также выглядит созревшим для перечисления, и / или отсутствуют классы или декораторы (если единственные различия заключаются в выводе, возможно, декоратор). Большие куски почти идентичного кода - это запах, указывающий на отсутствие метода (ов). Большие куски условностей - это запах, указывающий на отсутствующий класс (ы).
Dave Newton 12 апр. 2015, в 13:58
0

@DaveNewton Очень полезен для тех индикаторов "запаха", спасибо! Я думаю, что если бы вы увидели некоторые из моих других занятий, вы, вероятно, упали бы со стула ... и не очень хорошо;) Пора заняться некоторыми ... "рефакторингом", я думаю! Звучит не очень весело, но мой плагин minecraft начинает становиться очень большим, и, думаю, его будет сложнее поддерживать, если я не уберусь :(
runelynx 12 апр. 2015, в 14:01

Показать ещё 4 комментария

Теги:

java

3 ответа

1

После второго case ("locked") вы забыли переложить break; выражение. Он должен выглядеть так:

case "Locked":
  new FancyMessage("ID").color(GRAY)
  .then("" + graveData.getInt("ID")).color(AQUA)

        [...]   

  .send(commandSender);
  break; //important !

case "Unlocked":
  new FancyMessage("ID").color(GRAY)
        [...]

См. Также это для объяснения ошибки switch.

JonasCz 12 апр. 2015, в 12:34

1

Вы не сделали паузу после случая с Locked.. Так оно выполняет следующий случай

Из документов

Операторы break необходимы, потому что без них утверждения в блоках коммутатора проваливаются: все операторы после метки совпадения выполняются в последовательности, независимо от выражения последующих меток case

singhakash 12 апр. 2015, в 10:50

Ещё вопросы

Там нет break; для "Запертого" случая.
Не связано, но, пожалуйста, рассмотрите рефакторинг или три.
Каково ваше ожидаемое поведение?
@runelynx Я имею в виду, что у вас очень похожий код в трех местах, о нем очень трудно читать и думать, и его будет сложно поддерживать. Это также выглядит созревшим для перечисления, и / или отсутствуют классы или декораторы (если единственные различия заключаются в выводе, возможно, декоратор). Большие куски почти идентичного кода - это запах, указывающий на отсутствие метода (ов). Большие куски условностей - это запах, указывающий на отсутствующий класс (ы).
@DaveNewton Очень полезен для тех индикаторов "запаха", спасибо! Я думаю, что если бы вы увидели некоторые из моих других занятий, вы, вероятно, упали бы со стула ... и не очень хорошо;) Пора заняться некоторыми ... "рефакторингом", я думаю! Звучит не очень весело, но мой плагин minecraft начинает становиться очень большим, и, думаю, его будет сложнее поддерживать, если я не уберусь :(

candied_orange · Accepted Answer · 2015-04-12T12-03-00.000Z

тебе нужно

    break;
case "Unlocked":

чтобы остановить " провалиться ",

break - как каждый раздел должен заканчиваться, если вы не хотите автоматически запускать следующий. Это очень мощная и часто запутанная функция операторов switch. Один из них, который будет легче определить и понять в коде, если вы возьмете Dave Newtons за консультацию и рефакторинг. Рассмотреть возможность gone(), lock() и unlock() методы или даже классов, каждый с setStatus() метод. Удалите дублирование с крайним предрассудком.

Ааа, имеет смысл в рефакторинге - спасибо за заголовки !!