Spigot Abfrage funktioniert nur teilsweise.

[Shoqrii]

Hallo,

Die Arraylist Abfrage funktioniert grundlegend, und der Cooldown ist gegeben. Halte ich aber Rechtsklick gedrückt funktioniert es nach ca. 3 Schüssen nicht mehr. Warum?

public class Pistol_USP implements Listener {
    
    private ArrayList<Player> cd = new ArrayList<>();
    
    @EventHandler
    public void onInteract(PlayerInteractEvent e) {
        Player p = e.getPlayer();
        
        if(e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK) {
            if(p.getItemInHand().getType() == Material.WOOD_HOE) {
                if(!cd.contains(p)) {
                Gun(p);
                
                } else {
                    Bukkit.getScheduler().scheduleSyncDelayedTask(Main.getPlugin(), new Runnable() {
                        
                        @Override
                        public void run() {
                            
                            cd.remove(p);
                            
                        }
                    }, 18*1);
                }
            }
        }
        
    }
    
    private void Gun(Player p) {
        Arrow ar = p.launchProjectile(Arrow.class);
        
        if(!cd.contains(p)) {
        ar.setShooter(p);
        ar.setVelocity(p.getLocation().getDirection().multiply(2));
        cd.add(p);
        
        }
    }
    
}

 

[ES-Henne]

1. Klare Variablennamen
2. Methodennamen im lowerCamelCase
3. Klare Informationen
- Was willst du machen
- Was funktioniert
- Was funktioniert nicht
4. Ordentliche Fehlerbeschreibung
- "Geht nicht" ist keine Fehlerbeschreibung

So nebenbei, wenn ich mich richtig erinnere, dann wird das Event nur beim klicken, oder beim wechsel zu einem anderem Block getriggert. Bedeutet, dass der Code nur einmal ausgeführt wird. Keine Ahnung was du meinst mit ca. drei mal soll es funktionieren.

Du startest den Cooldown erst, wenn der Spieler die Aktion ein zweites mal ausführen möchte. Bedeutet, selbst wenn er zehn Sekunden wartet, dann darf er die Aktion nicht ausführen.

 

[BlackHole]

Dein Code ist anfällig für NPE (NullPointerException), weil Player.getItemInHand() null zurück liefern kann.

 

[Shoqrii]

Dein Code ist anfällig für NPE (NullPointerException), weil Player.getItemInHand() null zurück liefern kann.

Was kann ich dagegen tun?

 

[Shoqrii]

1. Klare Variablennamen
2. Methodennamen im lowerCamelCase
3. Klare Informationen
- Was willst du machen
- Was funktioniert
- Was funktioniert nicht
4. Ordentliche Fehlerbeschreibung
- "Geht nicht" ist keine Fehlerbeschreibung

So nebenbei, wenn ich mich richtig erinnere, dann wird das Event nur beim klicken, oder beim wechsel zu einem anderem Block getriggert. Bedeutet, dass der Code nur einmal ausgeführt wird. Keine Ahnung was du meinst mit ca. drei mal soll es funktionieren.

Du startest den Cooldown erst, wenn der Spieler die Aktion ein zweites mal ausführen möchte. Bedeutet, selbst wenn er zehn Sekunden wartet, dann darf er die Aktion nicht ausführen.

Grundlegend funktioniert es ja. Nur manchmal ignoriert er den Cooldown.

 

[BlackHole]

Was kann ich dagegen tun?

Du baust an entsprechenden Stellen eine Prüfung auf null ein. Also z.B. so:

if (p.getItemInHand() != null && p.getItemInHand().getType() == Material.WOOD_HOE) {