Wie kann ich eine ArrayList aus einer Klasse in eine andere übergeben?

jonny_2k12

jonny_2k12

Mitglied
Hi, frage steht oben, im anhang screenshots.


Klasse 1:
code1.PNG



Klasse 2:
code.2.PNG


wie kann ich hier einen wert der arraylist admin hinzufügen?

Danke im vorraus
jonny_2k12
 
kneitzel

kneitzel

Top Contributor
A) Code mite mit Code Tags (Button </> über dem Eingabefeld) posten. So mit Bildschirmfotos ist es nicht gut zu lesen.

B) du benötigst in der zweiten Klasse die Referenz auf die Instanz der anderen Klasse. Dann kannst du darauf zugreifen.

C) Strings vergleicht man mit equals und nicht mit ==.
 
jonny_2k12

jonny_2k12

Mitglied
A) Code mite mit Code Tags (Button </> über dem Eingabefeld) posten. So mit Bildschirmfotos ist es nicht gut zu lesen.

B) du benötigst in der zweiten Klasse die Referenz auf die Instanz der anderen Klasse. Dann kannst du darauf zugreifen.

C) Strings vergleicht man mit equals und nicht mit ==.
Hi, danke, bin noch sehr neu in Java (3 Tage), deshalb sorry wenn noch nicht alles richtig ist. :D
 
jonny_2k12

jonny_2k12

Mitglied
Hi, danke, bin noch sehr neu in Java (3 Tage), deshalb sorry wenn noch nicht alles richtig ist. :D
A) Code mite mit Code Tags (Button </> über dem Eingabefeld) posten. So mit Bildschirmfotos ist es nicht gut zu lesen.

B) du benötigst in der zweiten Klasse die Referenz auf die Instanz der anderen Klasse. Dann kannst du darauf zugreifen.

C) Strings vergleicht man mit equals und nicht mit ==.
Also wie soll ich die strings vergleichen?

Java:
UUID id = target.getUniqueId();
String rang = args[0];
if(rang equals "Admin") {
    admin.add(id);
    if(moderator.contains(id)) {
        moderator.remove(id);
    }
}

so?
 
Zuletzt bearbeitet:
kneitzel

kneitzel

Top Contributor
String sind eine Klasse in Java und erbt damit von Object.
Object bietet eine Methode equals(Object) und diese wäre zu nutzen. Also etwas wie:
Java:
if(rang.equals("Admin") {

Falls rang auch null sein könnte, würde es sich anbieten, dies umzudrehen:
Java:
if("Admin".equals(rang) {

Dieses equals wäre die Methode, mit der generell Instanzen verglichen werden sollten.

String bietet natürlich noch weitere Methoden - falls die Gross-/Kleinschreibung egal sein sollte, dann gäbe es z.B. equalsIgnoreCase:
Java:
if("Admin".equalsIgnoreCase(rang) {

Hi, danke, bin noch sehr neu in Java (3 Tage), deshalb sorry wenn noch nicht alles richtig ist. :D
Das ist absolut kein Thema - daher kommen diese Hinweise. Auch wenn diese teilweise kurz ausfallen: Das bitte nicht als Unhöflich oder Abwerten ansehen. Das liegt einfach nur daran, dass es sich z.B. bei mir nur um kurze Pausen handelt und daher nur so kurze Antworten entstehen. Du bist herzlich willkommen und wir helfen gerne weiter. (Erst Recht, wenn so Hinweise direkt aufgenommen und umgesetzt werden, wie z.B. die Verwendung der Code Tags :) )

Da bezüglich B) noch keine Rückmeldung gekommen ist: Hat das so ausgereicht um durch eigene Recherche weiter zu kommen oder brauchst Du dies ausführlicher? (Es ist kein Problem, da nachzufragen oder mehr einzufordern! Manchmal sind Antworten unverständlich oder so... Daher nicht zögern, wenn es nicht weiter hilft.)

Das einfach mal als eine Antwort mit etwas mehr Text :)
 
jonny_2k12

jonny_2k12

Mitglied
Da bezüglich B) noch keine Rückmeldung gekommen ist: Hat das so ausgereicht um durch eigene Recherche weiter zu kommen oder brauchst Du dies ausführlicher? (Es ist kein Problem, da nachzufragen oder mehr einzufordern! Manchmal sind Antworten unverständlich oder so... Daher nicht zögern, wenn es nicht weiter hilft.)
zu B): hab einfach gemacht:
Java:
public class RankCommand extends ChatListener {

(ChatListener ist die Klasse, in der die ArrayLists erstellt werden.)

Vielen Dank nochmal für deine Hilfe! :)
 
jonny_2k12

jonny_2k12

Mitglied
EDIT: hab den Fehler gefunden :)




@kneitzel Weitere Frage :D

Nun hab ich die ID des Players zur ArrayList hinzugefügt, aber das erhoffte ergebnis funktioniert noch nicht... :D
Kannst du hier irgendwo einen Fehler erkennen? :)

Java:
public class ChatListener implements Listener {
   
public List<UUID> admin = new ArrayList<>();
public List<UUID> moderator = new ArrayList<>();
public List<UUID> supporter = new ArrayList<>();
public List<UUID> builder = new ArrayList<>();
public List<UUID> youtuber = new ArrayList<>();
public List<UUID> premium = new ArrayList<>();
public List<UUID> spieler = new ArrayList<>();

   

