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

jonny_2k12 · 11. Nov 2020

Hi, frage steht oben, im anhang screenshots.

Klasse 1:

Klasse 2:

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

Danke im vorraus
jonny_2k12

kneitzel · 12. Nov 2020

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 · 12. Nov 2020

kneitzel hat gesagt.:
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. 😀

jonny_2k12 · 12. Nov 2020

jonny_2k12 hat gesagt.:
Hi, danke, bin noch sehr neu in Java (3 Tage), deshalb sorry wenn noch nicht alles richtig ist. 😀

kneitzel hat gesagt.:
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?

kneitzel · 12. Nov 2020

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) {

jonny_2k12 hat gesagt.:
Hi, danke, bin noch sehr neu in Java (3 Tage), deshalb sorry wenn noch nicht alles richtig ist. 😀

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 · 12. Nov 2020

kneitzel hat gesagt.:
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 · 12. Nov 2020

EDIT: hab den Fehler gefunden 🙂

@kneitzel Weitere Frage 😀

Nun hab ich die ID des Players zur ArrayList hinzugefügt, aber das erhoffte ergebnis funktioniert noch nicht... 😀
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);
       
   
    }




}

kneitzel · 12. Nov 2020

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 · 12. Nov 2020

kneitzel hat gesagt.:
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:

[CODE lang="java" title="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);

}

}[/CODE]

[CODE lang="java" title="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;
}

}
[/CODE]

[CODE lang="java" title="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);
}
}[/CODE]

kneitzel · 12. Nov 2020

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):
[CODE lang="java" highlight="3,5,6,22-26"]
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 ...
[/CODE]

Und der ChatListener:
[CODE lang="java" highlight="3, 5-7, 16-21"]
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 ...
[/CODE]

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:
[CODE lang="java" highlight="5,8,10"]
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);
}
}[/CODE]

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 · 12. Nov 2020

kneitzel hat gesagt.:
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):
[CODE lang="java" highlight="3,5,6,22-26"]
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 ...
[/CODE]

Und der ChatListener:
[CODE lang="java" highlight="3, 5-7, 16-21"]
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 ...
[/CODE]

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:
[CODE lang="java" highlight="5,8,10"]
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);
}
}[/CODE]

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 · 12. Nov 2020

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 · 12. Nov 2020

kneitzel hat gesagt.:
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 · 12. Nov 2020

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... 🙂

jonny_2k12 · 12. Nov 2020

kneitzel hat gesagt.:
if(userManagement.isAdmin(player1)) {

hier schreibt er: Cannot make a static reference to the non-static method isAdmin(Player) from the type UserManagement

als quickfix: change isAdmin to static

ist das richtig? also soll ich vor isAdmin static hinzufügen?

kneitzel · 12. Nov 2020

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.

jonny_2k12 · 12. Nov 2020

kneitzel hat gesagt.:
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.

hatte den implement vergessen 🙈

jonny_2k12 · 12. Nov 2020

kneitzel hat gesagt.:
(new ChatListener(userManagement), this);

noch ein weiterer Fehler, in der Main: userManagement cannot be resolved to a variable

kneitzel hat gesagt.:
public RankCommand(final Main plugin, final UserManagement userManagement) {

und im Rank Command Implicit super constructor ChatListener() is undefined. Must explicitly invoke another constructor

kneitzel · 12. Nov 2020

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 · 12. Nov 2020

kneitzel hat gesagt.:
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 · 12. Nov 2020

kneitzel hat gesagt.:
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?

kneitzel · 12. Nov 2020

jonny_2k12 hat gesagt.:
kennst du dich auch mit der Spigot api gut aus?

Das sagt mir bisher nichts ...

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

Mitglied

kneitzel

Gast

Mitglied

Mitglied

kneitzel

Gast

Mitglied

Mitglied

​

kneitzel

Gast

Mitglied

kneitzel

Gast

Mitglied

kneitzel

Gast

Mitglied

kneitzel

Gast

Mitglied

kneitzel

Gast

Mitglied

Mitglied

kneitzel

Gast

Mitglied

Mitglied

kneitzel

Gast

Ähnliche Themen