@Team Message wird nicht geCancelled

[Kraetzy]

Heyho,

Meine @team Nachricht erscheint im Chat obwohl ich die MSG selber geCancelled habe!

package chatsystem;

import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.AsyncPlayerChatEvent;

public class TeamChat implements Listener {
   
    @EventHandler
    public void onChat(AsyncPlayerChatEvent e) {
        Player p = e.getPlayer();
       
        if(e.getMessage().startsWith("@Team")) {
            if(p.hasPermission("system.tc")) {
                String msg = e.getMessage();
               
                for(Player team : Bukkit.getOnlinePlayers()) {
                    if(team.hasPermission("system.tc")) {
                        e.setCancelled(true);
                        team.sendMessage("§cTeamChat §8× §9" + e.getPlayer().getName() + msg.replaceAll("@Team", " §8» §7"));
                       
                    }
                }
               
               
            } else {
                e.setCancelled(false);
            }

        } else if(e.getMessage().startsWith("@team")) {
            if(p.hasPermission("system.tc")) {
                String msg = e.getMessage();
               
                for(Player team : Bukkit.getOnlinePlayers()) {
                    if(team.hasPermission("system.tc")) {
                        e.setCancelled(true);
                        team.sendMessage("§cTeamChat §8× §9" + e.getPlayer().getName() + msg.replaceAll("@team", " §8» §7"));
                       
                    }
                }
               
               
            } else {
                e.setCancelled(false);
            }

        } else if(e.getMessage().startsWith("@T")) {
            if(p.hasPermission("system.tc")) {
                e.setCancelled(true);
                String msg = e.getMessage();
               
                for(Player team : Bukkit.getOnlinePlayers()) {
                    if(team.hasPermission("system.tc")) {
                        team.sendMessage("§cTeamChat §8× §9" + e.getPlayer().getName() + msg.replaceAll("@T", " §8» §7"));
                       
                    }
                }
               
               
            } else {
                e.setCancelled(false);
            }

        } else if(e.getMessage().startsWith("@t")) {
            if(p.hasPermission("system.tc")) {
                String msg = e.getMessage();
               
                for(Player team : Bukkit.getOnlinePlayers()) {
                    if(team.hasPermission("system.tc")) {
                        team.sendMessage("§cTeamChat §8× §9" + e.getPlayer().getName() + msg.replaceAll("@t", " §8» §7"));
                       
                    }
                }
               
               
            } else {
                e.setCancelled(false);
            }

        }
       
    }
   
}

 

[deleted196100]

Ich würde dir empfehlen für das "@Team", "@team", "@T" einen Regulären Ausdruck (regex) zu schreiben und diesen zu Prüfen oder deine ganzen startswith Abfragen mit einem logischem "oder" -> "||" zusammenzufassen, so wie es aktuell ist hast du sehr viel redundanten Code was die Wartbarkeit deutlich einschränkt und eventuell auch dein Problem sein könnte (siehe unten).

• for(Player team : Bukkit.getOnlinePlayers()) {

• if(team.hasPermission("system.tc")) {

• e.setCancelled(true);

• team.sendMessage("§cTeamChat §8× §9" + e.getPlayer().getName() + msg.replaceAll("@Team", " §8» §7"));

• }

}

Dieser Teil macht keinen Sinn, du cancellst das Event in einer Schleife abhängig davon ob Spieler online sind die die Nachricht empfangen können. Ich würde dir raten das Event vor der Schleife zu cancelln.

• } else if(e.getMessage().startsWith("@t")) {

• if(p.hasPermission("system.tc")) {

• String msg = e.getMessage();

• for(Player team : Bukkit.getOnlinePlayers()) {

• if(team.hasPermission("system.tc")) {

• team.sendMessage("§cTeamChat §8× §9" + e.getPlayer().getName() + msg.replaceAll("@t", " §8» §7"));

• }

• }

• } else {

• e.setCancelled(false);

• }

In diesem Fall wird das Event gar nicht gecancellt wenn der Spieler die benötigte Permission besitzt, du hast uns zwar nicht mitgeteilt mit welcher Nachricht du das ganze getestet hast allerdings könnte das der Fehler sein wenn du mit "@t" getestet hast.
Du solltest außerdem noch einmal prüfen ob du dir überhaupt die nötige Permission zugewiesen hast.

Den Code würde ich in jedem Fall etwas aufräumen, so wie es jetzt ist hast du damit bald keinen Spaß mehr, glaub mir.

 

[KaeseKuchen_DEV]

Du hast in der ersten For-Schleife abgefragt ob jeder Spieler die Permission hat. Sobald jetzt ein Spieler die permission nicht hat wird die Nachricht trotzdem gesendet. Also musst du vor jede For-Schleife erst das Event canceln.

MfG KaeseKuchenDEV

 

[JTK222]

Du hast in der ersten For-Schleife abgefragt ob jeder Spieler die Permission hat. Sobald jetzt ein Spieler die permission nicht hat wird die Nachricht trotzdem gesendet. Also musst du vor jede For-Schleife erst das Event canceln.

MfG KaeseKuchenDEV

Wenn du schon helfen willst dann aber bitte richtig!
Sobald auch nur ein einziger Spieler die permission hat, wird gecancelt, was an dieser Stelle immer der Fall ist,
da ja bereits überprüft wurde ob der Sender die Rechte hat.

Ich denke auch dass hier beim testen ganz einfach die Permission fehlerhaft war.

 

[Figz]

Genauso gut, kannst du dir die Abfrage, ob die Nachricht mit "@Team" oder "@team" anfängt sparen.
Es würde schließlich auch mit "@tunte" ausgeführt werden

 

[Chrisliebär❤️]

Der TE hat sich nach dem Posten dieses Beitrag nicht ein einziges Mal im Forum sehen lassen. Eure Mühen sind also vergebens.