Hilfe beim kleinen Wetter Plugin. | Anfänger

TutMeistensNix · Apr 24th 2019

So, nachdem ich seit Oktober 2016 die Flinte ins Korn geworfen habe versuche ich es heute noch mal.
Als erstes möchte ich anmerken das ich keinen Plan habe was ich hier eigentlich mache, deswegen versucht bei einer Antwort so einfach wie möglich zu schreiben.

Ich möchte ein kleines Plugin machen, nix besonderes, ist ja nur zum üben.
Und zwar soll ein "normaler" Spieler, also kein Admin, per einfachen Chatbefehl /wetter das Wetter auf clear stellen können,
Grund dafür ist das es manchmal so neblig ist das man selbst im Haus nix mehr sieht und ein Admin nicht immer da ist.

Dies habe ich bis jetzt:

Java

package de.tutmeistensnix.java;


import net.risingworld.api.Plugin;
import net.risingworld.api.events.EventMethod;
import net.risingworld.api.events.Listener;
import net.risingworld.api.objects.Player;
import net.risingworld.api.events.player.PlayerCommandEvent;
import net.risingworld.api.events.world.WeatherChangeEvent;


public class Wetter extends Plugin implements Listener{


	@Override
	public void onDisable() {
		// TODO Auto-generated method stub
	}


	@Override
	public void onEnable() {
		registerEventListener(this);
	}
	@EventMethod
	public void onCommand(PlayerCommandEvent event){
		Player player = event.getPlayer();
		String command = event.getCommand();
		String[] cmd = command.split(" ");
		if(cmd[0].equals("/wetter")){
			player.sendTextMessage("[#FFFFFF]Das Wetter wird geändert");
		}
	}


}

Display More

Und ich denke mal ich muss HIER auf setNextWeather(Weather weather) zugreifen.
Aber wie mache ich das?

Danke für die Hilfe.

Gruß TutMeistensNix

Ludy · Apr 25th 2019

naja wie soll ich das sagen

aber das kann mann über die .permissions einstellen wer das darf und wer nicht

commands:
allow:
- setweather
deny:
- *

darum bin ich mir nicht sicher in wie weit sich das dann verträgt

lg Lüdy

TutMeistensNix · Apr 25th 2019

Permission ist ja schön und gut aber bringt mich beim Java ja nicht weiter.

Ich lasse alles beim alten.

red51 · Apr 25th 2019

Das WeatherChangeEvent ist hierfür leider nicht ganz geeignet, denn Events sind generell nur "Ereignisse", die aktuell stattfinden, unabhängig von deinem Plugin. D.h. dieses Event wird dann aufgerufen, wenn das Wetter geändert wird (entweder via Command, oder zufällig). Du kannst dann zwar das Event abfangen und hier ein neues Wetter festlegen, aber dann müsstest du solange warten, bis sich überhaupt mal das Wetter ändert (damit das Event entsprechend aufgerufen wird).

Wenn du "aktiv" ins Geschehen eingreifen möchtest, sind die Server und World Klassen von Relevanz. Diese bieten alle möglichen Funktionen, um irgendwas in der Welt oder an den Servereinstellungen zu ändern. Um zB das Wetter zu ändern, gibt es in der Server-Klasse die Funktion setWeather(). Um Zugriff auf den "Server" zu bekommen (wovon es nur 1 Instanz gibt), musst du in deinem Plugin die Funktion getServer() aufrufen (du kannst den Rückgabewert entweder irgendwo in einer Variable speichern und dann immer wieder verwenden, oder du rufst jedes Mal getServer() auf). Um also das Wetter zu ändern, könnte der Code so aussehen:

Java

Server server = getServer();
server.setWeather(Weather.Rain, false);

TutMeistensNix · Apr 26th 2019

Vielen Dank.

Genau das habe ich gesucht.

TutMeistensNix · May 13th 2019

Kann mal bitte jemand über meinen Code schauen?
Es läuft, gibt, so weit ich das sehe, keine Fehler aus.

Habe ich das so weit richtig gemacht oder würdet ihr sagen mach es lieber so oder so?
Mir geht es jetzt hier um die Sache mit dem Wetter.
Über Sinn oder Unsinn brauchen wir nicht sprechen, es geht nur um Übungsaufgaben für mich.

