God Mode funktioniert nicht..

[PepaMC]

Hallo,
ich habe eine kurze Frage.. ich glaube ich stehe echt auf dem Schlauch warum mein Code nicht funktioniert.. irgendwie erkennt er nicht wenn der Spieler in der ArrayList ist..

Alles ist importiert, daran kann's nicht liegen.

package Commands;

import java.util.ArrayList;

import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageEvent;

import Main.JRC;

public class GodCommand implements CommandExecutor, Listener {
   
    ArrayList<String> god = new ArrayList<String>();
   
    public boolean onCommand(CommandSender sender, Command cmd, String label,
            String[] args) {
Player p = (Player)sender;

    if (label.equalsIgnoreCase("god")) {
      if (p.hasPermission("jrc.god")) {
        if (args.length == 0) {
            if(!god.contains(p.getName())) {
            p.setHealth(20D);
            p.setFoodLevel(20);
            p.setFireTicks(0);
            p.setExhaustion(0.0F);
            p.sendMessage(JRC.prefix + "§7Dein Godmode ist nun eingeschaltet");
            god.add(p.getName());
            } else {
                god.remove(p.getName());
                p.sendMessage(JRC.prefix + "§7Dein Godmode ist nun ausgeschaltet");
            }
        } else if (args.length == 1) {
          if (p.getServer().getPlayer(args[0]) != null) {
            Player p2 = p.getServer().getPlayer (
              args[0]);
            if(!god.contains(p2.getName())) {
            p.setHealth(20D);
            p2.setFoodLevel(20);
            p2.setFireTicks(0);
            p.setExhaustion(0.0F);
            p.sendMessage(JRC.prefix + "§7Du hast §6" + p2.getName() + "§7 Godmode gegeben.");
            p2.sendMessage(JRC.prefix + "§7Du hast von §6" + p.getName() + " §7Godmode bekommen.");
            god.add(p2.getName());
            } else {
                p.sendMessage(JRC.prefix + "§7Du hast §6" + p2.getName() + "§7 Godmode entzogen.");
                p2.sendMessage(JRC.prefix + "§7Du hast von §6" + p.getName() + " §7Godmode entzogen bekommen.");
            }
          } else {
            p.sendMessage(JRC.prefix + "§cDieser Spieler wurde nicht gefunden.");
          }
        }
      }
      else {
        p.sendMessage(JRC.perm);
      }
    }
    return false;
    }
   
     @EventHandler(priority = EventPriority.HIGH)
        public void onDamage(EntityDamageEvent event){
         Player p = (Player)event.getEntity();
         p.sendMessage("§7ERKANNT!");
         if(god.contains(p.getName())) {
         event.setCancelled(true);
         p.sendMessage("§aERKANNT!");
     }
     }
   
}

Freue mich auf Antworten

LG.

 

[delta242]

Ich kann dir schon sagen warum das nicht geht. Vermutlich machst du in deiner Hauptklasse etwas wie:

getCommand("god").setExecutor(new GodCommand());
und
Bukkit.getPluginManager().registerListener(this, new GodCommand());

Nun hast du jeweils ein neues Objekt der Klasse GodCommand und dadurch hast du auch zweimal deine "god" Liste. Es gibt in dem Fall zwei Möglichkeiten:

1. Du definierst die "god" Liste als static, dadurch ist sie nicht von Objekt zu Objekt unterschiedlich (also "static ArrayList<String> god = new ...")
2. In deiner Hauptklasse erstellst du nur ein Objekt der Klasse, speicherst dies ab und registrierst den Command sowie den Listener mit dem gleichen Objekt (also sowas in der Art:
GodCommand godCommand = new GodCommand();
getCommand("god").setExecutor(godCommand);
Bukkit.getPluginManager().registerListener(this, godCommand);

 

[PepaMC]

Static.. darauf hätte ich auch kommen sollen
Vielen dank!