Spigot Textausgabe kommt zwei Mal

[Spagel]

Hallo, ich möchte das wenn ein Spieler Rechtsklick auf einen VIllager macht das dieser einen Text ausgibt.

Allerdiengst bekommt er pro Click den Text 2 Mal..

        @EventHandler
        public void FAQInteract(PlayerInteractEntityEvent e) {
            Player p = e.getPlayer();
            if(e.getRightClicked() instanceof Villager);
            Villager FAQ = (Villager) e.getRightClicked();
            
            if(FAQ.getCustomName().equalsIgnoreCase("§a§lFAQ")) {

    
                
                e.setCancelled(true);
                
                
            p.sendMessage("§c§lFAQ");

Hat jemand eine Idee was ich dagegen machen kann?

Danke im Vorraus!

 

[BloodSKreaper]

Guten Tag,

ich meine, dass das Event sowohl für die Mainhand als auch Offhand getriggert wird. Das heißt, dass du beispielsweise abfragen kannst, ob getHand() die MainHand ist. Dann sollte die Dopplung verhindert werden.

Freundliche Grüße
BloodSKreaper

Quelle: https://hub.spigotmc.org/jira/plugins/servlet/mobile#issue/SPIGOT-1570

 

[Spagel]

Guten Tag,

ich meine, dass das Event sowohl für die Mainhand als auch Offhand getriggert wird. Das heißt, dass du beispielsweise abfragen kannst, ob getHand() die MainHand ist. Dann sollte die Dopplung verhindert werden.

Freundliche Grüße
BloodSKreaper

Danke für den Tipp, hättest du noch eine idee wie ich das abfragen könnte? Bin gerade total Lost

 

[Chrisliebär❤️]

Das ist eigentlich wirklich einfach aus der Dokumentation des Events zu lesen, daher auch von mir nur der Link https://hub.spigotmc.org/javadocs/spigot/org/bukkit/event/player/PlayerInteractEntityEvent.html

 

[BloodSKreaper]

In deinem Code sind mir gerade noch einige Fehler aufgefallen. Ich habe meine Änderungen mit Kommentaren beschrieben.

    @EventHandler
    public void FAQInteract(PlayerInteractEntityEvent e) {
        Player p = e.getPlayer();
        if (e.getHand() == EquipmentSlot.HAND) { //Hier wird geprüft, ob die MainHand verwendet wird.
            if (e.getRightClicked() instanceof Villager) { //Hier war ein Semicolon, Abfrage war also sinnlos. Wurde durch eine Klammer ersetzt.
            //Performantere Alternative zur oberen Abfrage, aber Zeitgewinn nahezu vernachlässigbar.
                //if(e.getRightClicked().getType() == EntityType.VILLAGER){
                Villager FAQ = (Villager) e.getRightClicked();
                if (FAQ.getCustomName() != null) { //Null-Abfrage eingefügt um eine NullpointerExeption zu vermeiden, falls der NPC keinen CustomName hat
                    if (FAQ.getCustomName().equalsIgnoreCase("§a§lFAQ")) {
                        e.setCancelled(true);
                        p.sendMessage("§c§lFAQ");
                    }
                }
            }
        }
    }

Freundliche Grüße
BloodSKreaper

 

[De4thpr0x4]

Hier mal ein CodeSchnipsel

        if(e.getAction().equals(Action. SUCH DIR WAS AUS)){
            
        }

 

[BloodSKreaper]

e.getAction()

Diese Methode existiert nicht in der Klasse PlayerInteractEntityEvent

 

[De4thpr0x4]

Achso tut mir leid Fehler

 

[TrinityX]

In deinem Code sind mir gerade noch einige Fehler aufgefallen. Ich habe meine Änderungen mit Kommentaren beschrieben.

    @EventHandler
    public void FAQInteract(PlayerInteractEntityEvent e) {
        Player p = e.getPlayer();
        if (e.getHand() == EquipmentSlot.HAND) { //Hier wird geprüft, ob die MainHand verwendet wird.
            if (e.getRightClicked() instanceof Villager) { //Hier war ein Semicolon, Abfrage war also sinnlos. Wurde durch eine Klammer ersetzt.
            //Performantere Alternative zur oberen Abfrage, aber Zeitgewinn nahezu vernachlässigbar.
                //if(e.getRightClicked().getType() == EntityType.VILLAGER){
                Villager FAQ = (Villager) e.getRightClicked();
                if (FAQ.getCustomName() != null) { //Null-Abfrage eingefügt um eine NullpointerExeption zu vermeiden, falls der NPC keinen CustomName hat
                    if (FAQ.getCustomName().equalsIgnoreCase("§a§lFAQ")) {
                        e.setCancelled(true);
                        p.sendMessage("§c§lFAQ");
                    }
                }
            }
        }
    }

Freundliche Grüße
BloodSKreaper

Es ist zwar nett und bestimmt nur gut gemeint jemanden so helfen zu wollen, aber ich finde, dass es besser wäre mit Worten zu erklären wie es gemacht werden sollte, so dass der jeweils andere noch dazulernen kann indem er den Code dann selbst schreibt und nicht einfach kopieren kann.

Liebe Grüße,
Trinity

 

[PaintableToast]

Es ist zwar nett und bestimmt nur gut gemeint jemanden so helfen zu wollen, aber ich finde, dass es besser wäre mit Worten zu erklären wie es gemacht werden sollte, so dass der jeweils andere noch dazulernen kann indem er den Code dann selbst schreibt und nicht einfach kopieren kann.

Dies hat er ja aber mit den Hinweisen die er neben seiner Änderung geschrieben hat. Ich finde es teilweise nicht besser in Worten einfach nur zu sagen zum Beispiel du hast etwas Null definiert natürlich wäre es praktisch wenn man etwas in Worten erklärt, jedoch denke ich das man dann auch wirklich alles erklären sollte und nicht einfach nur ein Wort schreiben sollte oder so denn dann ist es wiederum lehrreicher einfach den Code zu bekommen ... Das ist meine Stellung dazu.

 

[BloodSKreaper]

Es ist zwar nett und bestimmt nur gut gemeint jemanden so helfen zu wollen, aber ich finde, dass es besser wäre mit Worten zu erklären wie es gemacht werden sollte, so dass der jeweils andere noch dazulernen kann indem er den Code dann selbst schreibt und nicht einfach kopieren kann.

Liebe Grüße,
Trinity

Hab ich das nicht getan? Prizipiell ist die Logik seines Programmes richtig, ich habe nur Flüchtigkeitsfehler markiert und verbessert sowie beschrieben. Das grundlegende Problem der doppelten Ausführung ist nicht auf den ersten Blick ersichtlich, leider auch nicht aus den Javadocs. Die Abfrage mit der Hand ist zwar prinzipiell nicht schwer, aber ich kann verstehen, dass es bei dem System mit den Equipmentslots schnell zur Verwirrung kommt, weshalb ich hier einfach mal aufzeigen wollte, wie es geht. Kurz zur Erinnerung: Das eigentliche Problem wurde mit einer Zeile Code gelöst. Eine ausführliche Beschreibung davon ist an dieser Stelle nicht nötig, da ich dies bereits in einem früheren Post getan habe, aber ohne Code.

Bin gerade total Lost

Wer hatte noch nie einen solchen Moment, wo die Lösung eigentlich offensichtlich ist, aber man einfach nicht darauf kommt?

Freundliche Grüße
BloodSKreaper

 

[Spagel]

In deinem Code sind mir gerade noch einige Fehler aufgefallen. Ich habe meine Änderungen mit Kommentaren beschrieben.

    @EventHandler
    public void FAQInteract(PlayerInteractEntityEvent e) {
        Player p = e.getPlayer();
        if (e.getHand() == EquipmentSlot.HAND) { //Hier wird geprüft, ob die MainHand verwendet wird.
            if (e.getRightClicked() instanceof Villager) { //Hier war ein Semicolon, Abfrage war also sinnlos. Wurde durch eine Klammer ersetzt.
            //Performantere Alternative zur oberen Abfrage, aber Zeitgewinn nahezu vernachlässigbar.
                //if(e.getRightClicked().getType() == EntityType.VILLAGER){
                Villager FAQ = (Villager) e.getRightClicked();
                if (FAQ.getCustomName() != null) { //Null-Abfrage eingefügt um eine NullpointerExeption zu vermeiden, falls der NPC keinen CustomName hat
                    if (FAQ.getCustomName().equalsIgnoreCase("§a§lFAQ")) {
                        e.setCancelled(true);
                        p.sendMessage("§c§lFAQ");
                    }
                }
            }
        }
    }

Freundliche Grüße
BloodSKreaper

Hallo

BloodSKreaper, Danke für die Anleitung und die Erklärung nebenbei da hat man noch was dazu gelernt!!

MFG


Spagel