Почему нить связи не работает в Java? - PullRequest
0 голосов
/ 06 марта 2019

Я работаю над небольшой игрой на основе Java, я пытаюсь запустить ракету игроком-1, пока игрок-1 не пропустит цель, как только Игрок-1 не достиг цели, тогда огненная крачка перейдет к Игроку-2, Игроку-2 будет стрелять до тех пор, пока не пропустит свою цель, наоборот.

Здесь Player-1 и Player-2 оба являются заданием Java Runnable, пожалуйста, найдите ниже код:

public class GameRender implements IGame {
    private Game game;
    private Player player1, player2;
    private Lock lock = new ReentrantLock();
    Condition notPlayer1TernCondition = lock.newCondition();
    Condition notPlayer2TernCondition = lock.newCondition();

    public GameRender(Game game, Player player1, Player player2) {
        this.game = game;
        this.player1 = player1;
        this.player2 = player2;
    }

    @Override
    public void create() {

    }

    @Override
    public void render() {
        //ExecutorService executorService = Executors.newFixedThreadPool(2);
        Player1Task plater1 = new Player1Task(player1.getTargetLocations(), true);
        Player2Task plater2 = new Player2Task(player2.getTargetLocations(), false);

        Thread t1 = new Thread(plater1);
        Thread t2 = new Thread(plater2);

        t1.start();
        t2.start();
    }

    @Override
    public void over() {

    }


    class Player1Task implements Runnable {
        private List<TargetLocation> playerOnesTargetLocationList;
        private boolean isHitTarget = false;
        private int fireCount = 0;
        private boolean yourTern;

        Player1Task(List<TargetLocation> playerOnesTargetLocationList, boolean yourTern) {
            this.playerOnesTargetLocationList = playerOnesTargetLocationList;
            this.yourTern = yourTern;
        }

        @Override
        public void run() {
            try {
                lock.lock();
                fire();
            } catch (InterruptedException e) {
                e.printStackTrace();
            } finally {
                lock.unlock();
            }
        }

        private void fire() throws InterruptedException {
            while (fireCount != playerOnesTargetLocationList.size()) {
                if (!yourTern) {
                    notPlayer1TernCondition.await();
                }
                TargetLocation location = playerOnesTargetLocationList.get(fireCount);
                if (player2.getOwnField().hasShip(location.getxPos(), location.getyPos())) {

                } else {
                    System.out.println("Player-1 else");
                    yourTern = false;
                    notPlayer2TernCondition.signalAll();
                }
                fireCount++;
            }

        }

    }

    class Player2Task implements Runnable {
        private List<TargetLocation> playerTwosTargetLocationList;
        private boolean isHitTarget = false;
        private int fireCount = 0;
        private boolean yourTern;

        Player2Task(List<TargetLocation> playerTwosTargetLocationList, boolean youTern) {
            this.playerTwosTargetLocationList = playerTwosTargetLocationList;
            this.yourTern = youTern;
        }

        @Override
        public void run() {
            lock.lock();
            try {
                fire();
            } catch (InterruptedException e) {
                e.printStackTrace();
            } finally {
                lock.unlock();
            }
        }

        private void fire() throws InterruptedException {

            while (fireCount != playerTwosTargetLocationList.size()) {
                if (!yourTern) {
                    notPlayer2TernCondition.await();
                }

                TargetLocation location = playerTwosTargetLocationList.get(fireCount);
                if (player1.getOwnField().hasShip(location.getxPos(), location.getyPos())) {

                } else {
                    System.out.println("p2 else");
                    yourTern = false;
                    notPlayer1TernCondition.signalAll();
                }
                fireCount++;
            }

        }
    }
}

вышекод не работает, как ожидалось, Первый раз Player-1 выполняется и после того, как этот код застрял.

Любое объяснение весьма приветствуется.

1 Ответ

1 голос
/ 06 марта 2019

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

Пояснение: Предположим, изначально Player1 имеет блокировку и срабатывает, так как yourTern игрока1 имеет значение true. Предположим, что пропускает , тогда вы дадите сигнал Player2 (, который еще не ожидает ) и установите yourTern в false и снимите блокировку (вызывая метод notPlayer1TernCondition.await() в следующей итерации).
Игрок2 получит эту блокировку и вызовет notPlayer2TernCondition.await(), поскольку yourTern имеет значение false.
Теперь оба потока будут переходить в состояние ожидания навсегда.

...