Java: TutMeistensNix.java

package de.tutmeistensnix.rw;


import net.risingworld.api.Plugin;
import net.risingworld.api.Server;
import net.risingworld.api.events.EventMethod;
import net.risingworld.api.events.Listener;
import net.risingworld.api.events.player.PlayerCommandEvent;
import net.risingworld.api.objects.Player;


public class TutMeistensNix extends Plugin implements Listener {
	@Override
	public void onEnable() {
		registerEventListener(this);
	}
	@Override
	public void onDisable() {
	}


	@EventMethod
	public void onCommand(PlayerCommandEvent event) {
		Server server	= getServer();
		Player player	= event.getPlayer();
		String command	= event.getCommand();
		String[] cmd	= command.split(" ");
		Wetter wetter			= new Wetter();
		Essen essen				= new Essen();
		Gesundheit gesundheit	= new Gesundheit();
		// ########## Wetter verändern ##########
		if(cmd[0].equals("/wetter")) {
			if(cmd.length >= 2) {
				String art = cmd[1];
				wetter.neuesWetter(server, player, art);
			} else {
				player.sendTextMessage("[#FFFF00]Du musst clear, regen, nebel oder sturm schreiben. \n[#FFFF00]Beispiel: /wetter clear");
			}
		} else if(cmd[0].equals("/essen")) { // ########## Hunger und Durst stillen ##########
			essen.wollenEssen(player);
		} else if(cmd[0].equals("/heilen")) { // ########## Wie ist meine Gesundheit ##########
			gesundheit.meineGesundheit(player);
		} else if(cmd[0].equals("/ungesund")) { // ########## Mach mich krank ##########
			if(cmd.length >= 2) {
				if(cmd[1].equals("knochenbruch")) {
					gesundheit.brechMirDieKnochen(player, true);
				} else if(cmd[1].equals("bluten")) {
					gesundheit.ichBlute(player, true);
				} else {
					int krank = Integer.parseInt(cmd[1]); // ########## String in Int umwandeln #########
					gesundheit.machMichKrank(player, krank);
				}
			}else {
				player.sendTextMessage("[#FFFF00]Du musst knochenbruch, bluten oder eine Zahl zwischen 1 - 100 angeben. \n[#FFFF00]Beispiel: /ungesund 50 oder /ungesund knochenbruch");
			}
		}
	}


}

Display More

Java: Wetter.java

package de.tutmeistensnix.rw;


import net.risingworld.api.Server;
import net.risingworld.api.objects.Player;
import net.risingworld.api.objects.Weather;


public class Wetter {
	public void neuesWetter(Server server, Player player, String art){
		switch(art){
		case "clear": {
			server.setWeather(Weather.Clear, true);
			server.broadcastTextMessage("[#FFFF00]Das Wetter wurde von " + player.getName() + " geändert.");
			break;
		}
		case "regen": {
			server.setWeather(Weather.Rain, true);
			server.broadcastTextMessage("[#FFFF00]Das Wetter wurde von " + player.getName() + " geändert.");
			break;
		}
		case "nebel": {
			server.setWeather(Weather.Fog, true);
			server.broadcastTextMessage("[#FFFF00]Das Wetter wurde von " + player.getName() + " geändert.");
			break;
		}
		case "sturm": {
			server.setWeather(Weather.Storm, true);
			server.broadcastTextMessage("[#FFFF00]Das Wetter wurde von " + player.getName() + " geändert.");
			break;
		}
		default: {
			player.sendTextMessage("[#FFFF00]Du musst clear, regen, nebel oder sturm schreiben. \n[#FFFF00]Beispiel: /wetter clear");
			break;
		}
		}
	}
}

Display More

Vielen Dank fürs anschauen.

TutMeistensNix

PatrickOtt · May 13th 2019

Ich würde sagen, dass es als Übung so passt, aber wenn du mal größere Plugins schreibst, würde ich den Aufbau etwas ändern. z.B. Eigene Classe für Events; Server, Wetter, Essen und Gesundheit zu einer globalen Variable machen und in der onEnable initialisieren, damit sie überall erreichbar sind und es 1. Weniger Schreibarbeit und 2. weniger Null-Fehler kommen.

Aber für diese Übung sollte es reichen.