    @EventHandler
    public void onChatEvent(AsyncPlayerChatEvent e) {
        Player player = (Player)e.getPlayer();
        UUID id = (UUID)player.getUniqueId();
        String message = (String)e.getMessage();
        if(admin.contains(id)) {
            Bukkit.broadcastMessage("§c§l[Admin] §r" + player + message);
        }else if(moderator.contains(id)) {
            Bukkit.broadcastMessage("§1[Moderator] §r" + player + message);
        }else if(supporter.contains(id)) {
            Bukkit.broadcastMessage("§2[Supporter] §r" + player + message);  
        }else if(builder.contains(id)) {
                Bukkit.broadcastMessage("§3[Builder] §r" + player + message);
        }else if(youtuber.contains(id)) {
            Bukkit.broadcastMessage("§5[Youtuber] §r" + player + message);
        }else if(premium.contains(id)) {
            Bukkit.broadcastMessage("§6[Premium] §r" + player + message);
        }else {
            Bukkit.broadcastMessage("§r[Spieler] §r" + player + message);
        }
        e.setCancelled(true);
       
   
    }




}

 
Zuletzt bearbeitet:
kneitzel

kneitzel

Top Contributor
Da möchte ich einmal kurz einhaken, dann das mag zwar technisch funktionieren in Deinem Fall, aber das ist so nicht wirklich sinnvoll.

Damit hast Du die Aussage: RankCommand ist ein ChatListener. Aber das ist ggf. nicht wirklich so richtig.

Ich kenne Deinen genauen Code nicht, daher ist es schwer, dir da eine konkrete Lösung zu geben. Aber auf den ersten Blick sieht es so aus, also ob Du da eine Art Userverwaltung hast. Daher wäre da eine eigene Klasse denkbar: UserManagement und diese Klasse hat dann die List<UUID> drin.
Auf diese Listen gibst Du aber keinen direkten Zugriff sondern statt dessen gehst Du hin und erstellst dann Methoden, die den Zugriff erledigen.
Also sowas wie

Java:
public void addAdmin(final Player player) {
    admin.add(player.getUniqueId());
}

public boolean isAdmin(final Player player) {
    admin.contains(player.getUniqueId());
}

Deine Klassen RankCommand und ChatListener kennen dann die Instanz vom UserManagement. Die hätten dann also sowas:
Java:
private UserManagement userManagement;

public void setUserManagement(final UserManagement userManagement) {
    this.userManagement = userManagement;
}

// Und dann in einer Methode der Klasse
userManagement.isAdmin(somePlayer);
// oder
userManagement.addAdmin(somePlayer);

Wichtig ist aber, dass Du bei der Erstellung der Instanzen jeweils das UserManagement setzt. Also Du erstellst eine Instanz von UserManagement und die gibst Du dann an die Instanzen der anderen Klassen...

Und das Problem, das Du Du ggf. in Deinem Post hattest:
Du hast zwei Instanzen gehabt: einmal vom ChatListener und einmal vom RankCommand. Damit hattest Du zwei Mal die ganzen Listen. Wenn Du also bei RankCommand einen Player in einer Liste hinzugefügt hast, dann ist der User bei dem ChatListener nicht enthalten.
 
jonny_2k12

jonny_2k12

Mitglied
Da möchte ich einmal kurz einhaken, dann das mag zwar technisch funktionieren in Deinem Fall, aber das ist so nicht wirklich sinnvoll.

Damit hast Du die Aussage: RankCommand ist ein ChatListener. Aber das ist ggf. nicht wirklich so richtig.

Ich kenne Deinen genauen Code nicht, daher ist es schwer, dir da eine konkrete Lösung zu geben. Aber auf den ersten Blick sieht es so aus, also ob Du da eine Art Userverwaltung hast. Daher wäre da eine eigene Klasse denkbar: UserManagement und diese Klasse hat dann die List<UUID> drin.
Auf diese Listen gibst Du aber keinen direkten Zugriff sondern statt dessen gehst Du hin und erstellst dann Methoden, die den Zugriff erledigen.
Also sowas wie

Java:
public void addAdmin(final Player player) {
    admin.add(player.getUniqueId());
}

public boolean isAdmin(final Player player) {
    admin.contains(player.getUniqueId());
}

Deine Klassen RankCommand und ChatListener kennen dann die Instanz vom UserManagement. Die hätten dann also sowas:
Java:
private UserManagement userManagement;

public void setUserManagement(final UserManagement userManagement) {
    this.userManagement = userManagement;
}

// Und dann in einer Methode der Klasse
userManagement.isAdmin(somePlayer);
// oder
userManagement.addAdmin(somePlayer);

Wichtig ist aber, dass Du bei der Erstellung der Instanzen jeweils das UserManagement setzt. Also Du erstellst eine Instanz von UserManagement und die gibst Du dann an die Instanzen der anderen Klassen...

Und das Problem, das Du Du ggf. in Deinem Post hattest:
Du hast zwei Instanzen gehabt: einmal vom ChatListener und einmal vom RankCommand. Damit hattest Du zwei Mal die ganzen Listen. Wenn Du also bei RankCommand einen Player in einer Liste hinzugefügt hast, dann ist der User bei dem ChatListener nicht enthalten.
Hi, ich verstehe technisch leider nur die Hälfte von dem was du geschrieben hast 😅

ich hänge einfach mal den kompletten Code beider Klassen an. wichtig ist nur dass man die UUIDs des Spieler speichert, nicht den Namen, weil wenn er sonst seinen Namen ändert ist im Chat sein Rang weg..

