41 уровень

07.08.2019
552просмотров
13комментариев

зачем

Вопрос по задаче

Решен

зачем делать проверку пере сообщением, что игрок победил?

if(!OnlineGame.isWinnerFound){
   System.out.println(getName() + ":победитель!");
}
OnlineGame.isWinnerFound = true;

если и так работает нормально

System.out.println(getName() + ":победитель!");
OnlineGame.isWinnerFound = true;

Комментарии (13)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

IceBerg

Уровень 41

Master

7 августа 2019, 19:18

полезный

Чтобы победитель был один, а не сколько успеют.

Haumi Java Developer

7 августа 2019, 19:31

Если не трудно, то приведите пример, когда могут несколько успеть. Если я ставлю двоим одинаковое значение rating, они оба становятся победителями, что с проверкой, что без нее.

IceBerg

Уровень 41

Master

7 августа 2019, 19:37

полезный

Понятия не имею какая у вас реализация, но изменение флага в true так поздно уже настораживает. У меня при входных данных

protected Gamer gamer1 = new Gamer("Ivanov", 100);
protected Gamer gamer2 = new Gamer("Petrov", 99);
protected Gamer gamer3 = new Gamer("Sidorov", 99);

Вывод

Sidorov:Начало игры
Petrov:Начало игры
Ivanov:Начало игры
Ivanov:Сбор ресурсов
Petrov:Сбор ресурсов
Sidorov:Сбор ресурсов
Sidorov:Рост экономики
Petrov:Рост экономики
Ivanov:Рост экономики
Sidorov:Убийство врагов
Petrov:Убийство врагов
Ivanov:Убийство врагов
Ivanov:проиграл
Petrov:проиграл
Sidorov:победитель!

Или

protected Gamer gamer1 = new Gamer("Ivanov", 100);
protected Gamer gamer2 = new Gamer("Petrov", 100);
protected Gamer gamer3 = new Gamer("Sidorov", 100);

Вывод

Ivanov:Начало игры
Petrov:Начало игры
Sidorov:Начало игры
Ivanov:Сбор ресурсов
Sidorov:Сбор ресурсов
Petrov:Сбор ресурсов
Ivanov:Рост экономики
Sidorov:Рост экономики
Petrov:Рост экономики
Ivanov:Убийство врагов
Petrov:Убийство врагов
Sidorov:Убийство врагов
Sidorov:проиграл
Petrov:проиграл
Ivanov:победитель!

Haumi Java Developer

7 августа 2019, 19:39

@Override
        public void run() {
            try {
                //Add your code here - добавь код тут
                for (String text : OnlineGame.steps){
                    System.out.println(getName() + ":" + text);
                    Thread.sleep(1000 / rating);
                }
                if(!OnlineGame.isWinnerFound){
                    System.out.println(getName() + ":победитель!");
                }
                OnlineGame.isWinnerFound = true;
            } catch (InterruptedException e) {
                System.out.println(getName() + ":проиграл");
                }
            }

Haumi Java Developer

7 августа 2019, 19:40

@Override
        public void run() {
            try {
                //Add your code here - добавь код тут
                for (String text : OnlineGame.steps){
                    System.out.println(getName() + ":" + text);
                    Thread.sleep(1000 / rating);
                }
                System.out.println(getName() + ":победитель!");
                OnlineGame.isWinnerFound = true;
            } catch (InterruptedException e) {
                System.out.println(getName() + ":проиграл");
                }
            }

Haumi Java Developer

7 августа 2019, 19:42

и так и так выводит нормально. Но валидатор принял только 1 вариант. Ну и не работает если ввести 2 одинаковых rating, для разных потоков... иначе вывод будет таким =) Ivanov:победитель! Sidorov:победитель! Petrov:проиграл

IceBerg

Уровень 41

Master

7 августа 2019, 19:45

полезный

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

Haumi Java Developer

7 августа 2019, 19:49

Если я меняю строчки местами

OnlineGame.isWinnerFound = true;
if(!OnlineGame.isWinnerFound){
                    System.out.println(getName() + ":победитель!");
                }

то вообще сообщение о победе не выходит и получается Ivanov:Рост экономики Ivanov:проиграл Petrov:проиграл

IceBerg

Уровень 41

Master

7 августа 2019, 19:52

полезный

Ну это очевидный факт, ты выкинул ключ, до того как открыл дверь)) Вот скопируй

if (!OnlineGame.isWinnerFound) {
                OnlineGame.isWinnerFound = true;
                System.out.println(getName() + ":победитель!");
            } else {
                System.out.println(getName() + ":проиграл");
            }

Haumi Java Developer

7 августа 2019, 20:08

@Override
        public void run() {
            try {
                //Add your code here - добавь код тут
                for (String text : OnlineGame.steps){
                    System.out.println(getName() + ":" + text);
                    Thread.sleep(1000 / rating);
                }
                if(!OnlineGame.isWinnerFound){
                    OnlineGame.isWinnerFound = true;
                    System.out.println(getName() + ":победитель!");
                }else {
                    System.out.println(getName() + ":проиграл");
                }
            } catch (InterruptedException e) {
               System.out.println(getName() + ":проиграл");
                }
            }

даже при таком решении, если ввожу

protected Gamer gamer1 = new Gamer("Ivanov", 9);
protected Gamer gamer2 = new Gamer("Petrov", 9);
protected Gamer gamer3 = new Gamer("Sidorov", 9);

выводит

Petrov:победитель!
Sidorov:победитель!
Ivanov:победитель!

Haumi Java Developer

7 августа 2019, 20:13

хотя если тыкать на выполнение программы, каждый раз вывод разный... №1

Ivanov:победитель!
Sidorov:победитель!
Petrov:проиграл

№2

Sidorov:проиграл
Ivanov:победитель!
Petrov:проиграл

№3

Ivanov:проиграл
Sidorov:победитель!
Petrov:проиграл

№4

Ivanov:проиграл
Sidorov:победитель!
Petrov:проиграл

№5

Ivanov:победитель!
Sidorov:победитель!
Petrov:проиграл

№6

Ivanov:проиграл
Sidorov:победитель!
Petrov:проиграл

IceBerg

Уровень 41

Master

7 августа 2019, 20:17

полезный

Вот мой вариант, кучу раз запускал - 1 победитель

public void run() {
            try {
                for (int i = 0; i < OnlineGame.steps.size(); i++) {
                    Thread.sleep(1000 / rating);
                    System.out.println(getName() + ":" + OnlineGame.steps.get(i));
                }
            } catch (InterruptedException e) {
                System.out.println(getName() + ":проиграл");
                return;
            }

            if (!OnlineGame.isWinnerFound) {
                OnlineGame.isWinnerFound = true;
                System.out.println(getName() + ":победитель!");
            } else {
                System.out.println(getName() + ":проиграл");
            }

        }

Haumi Java Developer

7 августа 2019, 20:23

Спасибо за потраченное время. я уже разобрался в чем проблема была. У меня не там sleep стоял.

for (String text : OnlineGame.steps){
                    System.out.println(getName() + ":" + text);
                    Thread.sleep(1000 / rating);
                }

а должно быть

for (String text : OnlineGame.steps){
                    Thread.sleep(1000 / rating);
                    System.out.println(getName() + ":" + text);
                }