Wichtig: Prüfe in der Praxis immer alles durch, damit du Leichtsinnsfehler findest.

TutMeistensNix · May 13th 2019

Danke für deine Antwort, ich habe ja schon Wetter, Essen und Gesundheit in je eine extra Klasse gepackt.
Um es übersichtlicher, für mich, zu machen.

Vielmehr kann ich doch nicht auseinander nehmen da die Sachen doch in die onCommand() müssen, oder habe ich da einfach ein Denkfehler?
zb meine Gesundheit Klasse.

Java: Gesundheit.java

package de.tutmeistensnix.rw;


import net.risingworld.api.objects.Player;


public class Gesundheit {
	public void meineGesundheit(Player player) {
		int meineWerte	= player.getHealth();
		boolean knochen	= player.hasBrokenBones();
		boolean bluten	= player.isBleeding();
		if(meineWerte < 90) {
			player.setHealth(100);
			player.sendTextMessage("[#FFFF00]Du wurdest geheilt");
		} else if(meineWerte > 90 && meineWerte < 99) {
			player.sendTextMessage("[#FFFF00]So krank bist du noch nicht, eine Zitrone sollte da reichen!");
		} else {
			player.sendTextMessage("[#FFFF00]Du bist volkommen gesund.");
		}
		if(knochen) {
			player.setBrokenBones(false);
			player.sendTextMessage("[#FFFF00]Dein Knochenbruch wurde geheilt, ich hoffe die Schmerzen waren erträglich.");
		} else {
			player.sendTextMessage("[#FFFF00]Zum Glück hast du dir nix gebrochen.");
		}
		if(bluten) {
			player.setBleeding(false);
			player.sendTextMessage("[#FFFF00]Deine Blutung wurde gestillt.");
		} else {
			player.sendTextMessage("[#FFFF00]Wie es aussieht hast du keine Blutung.");
		}
	}
	public void machMichKrank(Player player, int krank) {
		if(krank >= 1 && krank <= 100) {
			player.setHealth(krank);
			int binKrank = player.getHealth();
			player.sendTextMessage("[#FFFF00]Ich glaube dir geht es gar nicht gut, du hast ja nur noch " + binKrank + " % Gesundheit.");
		} else {
			player.sendTextMessage("[#FFFF00]Bitte nur eine Zahl zwischen 1 - 100 angeben.");
		}
	}
	public void brechMirDieKnochen(Player player, boolean knochenBruch) {
		player.setBrokenBones(knochenBruch);
		player.sendTextMessage("[#FFFF00]KNACK! Aua, ich glaube du hast dir gerade einen Knochen gebrochen!");
	}
	public void ichBlute(Player player, Boolean bluten) {
		player.setBleeding(true);
		player.sendTextMessage("[#FFFF00]Ich glaube du hast dich geschnitten, du blutest ja!");		
	}


}

Display More

PatrickOtt · May 13th 2019

Mit deiner Gesundheit ist alles ok.
Ich meine Folgendes:

Java: TutMeistensNix.java

public class TutMeistensNix extends Plugin /*implements Listener*/ {
	Wetter wetter;
	Server server;
	Essen essen;
	Gesundheit gesundheit;
        // Kann nun überall verwendet werden
           //(Beispiel: Von Fremde class: "plugin.wetter" oder "plugin.essen");


          //Beispiel dieser class: wetter.neuesWetter(server, player, art); ALSO GANZ NORMAL






