Bukkit Code Fragen

MrPyro13 · 3 August 2014

Hallo,

ich habe gerade angefangen Bukkit-Programmierung zu erlernen und habe nun folgenden Code auf einem YouTube Tutorial abgepinselt und habe dazu zwei kleine Fragen:

Hier der Code:

Code:

package de.mrpyro13.tutorial;

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

public class HealCommand implements CommandExecutor {

	@Override
	public boolean onCommand(CommandSender cs, Command cmd, String label, String[] args) {
		Player p = null;
		if (args.length == 0) {
			if (cs instanceof Player) {
				p = (Player) cs;
			} else {
				cs.sendMessage("$cDu kannst dich als Konsole nicht heilen");
				return true;
			}
		} else {
			String name = args[0];
			p = Bukkit.getPlayer(name);
		}
		if (p==null) {
			cs.sendMessage("§cDer Spieler ist nicht online!");
			return true;
		}
		
		p.setHealth(20);
		if (p != cs) {
		cs.sendMessage("§aDu hastn " + p.getName() + " geheilt" );
		p.sendMessage("§aDu wurdest von " + cs.getName() + " geheilt" );
		} else {
			p.sendMessage("§aDu hast dich geheilt");
		}
		return true;
	}
	

}

Frage 1: Muss das (Player) da stehen oder reicht p = cs; ?

p = (Player) cs;

Frage 2: Was sagt diese Zeile aus bzw ist diese notwendig?

String name = args[0];

FelixKlauke · 3 August 2014

1. Ja muss es, denn das Plugin kennt den cs ja nur als CommandSender.

2. Diese Zeile holt sich das nullte Argument, das den args beim befehl übergeben wurde.

z.B.

/heal Maja

Dann ist Maja das nullte Argument. Und ja, die muss da stehen, wenn du jemand anderen heilen möchtest.
Ich würde aber anstatt else noch einmal else if abfragen, ob die Argumentenlänge 1 ist.

Crafter213 · 3 August 2014

Code:

CommandSender cs

cs ist also nicht vom Typ Spieler, sondern vom Typ "CommandSender".
Ein CommandSender kann z.B. auch die Konsole sein, wenn der Command dort eingegeben wurde.
Da die Konsole aber z.B. keine Position in der Welt hat und auch nicht teleportiert werden kann, ist dies so auch nicht mit CommandSendern möglich. Auch kann die Konsole nicht sterben und hat keine Lebensanzeige

Code:

if (cs instanceof Player){
  p = (Player) cs;
}

Das prüft, ob der Command von einem Spieler gestartet wurde.
(Player) cs castet dann; aus dem "CommandSender" wird ein "Player"-Objekt. So kannst du nun auch auf den Spieler an sich zugreifen
und ihn teleportieren oder z.B. auch das Inventar verändern. Oder ihn halt heilen.

Code:

String name = args[0];

Direkt notwendig ists nicht, aber evtl. schöner und übersichtlicher:
Ein String "name" wird erstellt; den zu benutzen ist übersichtlicher als immer args[0] zu schreiben.

args[0] ist das erste Argument, das dem Command übergeben wurde.
Argumente sind alle Eingaben nach dem Command und durch leerzeichen getrennt:
/command [arg0] [arg1] [arg2] [...]
Es können quasi unbegrenzt viele Argumente übergeben werden, auch kein einziges.
Prüfe immer mit args.length, wie viele Argumente da sind.

MrPyro13 · 3 August 2014

Ich denke, dass ich es verstanden habe:

Code:

else {
			String name = args[0];
			p = Bukkit.getPlayer(name);
}

Der String NAME (zB MrPyro13) wird als Argument 0 gesetzt. Dann wird der Variable p ,der Spieler von dem Argument 0, zugeschrieben.
Wobei der Spieler von dem String NAME herausgefunden wurde...

Ich habe den String name nicht den (name) zugeordnet *facepalm*

best_ronsuper · 3 August 2014

MrPyro13 schrieb:
Hallo,

