Помогите После Cooldown не выполняются события

Anion · 5 май 2018

Здравствуйте,я начал писать плагин на способности и столкнулся с такой проблемой.
Есть код (Да-да,он говно)

Код:

package zrt.main;

import java.util.HashMap;
import java.util.UUID;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.entity.Arrow;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.*********ventory.ItemStack;
import org.*********ventory.meta.ItemMeta;
import org.bukkit.util.Vector;

public class Handler
  implements Listener
{
  private HashMap<UUID, Long> cooldown = new HashMap();
  private int cooldowntime = 40;

  @EventHandler
  public void interact(PlayerInteractEvent e)
  {
   Action a = e.getAction();
   ItemStack item = e.getPlayer().getItemInHand();
   if ((a == Action.LEFT_CLICK_AIR) && (item.getType() != Material.FEATHER)) {
     }
   if ((a == Action.LEFT_CLICK_AIR) && (item.getType() == Material.FEATHER)) {
     return;
   }
   Player p = e.getPlayer();
   Location loc = p.getLocation();
   ItemStack itemi = p.getItemInHand();
   float yaw = loc.getYaw();
   float pitch = loc.getPitch();
   if ((item.getType() == Material.FEATHER) && (itemi.getItemMeta().getDisplayName().equals(ChatColor.WHITE + "Прыжок")))
   {
     if (this.cooldown.containsKey(p.getUniqueId()))
     {
       long secondsleft = ((Long)this.cooldown.get(p.getUniqueId())).longValue() / 1000L + this.cooldowntime - (System.currentTimeMillis()/1000);
       if (secondsleft > 1) {
         p.sendMessage(ChatColor.RED + "Дождитесь отката способности("+ secondsleft+")");
       }
       e.setCancelled(true);
     }
     else
     {
       p.setVelocity(p.getLocation().getDirection().multiply(10));
       this.cooldown.put(p.getUniqueId(), Long.valueOf(System.currentTimeMillis()));
       return;
     }
     e.setCancelled(true);
   } else {
         p.sendMessage(ChatColor.RED + "Дождитесь отката способности");
     }
   e.setCancelled(true);
   if ((item.getType() == Material.BOOK) && (itemi.getItemMeta().getDisplayName().equals("Книга заклинателя стрел"))) {
     p.launchProjectile(Arrow.class);
   }
  }
}

проблема в том,что при использование способности идет cooldown 40 секунд,НО послед cooldown'a вообще ничего не происходит,те даже не пишется "Дождитесь отката способности",прошу помощи.

MrMagaChannel · 5 май 2018

Anion сказал(а):
HashMap<UUID, Long> cooldown = new HashMap()

Измени на:

PHP:

Map<UUID, Long> cooldownMap = new HashMap<>();

У тебя кулдаун добавляется только когда его нет, то есть одноразовый кулдаун.
Когда время истекает, то проверка secondsleft > 1 не проходит и ничего не происходит.
Тебе нужно добавлять в мапу новое время, если оно уже истекло и выполнять действие.
Перепиши все, а то очень криво сделал.

Cool_boy · 5 май 2018

Если игрок есть в мапе, код никогда не дойдёт к моменту где ты даёшь способность. Лучше бы ты переделал логику этой штуковины, но кажется, можно исправить так:

PHP:

if (secondsleft > 0) {
        p.sendMessage(ChatColor.RED + "Дождитесь отката способности("+ secondsleft+")");
} else {
    //дать способность и записать новое время
}

Anion · 5 май 2018

MrMagaChannel сказал(а):
Тебе нужно добавлять в мапу новое время, если оно уже истекло и выполнять действие.

Тут можешь подробнее рассказать?

Anion · 5 май 2018

Cool_boy сказал(а):
Если игрок есть в мапе, код никогда не дойдёт к моменту где ты даёшь способность. Лучше бы ты переделал логику этой штуковины, но кажется, можно исправить так:

PHP:

if (secondsleft > 0) { p.sendMessage(ChatColor.RED + "Дождитесь отката способности("+ secondsleft+")"); } else { //дать способность и записать новое время }

А как новое время записывать?(Знаю,что я совсем глупый)

Cool_boy · 5 май 2018

Anion сказал(а):
А как новое время записывать?(Знаю,что я совсем глупый)

this.cooldown.put(p.getUniqueId(), System.currentTimeMillis());

Anion · 5 май 2018

Cool_boy сказал(а):
this.cooldown.put(p.getUniqueId(), System.currentTimeMillis());

Теперь не пашет вообще)

Cool_boy · 5 май 2018

Anion сказал(а):
Теперь не пашет вообще)

Код? Но все же советовал бы полностью переписать,

MrMagaChannel сказал(а):
а то очень криво сделал.

Anion · 5 май 2018

Cool_boy сказал(а):
Код? Но все же советовал бы полностью переписать

~~Сделаешь все за меня?~~
Поможешь?

MrMagaChannel · 5 май 2018

Anion сказал(а):
~~Сделаешь все за меня?~~
Поможешь?

Держи пример из моего старого плагина - https://github.com/mrmagachannel/Ad...c/me/mrmaga/art/managers/CooldownManager.java
Код тут конечно не очень, но для тебя сойдет.