	@Override
	public void onEnable() {
		System.out.println("[TutMeistensNix] Enabled"); //<<< Damit erkennst du, ob das Plugin gestartet wird! (Konsole)
		//-------------------------------------------
		server = getServer();
		gesundheit = new Gesundheit();  // Initalisieren der Variablen!
		wetter = new Wetter();         // Muss nicht mehr neu initalisiert werden!
		essen = new Essen();
		//-------------------------------------------
		registerEventListener(new pluginListener(this); //<<< So kannst du die Events in eine andere Class tun;
	}


	@Override
	public void onDisable() {
		System.out.println("[TutMeistensNix] Desabled");
	}




	//@EventMethod


	/*public void onCommand(PlayerCommandEvent event) {  // >>>>>Das würde ich alles in pluginListener tun <<<<
		Server server	= getServer(); <<< Kann auch in die onEnabled! Muss nicht ständig neu initalisiert werden!
		Player player	= event.getPlayer();
		String command	= event.getCommand();
		String[] cmd	= command.split(" ");
		//Wetter wetter			= new Wetter();  <<<<< Nicht hier!
		//Essen essen				= new Essen();  <<<<< Nicht hier!
		//Gesundheit gesundheit	= new Gesundheit();  <<<<< Nicht hier!


		// ########## Wetter verändern ##########


		if(cmd[0].equals("/wetter")) {
			if(cmd.length >= 2) {
			String art = cmd[1];
				wetter.neuesWetter(server, player, art);
			} else {
				player.sendTextMessage("[#FFFF00]Du musst clear, regen, nebel oder sturm schreiben. \n[#FFFF00]Beispiel: /wetter clear");
			}
		} else if(cmd[0].equals("/essen")) { // ########## Hunger und Durst stillen ##########
			essen.wollenEssen(player);
		} else if(cmd[0].equals("/heilen")) { // ########## Wie ist meine Gesundheit ##########
			gesundheit.meineGesundheit(player);
		} else if(cmd[0].equals("/ungesund")) { // ########## Mach mich krank ##########
			if(cmd.length >= 2) {
				if(cmd[1].equals("knochenbruch")) {
					gesundheit.brechMirDieKnochen(player, true);
				} else if(cmd[1].equals("bluten")) {
					gesundheit.ichBlute(player, true);
				} else {
					int krank = Integer.parseInt(cmd[1]); // ########## String in Int umwandeln #########
					gesundheit.machMichKrank(player, krank);


				}
			}else {
				player.sendTextMessage("[#FFFF00]Du musst knochenbruch, bluten oder eine Zahl zwischen 1 - 100 angeben. \n[#FFFF00]Beispiel: /ungesund 50 oder /ungesund knochenbruch");


			}


		}


	}*/




}

Display More

Das Event würde dann hier laufen:

Java: pluginListener.java

public class pluginListener implements Listener{

	private final TutMeistensNix plugin;

	public pluginListener(TutMeistensNix plugin){
		this.plugin = plugin;
	}

   @EventMethod


	public void onCommand(PlayerCommandEvent event) {  
		//Server server	= getServer(); <<<<< Ist jetzt plugin.server
		Player player	= event.getPlayer();
		String command	= event.getCommand();
		String[] cmd	= command.split(" ");
		//Wetter wetter			= new Wetter();   <<< Das kann z.B. jetzt mit plugin.wetter aufgerufen werden
		//Essen essen				= new Essen();  
		//Gesundheit gesundheit	= new Gesundheit();  


		// ########## Wetter verändern ##########


		if(cmd[0].equals("/wetter")) {
			if(cmd.length >= 2) {
			String art = cmd[1];
				plugin.wetter.neuesWetter(server, player, art);
			} else {
				player.sendTextMessage("[#FFFF00]Du musst clear, regen, nebel oder sturm schreiben. \n[#FFFF00]Beispiel: /wetter clear");
			}
		} else if(cmd[0].equals("/essen")) { // ########## Hunger und Durst stillen ##########
/*Beispiel >>>*/plugin.essen.wollenEssen(player);
		} else if(cmd[0].equals("/heilen")) { // ########## Wie ist meine Gesundheit ##########
			plugin.gesundheit.meineGesundheit(player);
		} else if(cmd[0].equals("/ungesund")) { // ########## Mach mich krank ##########
			if(cmd.length >= 2) {
				if(cmd[1].equals("knochenbruch")) {
					plugin.gesundheit.brechMirDieKnochen(player, true);
				} else if(cmd[1].equals("bluten")) {
					plugin.gesundheit.ichBlute(player, true);
				} else {
					int krank = Integer.parseInt(cmd[1]); // ########## String in Int umwandeln #########
					plugin.gesundheit.machMichKrank(player, krank);


				}
			}else {
				player.sendTextMessage("[#FFFF00]Du musst knochenbruch, bluten oder eine Zahl zwischen 1 - 100 angeben. \n[#FFFF00]Beispiel: /ungesund 50 oder /ungesund knochenbruch");


			}


		}


	}
}

Display More

TutMeistensNix · May 13th 2019

Vielen lieben Dank für dein Beispiel.

Werde ich mir morgen mal in aller Ruhe anschauen.

Bin ja schon mal beruhigt das mit meiner Gesundheit alles in Ordnung ist.

TutMeistensNix · May 15th 2019

Ich lerne ja noch, durch lesen oder Videos gucken.
Nun hätte dich da mal eine Frage zu deinem Code.

Java

public class TutMeistensNix extends Plugin /*implements Listener*/ {
	Wetter wetter;
	Server server;
	Essen essen;
	Gesundheit gesundheit;
        // Kann nun überall verwendet werden
           //(Beispiel: Von Fremde class: "plugin.wetter" oder "plugin.essen");