mit den 2 Listen könntest du recht haben...
mein erster Fehler war, dass der Listener nicht in der Main registriert war.

jetzt funktioniert zwar der Spieler prefix, aber wenn ich meine UUID in RankCommand in eine Liste gebe, so merkt der ChatListener das tatsächlich nicht...


hier die Codes:

ChatListener:
package de.jonny_2k12.chatsystemjonny.listeners;

import java.util.ArrayList;
import java.util.List;
import java.util.UUID;

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 ChatListener implements Listener {
    
public List<UUID> admin = new ArrayList<>();
public List<UUID> moderator = new ArrayList<>();
public List<UUID> supporter = new ArrayList<>();
public List<UUID> builder = new ArrayList<>();
public List<UUID> youtuber = new ArrayList<>();
public List<UUID> premium = new ArrayList<>();
public List<UUID> spieler = new ArrayList<>();

    

    @EventHandler
    public void onChatEvent(AsyncPlayerChatEvent e) {
        String player = (String)e.getPlayer().getName();
        Player player1 = (Player)e.getPlayer();
        UUID id = (UUID)player1.getUniqueId();
        String message = (String)e.getMessage();
        if(admin.contains(id)) {
            Bukkit.broadcastMessage("§c§l[Admin] §r" + player + message);
        }else if(moderator.contains(id)) {
            Bukkit.broadcastMessage("§1[Moderator] §r" + player + message);
        }else if(supporter.contains(id)) {
            Bukkit.broadcastMessage("§2[Supporter] §r" + player + message);   
        }else if(builder.contains(id)) {
                Bukkit.broadcastMessage("§3[Builder] §r" + player + message);
        }else if(youtuber.contains(id)) {
            Bukkit.broadcastMessage("§5[Youtuber] §r" + player + message);
        }else if(premium.contains(id)) {
            Bukkit.broadcastMessage("§6[Premium] §r" + player + message);
        }else {
            Bukkit.broadcastMessage("§r[Spieler] §r" + player + message);
        }
        e.setCancelled(true);
        
    
    }




}


RankCommand:
package de.jonny_2k12.chatsystemjonny.commands;

import java.util.Arrays;
import java.util.UUID;

import org.bukkit.Bukkit;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;

import de.jonny_2k12.chatsystemjonny.listeners.ChatListener;
import de.jonny_2k12.chatsystemjonny.main.Main;

public class RankCommand extends ChatListener implements CommandExecutor {

    public RankCommand(Main plugin) {                   
        plugin.getCommand("rank").setExecutor(this);
        plugin.getCommand("rank").setDescription("/rank <Rang> <Player>");
    }
    
    @Override
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
        if(sender instanceof Player) {
            Player player = (Player) sender;
            if(player.hasPermission("chat.rank")) {
                if(args.length == 0) {
                    player.sendMessage("§6[Essentials] §4Bitte benutze §6/rank <Rang> <Player>§4!");
                } else if(args.length == 2) {
                    Player target = Bukkit.getPlayer(args[1]);
                    if(target != null) {
                        UUID id = target.getUniqueId();
                        String rang = args[0];
                        if(rang.equals("Admin")) {
                            admin.add(id);
                            if(moderator.contains(id)) {
                                moderator.remove(id);
                            }else if(supporter.contains(id)) {
                                supporter.remove(id);
                            }else if(builder.contains(id)) {
                                builder.remove(id);
                            }else if(youtuber.contains(id)) {
                                youtuber.remove(id);
                            }else if(premium.contains(id)) {
                                premium.remove(id);
                            }else if(spieler.contains(id));
                            spieler.remove(id);
        
                        }else if(rang.equals("Moderator")) {
                            moderator.add(id);
                            if(admin.contains(id)) {
                                admin.remove(id);
                            }else if(supporter.contains(id)) {
                                supporter.remove(id);
                            }else if(builder.contains(id)) {
                                builder.remove(id);
                            }else if(youtuber.contains(id)) {
                                youtuber.remove(id);
                            }else if(premium.contains(id)) {
                                premium.remove(id);
                            }else if(spieler.contains(id));
                            spieler.remove(id);
                        }else if(rang.equals("Builder")) {
                            builder.add(id);
                            if(moderator.contains(id)) {
                                moderator.remove(id);
                            }else if(supporter.contains(id)) {
                                supporter.remove(id);
                            }else if(admin.contains(id)) {
                                admin.remove(id);
                            }else if(youtuber.contains(id)) {
                                youtuber.remove(id);
                            }else if(premium.contains(id)) {
                                premium.remove(id);
                            }else if(spieler.contains(id));
                            spieler.remove(id);
                        }else if(rang.equals("Premium")) {
                            premium.add(id);
                            if(moderator.contains(id)) {
                                moderator.remove(id);
                            }else if(supporter.contains(id)) {
                                supporter.remove(id);
                            }else if(builder.contains(id)) {
                                builder.remove(id);
                            }else if(youtuber.contains(id)) {
                                youtuber.remove(id);
                            }else if(admin.contains(id)) {
                                admin.remove(id);
                            }else if(spieler.contains(id));
                            spieler.remove(id);
                        }else if(rang.equals("Youtuber")) {
                            youtuber.add(id);
                            if(moderator.contains(id)) {
                                moderator.remove(id);
                            }else if(supporter.contains(id)) {
                                supporter.remove(id);
                            }else if(builder.contains(id)) {
                                builder.remove(id);
                            }else if(admin.contains(id)) {
                                admin.remove(id);
                            }else if(premium.contains(id)) {
                                premium.remove(id);
                            }else if(spieler.contains(id));
                            spieler.remove(id);
                        }else if(rang.equals("Supporter")) {
                            supporter.add(id);
                            if(moderator.contains(id)) {
                                moderator.remove(id);
                            }else if(admin.contains(id)) {
                                admin.remove(id);
                            }else if(builder.contains(id)) {
                                builder.remove(id);
                            }else if(youtuber.contains(id)) {
                                youtuber.remove(id);
                            }else if(premium.contains(id)) {
                                premium.remove(id);
                            }else if(spieler.contains(id));
                            spieler.remove(id);
                        }else if(rang.equals("Spieler")) {
                            spieler.add(id);
                            if(moderator.contains(id)) {
                                moderator.remove(id);
                            }else if(admin.contains(id)) {
                                admin.remove(id);
                            }else if(builder.contains(id)) {
                                builder.remove(id);
                            }else if(youtuber.contains(id)) {
                                youtuber.remove(id);
                            }else if(premium.contains(id)) {
                                premium.remove(id);
                            }else if(supporter.contains(id));
                                supporter.remove(id);
                        }else {
                            player.sendMessage("§6[Essentials] §4Das ist kein gültiger Rang!");
                        }
                    } else
                        player.sendMessage("§6[Essentials] §4Der Spieler §6" + args[1] + "§4 ist nicht online!");
                }else
                    player.sendMessage("§6[Essentials] §4Bitte benutze §6/rank <Rang> <Player>§4!");
                
            } else
                player.sendMessage("§6[Essentials] §4§lDazu hast du keine Rechte!");
        } else {
            sender.sendMessage("Diesen Command können nur Spieler nutzen!");
        }
        return false;
    }

}