MrMagaChannel · 5 май 2018

По хорошему около 6-и классов нужно для нормальной реализации Cooldown-а.

MrMagaChannel · 5 май 2018

Cool_boy сказал(а):
this.cooldown.put(p.getUniqueId(), System.currentTimeMillis());

тут надо cooldowntime прибавить к currentTimeMillis

Cool_boy · 5 май 2018

MrMagaChannel сказал(а):
тут надо cooldowntime прибавить к currentTimeMillis

У него в коде это раньше делается... Или я запутался. 😆

Anion сказал(а):
(Long)this.cooldown.get(p.getUniqueId())).longValue() / 1000L + this.cooldowntime

iD3LSY · 6 май 2018

Написал в блокноте небольшой пример, если еще надо, могу именно с ним помочь (вроде должно работать, не тестил).

PHP:

Map<UUID, Long> cooldownMap = Maps.newHashMap();
long cooldownInterval = 40; // секунды

public void useAbility(Player player) {
    UUID uuid = player.getUniqueId();

    if(cooldownMap.containsKey(uuid) && cooldownMap.get(uuid) >= System.currentTimeMillis()) {
        cooldownMap.put(uuid, System.currentTimeMillis() + (cooldownInterval * 1000));

        player.sendMessage("Крутая способность применена!");
    } else {
        player.sendMessage("Крутая способность на откате. Осталось " + ((cooldownMap.getOrDefault(uuid, 0) - System.currentTimeMillis()) / 1000) + " секунд.");
    }
}

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

(ну а если пойдешь на это, то лучше кинь свой взор на Guava Caches)

MrMagaChannel · 6 май 2018

iD3LSY сказал(а):
Минус подобных схем, это то, что все игроки всегда будут в мапе. Можно таймером каждые 5 минут аннулировать уже откатившиеся кулдауны, но это уже дело за тобой)

Что-то бред, зачем, это, в мапе хранятся уже находящиеся в памяти UUID и Long-и. Обертка Long весит около 32-40 байтов и на 100 000 игроков это будет ~ 4 мбайт. Как видишь смысла самому очищать мапу нет, это будет происходить только при перезагрузках.

iD3LSY · 6 май 2018

MrMagaChannel сказал(а):
Что-то бред, зачем, это, в мапе хранятся уже находящиеся в памяти UUID и Long-и. Обертка Long весит около 32-40 байтов и на 100 000 игроков это будет ~ 4 мбайт. Как видишь смысла самому очищать мапу нет, это будет происходить только при перезагрузках.

Я о таких мелочах не думаю)

Вряд-ли бы он этим занялся, как минимум, а как максимум, докинул бы еще нагрузки. В целом, ты прав, может 5мБ и не так важны (на самом деле, там намного меньше).

MrMagaChannel · 6 май 2018

iD3LSY сказал(а):
Я о таких мелочах не думаю)

Вряд-ли бы он этим занялся, как минимум, а как максимум, докинул бы еще нагрузки. В целом, ты прав, может 5мБ и не так важны (на самом деле, там намного меньше).

100 000 игроков использовавших кулдаун за 1 аптайм это сильное преувеличение, я взял для наглядности, на деле будет в десятки раз меньше и очистка мапы кулдаунов шедулярами это бесполезное дело.

Anion · 6 май 2018

iD3LSY сказал(а):
Написал в блокноте небольшой пример, если еще надо, могу именно с ним помочь (вроде должно работать, не тестил).

PHP:

Map<UUID, Long> cooldownMap = Maps.newHashMap(); long cooldownInterval = 40; // секунды public void useAbility(Player player) { UUID uuid = player.getUniqueId(); if(cooldownMap.containsKey(uuid) && cooldownMap.get(uuid) >= System.currentTimeMillis()) { cooldownMap.put(uuid, System.currentTimeMillis() + (cooldownInterval * 1000)); player.sendMessage("Крутая способность применена!"); } else { player.sendMessage("Крутая способность на откате. Осталось " + ((cooldownMap.getOrDefault(uuid, 0) - System.currentTimeMillis()) / 1000) + " секунд."); } }

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

(ну а если пойдешь на это, то лучше кинь свой взор на Guava Caches)

Извиняюсь за наглость,~~я совсем айт~~
я только вчера начал этим заниматься,можете вживить код в мой?
Сам попробовал разобраться-не получилось

MrMagaChannel · 6 май 2018

Anion сказал(а):
Извиняюсь за наглость,~~я совсем айт~~
я только вчера начал этим заниматься,можете вживить код в мой?
Сам попробовал разобраться-не получилось

Зачем ты начинаешь с bukkit, учи сначала java и не будешь такие глупые темы создавать.

Anion · 6 май 2018

MrMagaChannel сказал(а):
Зачем ты начинаешь с bukkit, учи сначала java и не будешь такие глупые темы создавать.

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

Помогите После Cooldown не выполняются события

Участник

Активный участник

Активный участник

Участник

Участник

Активный участник

Участник

Активный участник

Участник

Активный участник

Активный участник

Активный участник

Активный участник

Активный участник

Активный участник

Активный участник

Активный участник

Участник

Активный участник

Участник