          //Beispiel dieser class: wetter.neuesWetter(server, player, art); ALSO GANZ NORMAL






	@Override
	public void onEnable() {
		System.out.println("[TutMeistensNix] Enabled"); //<<< Damit erkennst du, ob das Plugin gestartet wird! (Konsole)
		//-------------------------------------------
		server = getServer();
		gesundheit = new Gesundheit();  // Initalisieren der Variablen!
		wetter = new Wetter();         // Muss nicht mehr neu initalisiert werden!
		essen = new Essen();
		//-------------------------------------------
		registerEventListener(new pluginListener(this); //<<< So kannst du die Events in eine andere Class tun;
	}


.
.
.

Display More

Lerne ich nicht die ganze Zeit das bei Java immer der Typ einer Variable angegeben werden muss damit Java weiß was da auf ihn zu kommt?

Muss man das hier nicht machen?
Ich frage nur um es zu verstehen, habe ja keine Ahnung.

Devidian · May 15th 2019

Hat er doch, das hier ist die deklaration:

Code

// TYP variabelName


Wetter wetter;
Server server;
Essen essen;
Gesundheit gesundheit;

Und dann hier die Zuweisungen:

Code

server = getServer();
gesundheit = new Gesundheit();  // Initalisieren der Variablen!
wetter = new Wetter();         // Muss nicht mehr neu initalisiert werden!
essen = new Essen();

PatrickOtt · May 15th 2019

Quote from Devidian

Hat er doch, das hier ist die deklaration:

Code

// TYP variabelName


Wetter wetter;
Server server;
Essen essen;
Gesundheit gesundheit;

Und dann hier die Zuweisungen:

Code

server = getServer();
gesundheit = new Gesundheit();  // Initalisieren der Variablen!
wetter = new Wetter();         // Muss nicht mehr neu initalisiert werden!
essen = new Essen();

TutMeistensNix · May 15th 2019

Ich muss doch immer String, int, etc davor schreiben.

Devidian · May 15th 2019

String Int etc. sind Basistypen. Wenn du selber Klassen definierst also

Java

public class Wetter {
...
}

dann kannst du ja Wetter meinWetter = new Wetter(); machen. hier ist der Typ dann Wetter weil du ein Objekt der Klasse Wetter erzeugst indem du den Konstruktor aufrufst.

PatrickOtt · May 15th 2019

"Player" ist ja auch vom Typ "Object"!

Also gibt es auch in Rising World eine Klasse die "Player" heißt!

Java

public class Player {




}

TutMeistensNix · May 15th 2019

Ja, jetzt sehe ich es.
Hatte wohl Tomaten auf den Augen.

TutMeistensNix · Jun 21st 2019

Kann es sein das ein Plugin nicht mehr richtig funzt wenn der Server länger wie 24 Std am Stück läuft?

Devidian · Jun 21st 2019

je länger der Server läuft desto mehr plugins verursachen Fehler, Ist mir aufgefallen, daher starte ich meinen 2 mal pro tag neu aktuell. hängt auch davon ab wie viele Spieler so aktiv sind über tag. Ggf. muss man auch öfter neu starten.

Ich glaube das ist so eine generelle Krankheit von Sandbox games. 7 Days to Die zum Beispiel wird auch empfohlen täglich min ein mal neu zu starten - und ja ich habs selbst getestet ohne neustart spackt der auch rum. Und da gilt auch mehr spieler => öfter neustart.

TutMeistensNix · Jun 21st 2019

Ich starte auch jeden Tag neu, manuell.
Immer dann wenn ich dran denke, wie gerade eben.

Danke für deine Antwort.

Participate now!