Main:
package de.jonny_2k12.chatsystemjonny.main;

import org.bukkit.Bukkit;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;

import de.jonny_2k12.chatsystemjonny.commands.RankCommand;
import de.jonny_2k12.chatsystemjonny.listeners.ChatListener;
import de.jonny_2k12.chatsystemjonny.listeners.JoinListener;
import de.jonny_2k12.chatsystemjonny.listeners.QuitListener;

public class Main extends JavaPlugin{
    
    @Override
    public void onEnable() {
        Bukkit.getServer().getPluginManager().registerEvents(new JoinListener(), this);
        Bukkit.getServer().getPluginManager().registerEvents(new QuitListener(), this);
        Bukkit.getServer().getPluginManager().registerEvents(new ChatListener(), this);
        
        new RankCommand(this);
}
}
 
kneitzel

kneitzel

Top Contributor
Ich habe mal im Editor (also ohne jede Kontrolle durch einen Compiler - Tippfehler sind also möglich) paar Dinge geändert.
Aber das sind nur Auszüge!

Also als erstes habe ich ein UserManagement gebaut:
Java:
package de.jonny_2k12.chatsystemjonny.listeners;

import java.util.ArrayList;
import java.util.List;
import java.util.UUID;
import org.bukkit.entity.Player;

public class UserManagement {
    private List<UUID> admin = new ArrayList<>();
    private List<UUID> moderator = new ArrayList<>();
    private List<UUID> supporter = new ArrayList<>();
    private List<UUID> builder = new ArrayList<>();
    private List<UUID> youtuber = new ArrayList<>();
    private List<UUID> premium = new ArrayList<>();
    private List<UUID> spieler = new ArrayList<>();

    public void addAdmin(final Player player) {
        admin.add(player.getUniqueId());
    }
  
    public boolean isAdmin(final Player player) {
        return admin.contains(player.getUniqueId());
    }

    public void removeAdmin(final Player player) {
        if(admin.contains(player.getUniqueId()) {
            admin.remove(player.getUniqueId());
        }
    }

    // Die anderen Methoden so aufbauen wie die für Admin!
}

Wichtig ist: So wie ich die Methoden für Admin geschrieben habe, kannst Du 1:1 die Methoden für die anderen Listen schreiben.
Also einfach die Methoden kopieren und dann [aA]dmin durch z.B. [mM]oderator ersetzen.

Dann habe ich den anderen Klassen ein UserManagement gegeben und das wird im Konstruktor mit übergeben.
Und dann im Code wird dann auf userManagement zugegriffen. Da beim remove die Prüfung enthalten ist, muss die im RankController nicht mehr stattfinden. So Du das als feste Regel haben willst, kannst Du das Entfernen sogar in das UserManagement rein nehmen:
Java:
    public void addAdmin(final Player player) {
        admin.add(player.getUniqueId());
        removeSupporter(player);
        remove.....(player);
        remove.....(player);
        ...
    }
Dann würden die remove Aufrufe nach dem addAdmin und so entfallen...

RankCommand würde dann (Ausschnitt):
Java:
public class RankCommand extends ChatListener implements CommandExecutor {

    private UserManagement userManagement;

    public RankCommand(final Main plugin, final UserManagement userManagement) {                 
        this.userManagement = userManagement;
        plugin.getCommand("rank").setExecutor(this);
        plugin.getCommand("rank").setDescription("/rank <Rang> <Player>");
    }
  