cs.sendMessage("$cDu kannst dich als Konsole nicht ...

Pass auf dass du überall § benutz

oder du lässt dir die & übersetzten, was aber eig. Sinnlos ist da du direkt § benutzen kannst

MrPyro13 · 3 August 2014

@best_ronsuper
Ja, danke ;D

Noch mal eine Frage:
Wie erstelle ich ein Subcommand in einer extra Klasse? Also zB /meinplugin help oder /meinplugin heal. Ist das möglich?
Ich habe mir vorgenommen, für jeden Befehl und Subbefehl eine eigene Klasse zu erstellen.

Yellowphoenix18 · 3 August 2014

Ja,das ist eigentlich nicht schwer.
Hier einmal ein Code:

if (cmd.getName().equalsIgnoreCase("gm")) {
if (p.hasPermission("ucx.gamemode")) {

if (args.length == 0) {

p.sendMessage("Welcher Gamemode??");
}
if (args.length == 1) {
if (args[0].equalsIgnoreCase("1")) {

p.performCommand("gamemode 1");
p.sendMessage("Dein GameMode wurde geändert");
}
if (args[0].equalsIgnoreCase("0")) {

p.performCommand("gamemode 0");
p.sendMessage("Dein GameMode wurde geändert");
}

}
}
}

So,hier siehst du nun,als erstes den HauptCommand,danach die Permission und dannach if (args.length == 0) {,hier wird überprüft,wie viele Argumente der Befehl hat,in diesem Fall 1,daher fehlt aber noch ein Argument und es wird eine Frage gestellt.Später kommt Die Argumenten-Länge 2,dort wird dann immer nach dem zweiten Argument geguckt beim ersten Punkt "1",somit muss der Befehl /gm 1 eingegeben werden und der Teil wird ausgeführt.Um diesen Befehl nun zu setzen musst du nur den HauptCommand(Hier "gm") in die plugin.yml eintragen.

Ich hoffe ich konnte helfen,mfg YellowPhoenix18

MrPyro13 · 3 August 2014

Ich weiß nicht so ganz...
Gehen wir von einem SurvivalGames Plugin etc. aus. Es gibt /sg start, /sg stop und /sg vote (als Beispiel)
Nun möchte ich den /sg start Befehl in einer eigenen Klasse den /sg stop Befehl in einer eigenen Klasse und den /sg vote Befehl in einer eigenen Klasse.

Crafter213 · 3 August 2014

Du hast natürlich die Möglichkeit, ein Package für den Command und viele eigene Klassen zu nutzen... das würde ch aber nciht machen.
Eine Klasse pro "echtem" Command ist gaanz gut; Subcommands gehen da recht gut ebenfalls rein.
Ich habs so gelöst (Auszug):

Code:

{
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        
        if(!sender.hasPermission(cmd.getPermission())) {
            sender.sendMessage("No Permission");
            return true;
        }
        if(args.length < 1) {
            sender.sendMessage("Usage:");
            sender.sendMessage("  /haupt - Show help");
            sender.sendMessage("  /haupt [Command; Params] - Commands:");
            sender.sendMessage("    sub1 - Bla");
            sender.sendMessage("    sub2 <weitere Params> - Blub");
            return true;
        }
        
        
        if(args[0].equalsIgnoreCase("sub1")) {
            haupt_sub1(sender);
            return true;
        }
        if(args[0].equalsIgnoreCase("sub2")) {
            Castle_sub2(sender, args);
            return true;
        }        
        
        sender.sendMessage("Unknown Subcommand!");
        return true;
    }
    
    void haupt_sub1(CommandSender sender) {
        if(!sender.hasPermission("PluginName.haupt.sub1")) {
            sender.sendMessage("No permission");
            return;
        }
        [...]
        return;
    }
    void haupt_sub2(CommandSender sender, String[] args) {
        if(!sender.hasPermission("Pluginname.haupt.sub2")) {
            sender.sendMessage("No permission");
            return;
        }
        if(args.length < [Anzahl der benötigten Parameter]) {
            sender.sendMessage("Usage: [...]");
            return;
        }
        [...]
        return;
    }

}

MrPyro13 · 3 August 2014

Ok, das mit den Subcommands habe ich so halbwegs gecheckt

Doch was ich nicht verstehen will, ist die Übernahme einer 2. Person bei zB /heal anderer_User .

Das geht doch mit:

Code:

Player p = null; <--- Definiert die Variable leer
p = (Player) cs <--- Legt fest, dass die Variable p ein Spieler sein muss (als CommandSender)
String name = args[0]; <--- Definiert das Argument 0 als Zeichenkette/String NAME
p = Bukkit.getPlayer(name); <--- ?????

Und hier mein Code dazu (Schicke einfach mal den ganzen Command-Code):

Code:

package de.mrpyro13.myplugin;

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

public class MyPlugin_Command implements CommandExecutor {

	@Override
	public boolean onCommand(CommandSender cs, Command cmd, String label, String[] args) {
		if (args.length == 0) {
			cs.sendMessage("§6Willkommen " + cs.getName() + "§6, bei der myPlugin-Information");
			cs.sendMessage("§aName    : myPlugin");
			cs.sendMessage("§aAutor   : MrPyro13");
			cs.sendMessage("§aVersion : v0.1");
			return true;
		} else if (args.length == 1) {
			if (args[0].equalsIgnoreCase("help")) {
				cs.sendMessage("§6myPlugin Befehle");
				cs.sendMessage("§a/myplugin     " + " §6- " + "§aInformationen zum Plugin");
				cs.sendMessage("§a/myplugin help" + " §6- " + "§aBefehlsliste des Plugins");
				cs.sendMessage("§a/myplugin message [Spieler]" + " §6- " + "§aSende eine Test-Nachricht");
				return true;
			} else if (args[0].equalsIgnoreCase("heal")) {
				if (args.length == 1) {
					if (cs instanceof ConsoleCommandSender) {
						cs.sendMessage("§cFehler: Dieser Befehl ist nur fuer Spieler!");
						return true;
					}
					Player p = null;
					p = (Player) cs;
					p.setHealth(20.0);
					p.sendMessage("§aDu hast dich geheilt!");
					return true;
				}
				if (args.length == 2) {
					Player p = null;
					p = (Player) cs;
					String name = args[1];
					p = Bukkit.getPlayer(name);
					if (p != cs) {
						p.setHealth(20.0);
						cs.sendMessage("§aDu hastn " + p.getName() + " geheilt" );
						p.sendMessage("§aDu wurdest von " + cs.getName() + " geheilt" );
						return true;
						} else {
							p.setHealth(20.0);
							p.sendMessage("§aDu hast dich geheilt");
							return true;
						}
																		
				}
																										
			}
		} 
		
		cs.sendMessage("§cFehler! Nutze /myplugin help um alle Befehle zu sehen");
		return true;
	}

}

Das Selbstheilung funktioniert, jedoch die Fremdheilung geht nicht. Da kommt die allerletzte Nachricht:

cs.sendMessage("§cFehler! Nutze /myplugin help um alle Befehle zu sehen");

Crafter213 · 3 August 2014

Code:

p = (Player) cs <--- Legt fest, dass die Variable p ein Spieler sein muss (als CommandSender)

Nein.
Das gibt die dir den Spieler zu cs
Bsp: MCPlayer nutzt den Command, dann ist cs= MCPlayer_ABER_vomTypCommandSender
(Player) cs -> das gibt dir wieder MCPlayer; also den, der den Command aufruft

Code:

String name = args[0]; <--- Definiert das Argument 0 als Zeichenkette/String NAME

Nein; Argument Nr. 0 wird hier nicht definiert, sondern aufgerufen.
"name" wird als String deklariert (Typ = String) und mit agrs[0] definiert; in "name" steht also das Gleiche wie in "agrs[0]"

Code:

p = Bukkit.getPlayer(name); <--- ?????

Das sollte so funktionieren.
Danach noch

Code:

if(p == null) {
    cs.sendMessage("Error getting Player " + name);
    return true;
}

MrPyro13 · 3 August 2014

So... Alles geht, außer dieser Abschnitt.

Code:

else if (args.length == 2) {
					Player p = null;
					String name = args[1];
					p = Bukkit.getPlayer(name);
					cs.sendMessage("Du hast");
					return true;
				} else {
					cs.sendMessage("§cFehler! Nutze /myplugin help um alle Befehle zu sehen");
					return true;

Crafter213 · 3 August 2014

p.setHealth(20);

MrPyro13 · 3 August 2014

Dieser Abschnitt (oben) wird ja übersprungen...
Der nimmt dann nur noch diese Zeile und fertig:

Code:

cs.sendMessage("§cFehler! Nutze /myplugin help um alle Befehle zu sehen");
return true;

//EDIT: Kann geschlossen werden. Der Code ist mir zu unübersichtlich, ich starte das kleine "Projekt" morgen nochmal neu.

minecraft-server.eu Europas Größte Serverliste

Bukkit Code Fragen

MrPyro13

Workaholic

FelixKlauke

Erzengel

Crafter213

Braumeister

MrPyro13

Workaholic

best_ronsuper

Guest

MrPyro13

Workaholic

Yellowphoenix18

Vorarbeiter

MrPyro13

Workaholic

Crafter213

Braumeister

MrPyro13

Workaholic

Crafter213

Braumeister

MrPyro13

Workaholic

Crafter213

Braumeister

MrPyro13

Workaholic