Выполнение действия один раз в PlayerTickEvent

[xmafee]

Всем привет. Как выполнять действие один раз:

    @SubscribeEvent
    public void onPlayerTick(TickEvent.PlayerTickEvent event) {
        EntityPlayer player = event.player;
        EntityPlayerMP playerMP = (EntityPlayerMP) event.player;
        int ticktime = playerMP.getStatFile().readStat(StatList.PLAY_ONE_MINUTE);
        int minutes = ticktime / 20 / 60;

        if(event.phase == TickEvent.Phase.END) {
            if (minutes == 24) {
                //player.sendMessage(new TextComponentString("It's work!"));
            }
        }
    }

Я хочу получать время игрока, если время игрока == указанному времени - выполнять действия. Не могу понять, как сделать так, чтобы действие выполнилось один раз, тк если делать через player.existedTicks - это немного не то уже будет. Возможно есть другой метод выполнение?

 

[Deco20-19]

Ну, если принципиально время игрока использовать нет желания, то можно пойти строить костыли от

System.currentTimeMillis(); //Возвращает обычное юниксовское время, хотя кого я учу..

.
Сохранять его где-то.
Метод - нативный жвм-овский, поэтому дальнейшая реализация уже любая.

 

[tox1cozZ]

if (minutes == 24 && !player.getEntityData().getBoolean("processPlayingMinutes")) {
    //player.sendMessage(new TextComponentString("It's work!"));
    player.getEntityData().setBoolean("processPlayingMinutes", true);
}

 

[xmafee]

@tox1cozZ спасибо, это работает, но можешь пожалуйста обьяснить что это и как это работает?

 

[tox1cozZ]

У каждой сущности есть свои данные, которые сохраняются - NBTTagCompoud.
Мы проверяем, если в игроке не записан наш флаг - выполняем нужное действие и записываем, тем самым даём выполняться нужному коду лишь единожды.

 

[xmafee]

Понял, всё чётко и ясно, спасибо большое

 

[xmafee]

@tox1cozZ

Но всё же, я не могу понять одного, почему оно сохраняет значение для всех?

    int x = 0;
    @SubscribeEvent
    public void onBlockPlaced(BlockEvent.BreakEvent event) {
        EntityPlayer player = event.getPlayer();
        EntityPlayerMP playerMP = (EntityPlayerMP) event.getPlayer();
        player.getEntityData().setInteger("one", x++);

        player.sendMessage(new TextComponentString("count: " + player.getEntityData().getInteger("one")));
    }

Если один игрока сламал n количество блоков, то у другого игрока после ломания блока n+1

 

[Heathcliff]

Игроки у тебя разные, а Х один

 

[xmafee]

хм, скорее всего ты прав. что предлагаешь сделать?

 

[Heathcliff]

Создать локальную переменную, присвоить ей значение "one", и уже ее увеличивать и сохранять

 

[xmafee]

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

 

[Heathcliff]

Нет, ты вызываешь событие каждый новый раз, создавая новую переменную, которой присваиваешь значение из игрока

 

[xmafee]

        int x;
        x = player.getEntityData().getInteger("one");
        player.getEntityData().setInteger("one", x++);

не хочет работать

 

[Heathcliff]

int x = player.getEntityData().getInteger("one");
x++;       
player.getEntityData().setInteger("one", x);

 

[tox1cozZ]

Постфиксный инкремент присваивает значение после записи в нбт игрока.
Либо делай его на предыдущей строке отдельно, либо замени на префиксную форму: ++x

 

[tox1cozZ]

И да, для удобного менеджмента кастомными данными сущностей есть капабилити. Гугл в помощь.

 

[xmafee]

Спасибо за помощь, работает. Понял ошибку.