    @Override
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
        if(sender instanceof Player) {
            Player player = (Player) sender;
            if(player.hasPermission("chat.rank")) {
                if(args.length == 0) {
                    player.sendMessage("§6[Essentials] §4Bitte benutze §6/rank <Rang> <Player>§4!");
                } else if(args.length == 2) {
                    Player target = Bukkit.getPlayer(args[1]);
                    if(target != null) {
                        String rang = args[0];
                        if(rang.equals("Admin")) {
                            userManagement.addAdmin(target);
                            userManagement.removeModerator(target);
                            userManagement.removeSupporter(target);
                            // Ab hier müsstest du weiter anpassen ...

Und der ChatListener:
Java:
public class ChatListener implements Listener {

    private UserManagement userManagement;

    public ChatListener(final UserManagement userManagement) {                 
        this.userManagement = userManagement;
    }
  

    @EventHandler
    public void onChatEvent(AsyncPlayerChatEvent e) {
        String player = (String)e.getPlayer().getName();
        Player player1 = (Player)e.getPlayer();
        UUID id = (UUID)player1.getUniqueId();
        String message = (String)e.getMessage();
        if(userManagement.isAdmin(player1)) {
            Bukkit.broadcastMessage("§c§l[Admin] §r" + player + message);
        }else if(userManagement.isModerator(player1)) {
            Bukkit.broadcastMessage("§1[Moderator] §r" + player + message);

        // Ab hier selbst ändern ...

Und die Main muss natürlich auch angepasst werden, denn da erstellen wir das UserManagement einmalig und übergeben es bei den Konstuktoren der anderen beiden Klassen:
Java:
public class Main extends JavaPlugin{
  
    @Override
    public void onEnable() {
        UserManagement userManagement = new UserManagement();
        Bukkit.getServer().getPluginManager().registerEvents(new JoinListener(), this);
        Bukkit.getServer().getPluginManager().registerEvents(new QuitListener(), this);
        Bukkit.getServer().getPluginManager().registerEvents(new ChatListener(userManagement), this);
      
        new RankCommand(this, userManagement);
    }
}

Das wäre dann mein Vorschlag etwas ausführlicher.

Edit: Die markierten Zeilen waren verschoben - er hatte eine Leerzeile am Anfang im Editor drin, die er wohl entfernt hat und dadurch gab es die Verschiebung ... wurde behoben...
 
jonny_2k12

jonny_2k12

Mitglied
Ich habe mal im Editor (also ohne jede Kontrolle durch einen Compiler - Tippfehler sind also möglich) paar Dinge geändert.
Aber das sind nur Auszüge!

Also als erstes habe ich ein UserManagement gebaut:
Java:
package de.jonny_2k12.chatsystemjonny.listeners;

import java.util.ArrayList;
import java.util.List;
import java.util.UUID;
import org.bukkit.entity.Player;

public class UserManagement {
    private List<UUID> admin = new ArrayList<>();
    private List<UUID> moderator = new ArrayList<>();
    private List<UUID> supporter = new ArrayList<>();
    private List<UUID> builder = new ArrayList<>();
    private List<UUID> youtuber = new ArrayList<>();
    private List<UUID> premium = new ArrayList<>();
    private List<UUID> spieler = new ArrayList<>();

    public void addAdmin(final Player player) {
        admin.add(player.getUniqueId());
    }

    public boolean isAdmin(final Player player) {
        return admin.contains(player.getUniqueId());
    }

    public void removeAdmin(final Player player) {
        if(admin.contains(player.getUniqueId()) {
            admin.remove(player.getUniqueId());
        }
    }

    // Die anderen Methoden so aufbauen wie die für Admin!
}

Wichtig ist: So wie ich die Methoden für Admin geschrieben habe, kannst Du 1:1 die Methoden für die anderen Listen schreiben.
Also einfach die Methoden kopieren und dann [aA]dmin durch z.B. [mM]oderator ersetzen.

Dann habe ich den anderen Klassen ein UserManagement gegeben und das wird im Konstruktor mit übergeben.
Und dann im Code wird dann auf userManagement zugegriffen. Da beim remove die Prüfung enthalten ist, muss die im RankController nicht mehr stattfinden. So Du das als feste Regel haben willst, kannst Du das Entfernen sogar in das UserManagement rein nehmen:
Java:
    public void addAdmin(final Player player) {
        admin.add(player.getUniqueId());
        removeSupporter(player);
        remove.....(player);
        remove.....(player);
        ...
    }
Dann würden die remove Aufrufe nach dem addAdmin und so entfallen...

RankCommand würde dann (Ausschnitt):
Java:
public class RankCommand extends ChatListener implements CommandExecutor {

    private UserManagement userManagement;

    public RankCommand(final Main plugin, final UserManagement userManagement) {               
        this.userManagement = userManagement;
        plugin.getCommand("rank").setExecutor(this);
        plugin.getCommand("rank").setDescription("/rank <Rang> <Player>");
    }

    @Override
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
        if(sender instanceof Player) {
            Player player = (Player) sender;
            if(player.hasPermission("chat.rank")) {
                if(args.length == 0) {
                    player.sendMessage("§6[Essentials] §4Bitte benutze §6/rank <Rang> <Player>§4!");
                } else if(args.length == 2) {
                    Player target = Bukkit.getPlayer(args[1]);
                    if(target != null) {
                        String rang = args[0];
                        if(rang.equals("Admin")) {
                            userManagement.addAdmin(target);
                            userManagement.removeModerator(target);
                            userManagement.removeSupporter(target);
                            // Ab hier müsstest du weiter anpassen ...

Und der ChatListener:
Java:
public class ChatListener implements Listener {

    private UserManagement userManagement;

    public ChatListener(final UserManagement userManagement) {               
        this.userManagement = userManagement;
    }


    @EventHandler
    public void onChatEvent(AsyncPlayerChatEvent e) {
        String player = (String)e.getPlayer().getName();
        Player player1 = (Player)e.getPlayer();
        UUID id = (UUID)player1.getUniqueId();
        String message = (String)e.getMessage();
        if(userManagement.isAdmin(player1)) {
            Bukkit.broadcastMessage("§c§l[Admin] §r" + player + message);
        }else if(userManagement.isModerator(player1)) {
            Bukkit.broadcastMessage("§1[Moderator] §r" + player + message);

        // Ab hier selbst ändern ...

Und die Main muss natürlich auch angepasst werden, denn da erstellen wir das UserManagement einmalig und übergeben es bei den Konstuktoren der anderen beiden Klassen:
Java:
public class Main extends JavaPlugin{

    @Override
    public void onEnable() {
        UserManagement userManagement = new UserManagement();
        Bukkit.getServer().getPluginManager().registerEvents(new JoinListener(), this);
        Bukkit.getServer().getPluginManager().registerEvents(new QuitListener(), this);
        Bukkit.getServer().getPluginManager().registerEvents(new ChatListener(userManagement), this);
    
        new RankCommand(this, userManagement);
    }
}

Das wäre dann mein Vorschlag etwas ausführlicher.

Edit: Die markierten Zeilen waren verschoben - er hatte eine Leerzeile am Anfang im Editor drin, die er wohl entfernt hat und dadurch gab es die Verschiebung ... wurde behoben...
ok, danke, problem wäre dann glaube ich im UserManager, dass er ja bei player.getUniqueID() nicht nimmt, da es ja noch keine Variable mit dem Spieler player gibt...
 
kneitzel

kneitzel

Top Contributor
Der player wird ja als Parameter übergeben:

Java:
    public void addAdmin(final Player player) {
        admin.add(player.getUniqueId());
    }

Diese Methode erwartet also, dass ihm eine Player Instanz übergeben wird.

Das ist also vergleichbar mit Jemand schickt Dich ein Auto Kaufen und gibt Dir das Geld mit.
Du musst also selbst gar kein geld haben. Das Geld wird dir ja mitgegeben.

So ist es auch hier: Der UserManager muss keine Player kennen. Er nimmt diese ja entgegen und arbeitet dann nur mit der Instanz, die er bekommen hat...
 
jonny_2k12

jonny_2k12

Mitglied
Der player wird ja als Parameter übergeben:

Java:
    public void addAdmin(final Player player) {
        admin.add(player.getUniqueId());
    }

Diese Methode erwartet also, dass ihm eine Player Instanz übergeben wird.

Das ist also vergleichbar mit Jemand schickt Dich ein Auto Kaufen und gibt Dir das Geld mit.
Du musst also selbst gar kein geld haben. Das Geld wird dir ja mitgegeben.

So ist es auch hier: Der UserManager muss keine Player kennen. Er nimmt diese ja entgegen und arbeitet dann nur mit der Instanz, die er bekommen hat...
Alles klar 😄 du merkst ich verstehe noch nicht so viel von java aber fürn 3. tag gehts ja😁
 
kneitzel

kneitzel

Top Contributor
Ja klar - du schlägst Dich super. Ich wünsche Dir da viel Spaß und Erfolg. Und bei Fragen weisst Du ja, wo du Dich hinwenden kannst... :)
 
kneitzel

kneitzel

Top Contributor
Nein, das ist falsch. Hast Du das u groß geschrieben? Dass muss klein geschrieben sein:
userManagement ist die Instanzvariable, die wir hinzugefügt haben.
UserManagement ist die Klasse, die wir erstellt haben.
 
kneitzel

kneitzel

Top Contributor
Hast Du die Zeile
Java:
UserManagement userManagement = new UserManagement();
vergessen?

Diese Zeile deklariert eine neue Variable userManagement vom Typ UserManagement und erzeugt eine neue Instanz von UserManagement und weist diese dann der Variablen zu.

Und ich hatte vergessen: die Klasse RankCommand erbt natürlich nicht mehr von ChatListener. Diese Anpassung hatte ich in dem Code nicht drin...
Java:
public class RankCommand implements CommandExecutor {
Also ohne das "extends ChatListener".
 
jonny_2k12

jonny_2k12

Mitglied
Hast Du die Zeile
Java:
UserManagement userManagement = new UserManagement();
vergessen?

Diese Zeile deklariert eine neue Variable userManagement vom Typ UserManagement und erzeugt eine neue Instanz von UserManagement und weist diese dann der Variablen zu.

Und ich hatte vergessen: die Klasse RankCommand erbt natürlich nicht mehr von ChatListener. Diese Anpassung hatte ich in dem Code nicht drin...
Java:
public class RankCommand implements CommandExecutor {
Also ohne das "extends ChatListener".
jo, danke, jetzt sollte es klappen
 
jonny_2k12

jonny_2k12

Mitglied
Hast Du die Zeile
Java:
UserManagement userManagement = new UserManagement();
vergessen?

Diese Zeile deklariert eine neue Variable userManagement vom Typ UserManagement und erzeugt eine neue Instanz von UserManagement und weist diese dann der Variablen zu.

Und ich hatte vergessen: die Klasse RankCommand erbt natürlich nicht mehr von ChatListener. Diese Anpassung hatte ich in dem Code nicht drin...
Java:
public class RankCommand implements CommandExecutor {
Also ohne das "extends ChatListener".
kennst du dich auch mit der Spigot api gut aus?
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
M Zugriff auf eine ArrayList in einer anderen Klasse Java Basics - Anfänger-Themen 4
D Bestimmte Werte von Objekten aus einer ArrayList in eine andere ArrayList kopieren Java Basics - Anfänger-Themen 14
J Objekte einer ArrayList in eine Tabelle einfügen Java Basics - Anfänger-Themen 2
I Sortiert eine HashMap nicht gleich wie eine ArrayList? Java Basics - Anfänger-Themen 1
L Iterieren durch eine ArrayList. Integer Array wird übergeben Java Basics - Anfänger-Themen 17
I Klassen Eine ArrayList<Long>, die sich automatisch sortiert Java Basics - Anfänger-Themen 20
L Collections Ist eine Arraylist hier das richtige? Java Basics - Anfänger-Themen 12
G Datentypen verschiedene Objekte in eine ArrayList, Generics Java Basics - Anfänger-Themen 2
K Variablen Ist es möglich eine ArrayList vom Typ Array zu erstellen? Java Basics - Anfänger-Themen 2
V Mehrere Dateien aus JFileChooser in eine ArrayList speichern Java Basics - Anfänger-Themen 2
A Wie bekomme ich zwei Strings in eine ArrayList Java Basics - Anfänger-Themen 4
L Eine ArrayList auf gleiche Inhalte prüfen Java Basics - Anfänger-Themen 10
S ArrayList in eine Txt-Datei speichern Java Basics - Anfänger-Themen 3
N XML-Datei in eine ArrayList Java Basics - Anfänger-Themen 6
J ArrayList<Point> in eine neue ArrayList<Point> kopieren Java Basics - Anfänger-Themen 8
G Die Objekte eine ArrayList sortieren Java Basics - Anfänger-Themen 50
G Eine ArrayList effizient sortieren Java Basics - Anfänger-Themen 5
M Allgemein Array in eine ArrayList überführen? Java Basics - Anfänger-Themen 7
newcomerJava Nach doppelter Zahl eine Ausgabe Java Basics - Anfänger-Themen 10
S Eine Liste kopieren Java Basics - Anfänger-Themen 13
A eine neue normale String-Array von einer String-Array, die in for schleife ist, schaffen Java Basics - Anfänger-Themen 3
java3690 eine liste sortieren Java Basics - Anfänger-Themen 12
N Input/Output Eine Frage über system.out.println. Java Basics - Anfänger-Themen 10
K Übergabe des Wertes einer Variable aus main() in eine Klassenmethode Java Basics - Anfänger-Themen 8
J Wie kann ich hier eine While schleife einbauen? Java Basics - Anfänger-Themen 3
P Was genau bringt mir es ein Array in eine Liste zu bringen Java Basics - Anfänger-Themen 3
J Eine Frage zur Schreibweise == ? : Java Basics - Anfänger-Themen 3
I Wo am besten eine String Konstante initialisieren? Java Basics - Anfänger-Themen 5
N Wie teste ich eine geworfene Exception? Java Basics - Anfänger-Themen 8
B Eine Methode erstellen Java Basics - Anfänger-Themen 3
L Konstruktor für eine Map-Datei/Map-Datei einlesen Java Basics - Anfänger-Themen 5
Y Methoden Wie kann ich eine if-Abfrage bei Setters bauen? Java Basics - Anfänger-Themen 6
KogoroMori21 Mit einer Schleife eine Treppe zeichnen Java Basics - Anfänger-Themen 29
J Eine Position im String durch einen Integer - Wert teilen Java Basics - Anfänger-Themen 5
M Verständnisfrage zu eine Online Aufgabe Java Basics - Anfänger-Themen 7
N Wie kann ich eine meine Variable Final machen? Java Basics - Anfänger-Themen 1
H Datentypen Was für eine Format verbirgt sich hinter dem Integer-Wert 053? Java Basics - Anfänger-Themen 2
C Methoden Java ist auch eine Insel Listing 2.40 Recursion Java Basics - Anfänger-Themen 7
F Eine Zahl mit Arrays vergleichen Java Basics - Anfänger-Themen 7
P Eine komplizierte Sache Java Basics - Anfänger-Themen 1
B Nur eine bestimmte Anzahl an Arrays ausgeben Java Basics - Anfänger-Themen 9
T Wie eine Komposition garantieren Java Basics - Anfänger-Themen 5
M Wie analysiert JSON eine toString-Ausgabe ? Java Basics - Anfänger-Themen 1
V Array auf eine Zahl durchsuchen Java Basics - Anfänger-Themen 15
bensdt Klasse erweitern um eine Methode "void decrement()" Java Basics - Anfänger-Themen 5
S Datenbank Tabelle eine Zeile an einer bestimmten Stelle einfügen Java Basics - Anfänger-Themen 2
X Nach einem Bruch testen ob es eine ganze Zahl ist Java Basics - Anfänger-Themen 6
B Von Array nur eine bestimmte Anzahl bekommen Java Basics - Anfänger-Themen 3
C "HelloWorld" - Dateien erstellt, aber ist es eine class-Datei? Java Basics - Anfänger-Themen 2
L Muss eine HttpURLConnection geschlossen werden? Java Basics - Anfänger-Themen 2
D OOP- Eine Klasse in mehrere Klassen aufteilen Java Basics - Anfänger-Themen 7
V Erste Schritte Eine Array-Variable mit Benutzereingaben befüllen Java Basics - Anfänger-Themen 3
N Eine Zahl solange teilen bis es nicht möglich ist und die Ergebnisse ausgeben Java Basics - Anfänger-Themen 23
O Ziehen im Array um einen Schritt in eine einzige beliebige Richtung Java Basics - Anfänger-Themen 5
E Kreis soll eine Raupe darstellen Java Basics - Anfänger-Themen 37
P Alles eine Typfrage Java Basics - Anfänger-Themen 7
B Eine ganze Zahl zerlegen. Java Basics - Anfänger-Themen 4
D Guten Tag mache gerade eine Umschulung zum FiSi war leider jetzt 2 Wochen Krank und wir hatten Prozendurale Programmierung. Java Basics - Anfänger-Themen 3
L Iterieren über eine CSV Datei Java Basics - Anfänger-Themen 7
W Zwei Programme sollen auf eine Klasse zugreifen Java Basics - Anfänger-Themen 18
V Wie wiederhole ich eine for-Schleife beim erreichen eines Wertes Java Basics - Anfänger-Themen 9
A Kann man eine Methode als Variable speichern und danach noch verändern? Java Basics - Anfänger-Themen 6
L Klassen Wie “installiert” man eine Klasse? Java Basics - Anfänger-Themen 2
E Datentypen Einfügen von Objekten in eine Map Java Basics - Anfänger-Themen 2
X Wo steht, ob eine Datei big, little oder bi-endian codiert ist? Java Basics - Anfänger-Themen 3
N Bewegtes Objekt soll sich um eine Parallele bewegen Java Basics - Anfänger-Themen 0
O Iterator für eine geordnete Menge Java Basics - Anfänger-Themen 134
J Lesen und schreiben in eine Datei(Anfänger- Dateizugriffe) Java Basics - Anfänger-Themen 3
M Wie lang eine Funktion/Methode? Java Basics - Anfänger-Themen 51
R Warnung, wenn eine Variable eingegeben wird Java Basics - Anfänger-Themen 6
D Richtig Vorbereiten auf eine Programmier Klausur Studium. Java Basics - Anfänger-Themen 7
A Eine Krone in der Textausgabe - Mein Code Java Basics - Anfänger-Themen 11
scratchy1 Kann man in einer Methode eine andere aufrufen? Java Basics - Anfänger-Themen 2
B CSS Klassen in eine Liste schreiben Java Basics - Anfänger-Themen 4
L Klassen Kann eine Unterklasse einer abstrakten Klasse ein Interface implementieren? Java Basics - Anfänger-Themen 2
mor16Euro Erste Schritte Wie nennt man eine While oder For schleife die nie Ausgeführt wird Java Basics - Anfänger-Themen 5
N Best Practice Rückgabe eines Terminal Befehls in eine Variable speichern Java Basics - Anfänger-Themen 27
scratchy1 Wie deklariert man eine Methode, die ein Objekt zurückgeben soll? Java Basics - Anfänger-Themen 22
B Wie kann man an ein JPanel eine Scrollbar anheften? Java Basics - Anfänger-Themen 8
H Eine befühlte Klasse weiter geben an Controller Java Basics - Anfänger-Themen 12
K Durch eine Text eingabe etwas Ausführen Java Basics - Anfänger-Themen 5
A Überprüfen, ober eine Zahl Ziffer enthält Java Basics - Anfänger-Themen 12
O Bedingter Operator eine Frage! Java Basics - Anfänger-Themen 10
C Methoden Über eine einfach verkettete Liste Java Basics - Anfänger-Themen 8
J Eine Liste von Listen erstellen Java Basics - Anfänger-Themen 11
A Wie gebe ich bei android eine string im programm aus? Java Basics - Anfänger-Themen 4
R Auf eine Stelle [i] des Arrays zugreifen Java Basics - Anfänger-Themen 7
J Button klick bei eine bestimmten Zahl Java Basics - Anfänger-Themen 8
J Algorithmus für eine Reihe implementieren Java Basics - Anfänger-Themen 2
F Input/Output die eine txt-Datei funktioniert die andere nicht Java Basics - Anfänger-Themen 8
V Compiler-Fehler Warum eine IndexOutOfBoundsException? Java Basics - Anfänger-Themen 3
S Daten aus zwei Verschiedenen Tabellen in eine ArrayListe Java Basics - Anfänger-Themen 4
B Kann man eine Tabelle in Eclipse erzeugen und damit arbeiten? Java Basics - Anfänger-Themen 8
I Wie benutzt man eine Klasse? Java Basics - Anfänger-Themen 13
D Wie kann man eine solche Fehlermeldung abfangen Java Basics - Anfänger-Themen 4
M Wie kann ich eine Eingabe im Java-Swing übergeben? Java Basics - Anfänger-Themen 1
H OOP eine Zahl immer weiter durch 2 teilen Java Basics - Anfänger-Themen 15
F Aus geraden eine Kurve machen Java Basics - Anfänger-Themen 6
Queiser Datentypen 2 generische Datentypen für eine Schnittstelle Java Basics - Anfänger-Themen 1
G Arraynamen über eine Schleife ansprechen Java Basics - Anfänger-Themen 4

Ähnliche Java Themen

Anzeige

Neue Themen


Oben