Как мне сбросить мои значения в операторе switch? - PullRequest
Новая
       17

Как мне сбросить мои значения в операторе switch?

0 голосов
/ 06 мая 2019

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

1) Выберите реализацию отслеживания блоков по мере их создания или перехода непосредственно к решению. 2) Цикл while контролирует следующие три шага. 3) Выберите коробку 4) Определите, можно ли поставить эту клетку
5) Поставить коробку
Я использовал 6 логических переменных, каждая из которых представляет собой поле. Оператор switch вызвал один из 6 методов в зависимости от случайного числа, заданного для ящика. Эти шесть методов проанализировали текущий стек ящиков и определили, какую из 13 комбинаций ящиков нужно вызвать. 

package com.chinus;

i
    private static void trace() {
        int boxesUsedSoFar = 0;
        boolean gameIsDone = false;
        int usedOnce;

        while (gameIsDone != true) {
            int number = (int) ((Math.random() * 6) + 1);
            boxesUsedSoFar++;

            System.out.println("Current Box Number : " + number);
            switch (number) {
                case 1:
                    if (number == 1) {
                        box1 = true;

                    }
                    if (box1 == true && box2 == true && box3 == true) {
                        combo6();
                    } else if (box1 == true && box2 == true) {
                        combo4();
                    } else if (box1 == true && box3 == true) {
                        combo7();
                    } else if (box1 == true) {
                        combo1();
                    }
                    else{
                        restart();
                    }
                    box1=false;
                    break;

                case 2:
                    if (number == 2) {
                        box2 = true;


                    }

                    if (box1 == true && box2 == true && box3 == true) {
                        combo6();
                    } else if (box1 == true && box2 == true) {
                        combo4();
                    } else if (box2 == true && box3 == true) {
                        combo5();
                    } else if (box2 == true) {
                        combo2();
                    }
                    else{
                        restart();
                    }
                    box2=false;

                    break;
                case 3:
                    if (number == 3) {
                        box3 = true;
                        int count3 = 0;
                    }
                    boolean box3used =false;


                    if (box1 == true && box2 == true && box3 == true) {
                        combo6();
                        box3used =true;
                    } else if (box1 == true && box3 == true) {
                        combo7();
                        box3used =true;
                    } else if (box2 == true && box3 == true) {
                        combo5();
                        box3used =true;
                    }else if (box3 == true) {
                        combo3();
                        box3used = true;
                    }
                    else {
                        restart();
                    }

                    break;
                case 4:
                    if (number == 4) {
                        box4 = true;
                    }
                    if (box5 == true && box4 == true && box3 == true && box2 == true && box1 == true) {
                        combo12();
                    } else if (box4 == true && box2 == true & box3 == true && box1 == true) {
                        combo10();
                    } else if (box4 == true && box2 == true & box1 == true) {
                        combo8();
                    } else {
                        restart();
                    }

                    break;

                case 5:
                    if (number == 5) {
                        box5 = true;
                    }
                    if (box5 == true && box4 == true && box3 == true && box2 == true && box1 == true) {
                        combo12();
                    } else if (box5 == true && box2 == true & box3 == true && box1 == true) {
                        combo11();
                    } else if (box5 == true && box2 == true & box3 == true) {
                        combo9();
                    } else {
                        restart();
                    }
                    break;

                case 6:
                    if (number == 6) {
                        box6 = true;
                    }
                    if (box4 == true && box5 == true) {
                        combo13();
                        gameIsDone = true;
                    } else {
                        restart();
                    }
                    break;

                default:

            }
            System.out.println("Boxes used so far : " + boxesUsedSoFar);
        }
    }


    private static void restart() {
        box1 = false;
        box2 = false;
        box3 = false;
        box4 = false;
        box5 = false;
        box6 = false;
    }
}

Actual:
Current Box Number : 2
       ___   ___
      |   | |   |
      | 2 | | 3 |
      |___| |___|
Boxes used so far : 49820
Current Box Number : 2
       ___   ___
      |   | |   |
      | 2 | | 3 |
      |___| |___|



Expect:Current Box Number : 2
       ___   ___
      |   | |   |
      | 2 | | 3 |
      |___| |___|
Boxes used so far : 49820
Current Box Number : 2
       ___  
      |   | 
      | 2 | 
      |___|

1 Ответ

0 голосов
/ 06 мая 2019

Вы в основном там.

Я изменил несколько вещей:

  • Вам не нужно использовать выражение if, которое является избыточным с вашимсменить заявление.

Когда вы говорите 

switch(number)
case 1:

Технически вы спрашиваете 

if(number == 1)

Поэтому нам не нужно проверять число внутри оператора switch.

  • Упрощенная проверка логического значения в операторе 'if'

При проверке логической переменной в операторе if вам не нужно произносить 

if(boolean == true)

Вы можете просто сказать 

if(boolean)

По умолчанию это проверяет, является ли значение истинным.

Что касается логики в вашем коде, я имею толькопоказаны детали, имеющие отношение к вашему вопросу.Вы хотите проверить состояние текущего поля в начале любого оператора.В «case: 1» первым делом я проверил, действительно ли box1 истинно.Если это так, я вызвал метод reset и выгнал нас из оператора switch.

Если box1 имеет значение false (что означает, что мы не использовали его ранее), тогда мы можем фактически запустить нормальную логику, а затем установить для box1 значение true.Я прокомментировал, куда пойдет некоторая ваша логика, поскольку я не совсем уверен, что вы пытаетесь с этим делать.

        int boxesUsedSoFar = 0;
        boolean gameIsDone = false;

        while (gameIsDone != true) {
            int number = (int) ((Math.random() * 6) + 1);
            boxesUsedSoFar++;

            System.out.println("Current Box Number : " + number);
            switch (number) {
                case 1:
                    if (box1) {
                        //box1 is false by default, if it was true here, it means we have a duplicate
                        restart();
                        break;
                    }
                    /*

                    Your code to call a combo method goes here, repeat for each case.

                    */
                    box1 = true;
                    break;
                case 2:
                    if (box2) {
                        restart();
                        break;
                    }
                    box2 = true;
                    break;
                case 3:
                    if (box3) {
                        restart();
                        break;
                    }
                    box3 = true;
                    break;
                case 4:
                    if (box4) {
                        restart();
                        break;
                    }
                    box4 = true;
                    break;
                case 5:
                    if (box5) {
                        restart();
                        break;
                    }
                    box5 = true;
                    break;
                default:
                    if (box6) {
                        restart();
                        break;
                    }
                    box6 = true;
                    break;

            }
            System.out.println("Boxes used so far : " + boxesUsedSoFar);
        }
    }

Если что-то здесь неясно, пожалуйста, дайте мне знать.

...