Plugin /back Befehl aber auch ne Frage zu event.player:getPosition(); in Java?

TutMeistensNix · Oct 27th 2016

Hallo.

Unter Lua hatten/haben wir die Möglichkeit die aktuelle Position des Spielers mit event.player:getPosition(); auszulesen.
Nun gibt es aber nicht mehr getPosition() wie lautet es den nun?

Ich habe nun vor ein kleines Script zu schreiben welches den Todespunkt in der Datenbank speichert damit der Spieler via /back zurück kommt um seine Leiche leer zu räumen.
Habe soweit auch einiges fertig aber die MySQL Datenbank hält mich wieder auf, da diese nix speichert, aber das ist ein anderes Thema.

Kommt ein Spieler das erste mal auf den Server wird geprüft ob er schon mal da war, wenn nein dann wird ein Back Punkt erstellt anhand der getSpawnPosition() ist das ok?
Das müsste doch der Spawn Punkt sein, wenn ich das richtig verstehe?

Gerne könnt ihr ja mal über den unfertigen Code schauen, und bin für Tipps sehr dankbar.

Java

package net.risingworld.tmnplugin.BackSystem;


import java.sql.ResultSet;
import java.sql.SQLException;


import net.risingworld.api.Plugin;
import net.risingworld.api.database.WorldDatabase;
import net.risingworld.api.events.EventMethod;
import net.risingworld.api.events.Listener;
import net.risingworld.api.events.player.PlayerCommandEvent;
import net.risingworld.api.events.player.PlayerDeathEvent;
import net.risingworld.api.events.player.PlayerSpawnEvent;
import net.risingworld.api.objects.Player;
import net.risingworld.api.utils.Vector3f;


public class BackSystem extends Plugin implements Listener {
	@Override
	public void onEnable(){
		registerEventListener(this);
		System.out.println("Back Plugin wurde erfolgreich geladen!");
	}
	@Override
	public void onDisable(){
	}
	@EventMethod
	public void onPlayerSpawn(PlayerSpawnEvent event){
		Player player = event.getPlayer();
		try{
			WorldDatabase db = getWorldDatabase();
			if(db != null){
				ResultSet result = db.executeQuery("SELECT * FROM `tmn-back-system` WHERE `PlayerName` = '" + player.getName() + "'");
				if(!result.next()){
					Vector3f NowTargetPos = event.getSpawnPosition();
					float NowNewPosx = NowTargetPos.x;
					float NowNewPosy = NowTargetPos.y;
					float NowNewPosz = NowTargetPos.z;
					db.executeUpdate("INSERT INTO `tmn-back-system` (PlayerName, BackPos_x, BackPos_y, BackPos_z) VALUES ('" + player.getName() + "', '" + NowNewPosx + "', '" + NowNewPosy + "', '" + NowNewPosz + "');");
					player.sendTextMessage("Ein /back Punkt wurde für dich erstellt.");
				}
			}
		}
		catch(SQLException e){
			e.printStackTrace();
		}
	}
	@EventMethod
	public void onPlayerDeath(PlayerDeathEvent event){
		Player player = event.getPlayer();
		player.sendTextMessage("[BACK] Neuer /back Punkt wurde erstellt...");
		Vector3f BackTargetPos = event.getDeathPosition();
		float BackNewPosx = BackTargetPos.x;
		float BackNewPosy = BackTargetPos.y;
		float BackNewPosz = BackTargetPos.z;
		WorldDatabase db = getWorldDatabase();
		if(db != null){
			db.executeUpdate("UPDATE `tmn-back-system` SET `BackPos_x` = '" + BackNewPosx + "', `BackPos_y` = '" + BackNewPosy + "', `BackPos_z` = '" + BackNewPosz + "' WHERE `PlayerName` = '" + player.getName() + "'");
		}
	}
	@EventMethod
	public void onPlayerCommand(PlayerCommandEvent event){
		Player player = event.getPlayer();
		String command = event.getCommand();
		String[] cmd = command.split(" ");
		if(cmd[0].equals("/back")){
			player.sendTextMessage("[BACK] Neuer /back Punkt wurde erstellt...");
			Vector3f BackTargetPos = event.XXXXXX
			float BackNewPosx = BackTargetPos.x;
			float BackNewPosy = BackTargetPos.y;
			float BackNewPosz = BackTargetPos.z;
			WorldDatabase db = getWorldDatabase();
			if(db != null){
				db.executeUpdate("UPDATE `tmn-back-system` SET `BackPos_x` = '" + BackNewPosx + "', `BackPos_y` = '" + BackNewPosy + "', `BackPos_z` = '" + BackNewPosz + "' WHERE `PlayerName` = '" + player.getName() + "'");
			}
		}
	}


}

Display More

Grüßle TMN

red51 · Oct 27th 2016

Quote from TutMeistensNix

Nun gibt es aber nicht mehr getPosition() wie lautet es den nun?

Die Funktion gibt es weiterhin, einfach player.getPosition() aufrufen Hier findet sich generell eine Übersicht über alle Spielerfunktionen: Player

Quote from TutMeistensNix

Kommt ein Spieler das erste mal auf den Server wird geprüft ob er schon mal da war, wenn nein dann wird ein Back Punkt erstellt anhand der getSpawnPosition() ist das ok?

Ja, also wenn du "getSpawnPosition()" auf das Eventobjekt aufrufst, dann erhälst du exakt die Position, an welcher der Spieler online kommt. Bedenke, dass PlayerSpawnEvent nur einmal aufgerufen wird, nämlich wenn der Spieler zum Server verbindet und gespawnt ist (nach seinem Ableben wird immer PlayerRespawnEvent aufgerufen, erst wenn der Spieler disconnected und wieder zum Server verbindet, wird das PlayerSpawnEvent aufgerufen - aber ich denke das entspricht auch der Intention hinter deinem Code?)

TutMeistensNix · Oct 28th 2016

Ahh, bei mir ging es nicht da ich ja event.getPosition() versucht habe.
Wie sieht es den nun aus, ich habe hier zwei Beispiele, kann ich beide so schreiben?

Mir geht es um die Zeile 5, bzw. Zeile 8.

Java

@EventMethod
	public void onPlayerDeath(PlayerDeathEvent event){
		Player player = event.getPlayer();
		player.sendTextMessage("[BACK] Neuer /back Punkt wurde erstellt...");
		Vector3f BackTargetPos = event.getDeathPosition();
		float BackNewPosx = BackTargetPos.x;
		float BackNewPosy = BackTargetPos.y;
		float BackNewPosz = BackTargetPos.z;
		WorldDatabase db = getWorldDatabase();
		if(db != null){
			db.executeUpdate("UPDATE `tmn-back-system` SET `BackPos_x` = '" + BackNewPosx + "', `BackPos_y` = '" + BackNewPosy + "', `BackPos_z` = '" + BackNewPosz + "' WHERE `PlayerName` = '" + player.getName() + "'");
		}
	}

Display More

Java

@EventMethod
	public void onPlayerCommand(PlayerCommandEvent event){
		Player player = event.getPlayer();
		String command = event.getCommand();
		String[] cmd = command.split(" ");
		if(cmd[0].equals("/back")){
			player.sendTextMessage("[BACK] Neuer /back Punkt wurde erstellt...");
			Vector3f BackTargetPos = player.getPosition();
			float BackNewPosx = BackTargetPos.x;
			float BackNewPosy = BackTargetPos.y;
			float BackNewPosz = BackTargetPos.z;
			WorldDatabase db = getWorldDatabase();
			if(db != null){
				db.executeUpdate("UPDATE `tmn-back-system` SET `BackPos_x` = '" + BackNewPosx + "', `BackPos_y` = '" + BackNewPosy + "', `BackPos_z` = '" + BackNewPosz + "' WHERE `PlayerName` = '" + player.getName() + "'");
			}
		}
	}

Display More

Ja, wenn er stirbt habe ich ja seine Position durch getDeathPosition().
Dann wird der Spieler halt durch den PlayerRespawnEvent zum Punkt gebracht und kann selbst entscheiden ob er /back benutzt oder nicht.

red51 · Oct 28th 2016

Das sieht gut aus, das kannst du so machen

Aber: Das Speichern in der Datenbank macht eigentlich nur Sinn, wenn die per Befehl "/back" gespeicherte Position auch nach dem Disconnect des Spielers verfügbar bleiben soll (also quasi dauerhafte Gültigkeit haben soll, selbst nach Serverrestart etc). Wenn es nur um eine temporäre Information geht, kannst du die Position bspw. auch einfach als Attribut des Spielers speichern (Attribute können für jeden Spieler festgelegt und anhand eines eindeutigen Strings beschrieben werden, dabei kann jedes erdenkliche Objekt als Attribut gespeichert werden - also eine Zahl, ein Vector3f, ein anderer Spieler, ein GUI Element, ein String usw). Hier ein Beispiel (wie gesagt, wenn du es weiterhin per Datenbank speichern möchtest, kannst du das hier eigentlich ignorieren^^):

Java

Player player = event.getPlayer();
Vector3f backTargetPos = event.getDeathPosition();


//Als Attribut mit dem Key "tmn-back-pos" speichern
//Sicherheitshalber einen neuen Vector3f dafür erzeugen
player.setAttribute("tmn-back-pos", new Vector3f(backTargetPos));


//...


//Irgendwo anders, wenn die Position abgerufen werden soll
//Type-casting ist wichtig! Da getAttribute() nur ein
//universelles "Object" zurückgibt
Vector3f position = (Vector3f) player.getAttribute("tmn-back-pos");


//Unbedingt prüfen ob != null, also ob Attribute zuvor gesetzt wurde
if(position != null){
    //Spieler teleportieren o.ä.
    player.setPosition(position);
}

Display More

TutMeistensNix · Oct 28th 2016

Ignorieren bestimmt nicht, brauche es bestimmt irgendwann.

Im Moment Speicher ich ja auch die letzten Standorte der Spieler in der DB und es kommt sehr gut an weil sie dann egal wann immer zurück springen können.

Participate now!