Befehl aus externer Funktion ggf. abbrechen/beenden

TheQuadratos · 24. November 2012

Guten Abend Allerseits,

momentan bin ich am scripten, doch nun stellt sich mir folgende Frage:

Kann man von einer externen Funktionen einen Befehl abbrechen?

BEISPIEL:
Wir haben über OCMD einen Befehl erstellt. In diesem Rufen wir eine externe Funktion auf.

Befehl:

Code

ocmd:kill(playerid, params[])
{
	KillPlayer(playerid);
	SendClientMessage(playerid, -0, "Spieler erfolgreich gekillt!");
	return 1;
}

Nun wird in der Funktion geprüft ob der Spieler im Krankenhaus ist. Sollte dies zutreffen wird ja die Funktion
mit return 1 verlassen. Ich möchte jedoch, dass der Befehl abgebrochen wird, also das SendClientMessage nicht
mehr aufgerufen wird.

Funktion:

Code

KillPlayer(playerid)
{
 	if(ImKrankenhaus(playerid))) return 1;
 	SetPlayerHealth(playerid, 0.0);
 	return 0;
}

Möchte aber in dem Befehl keine if-Prüfung wie z.B.

Code

if(KillPlayer(playerid)) return 1;

Gibt es eine Möglichkeit?

Info: Das ist lediglich ein Beispiel und ist schlicht und einfach dargestellt.

Grüße
TheQuadratos

Campbell · 24. November 2012

Eine Möglichkeit etwas 'abzubrechen' ist mir nicht bekannt. Soweit ich weiß ist nur durch Validierung der Rückgabe möglich, einen Teil nicht ausführen zu lassen. Du musst dir das ja so vorstellen, dass das Script von oben nach unten arbeitet. Demnach muss auch dein Befehl irgendwann etwas zurückgeben, damit es an anderer Stelle weitergeht. Ob ein 'Abbrechen' dann überhaupt sinnvoll ist, ist fraglich.

Akino Kiritani · 24. November 2012

if(KillPlayer(playerid) == 1) return 1;

ergibt dann

ocmd:kill(playerid, params[])
{
if(KillPlayer(playerid) == 1) return 1; // oder was passieren soll
SendClientMessage(playerid, -0, "Spieler erfolgreich gekillt!");
return 1;
}

Tround · 24. November 2012

Zitat von TheQuadratos

Möchte aber in dem Befehl keine if-Prüfung wie z.B.

..
glaube ohne if bedingung wird das nicht klappen.

TheQuadratos · 24. November 2012

Es muss doch eine möglichkeit geben. Mit if-Bedingungen schaut der Code nicht ganz
so geil aus wie ohne.

Mit goto habe ich es auch schon versucht, leider kann man mit goto nicht in eine
andere Funktion eingreifen.

Hat jemand eine gute Lösung oder Tipps?

Campbell · 24. November 2012

Dein Befehl muss etwas zurückgeben (return), daher muss nach dem Ausführen der Funktion noch mindestens ein return kommen. Man kann sie nicht 'abbrechen'.

TheQuadratos · 24. November 2012

Ja, das habe ich doch auch verstanden.

Hier wollte ich fragen, ob es so etwas wie continue gibt was man ja wie
bekanntlich in Schleifen benutzt.

Also eine art continue für Funktionen oder so etwas in der Art.

Campbell · 24. November 2012

Nein. Es gibt zwar goto (http://www.compuphase.com/pawn/Pawn_Language_Guide.pdf - Seite 117), dennoch ist es nicht in Funktionen zu verwenden.

InternetInk aka Benjamin · 24. November 2012

goto ist Müll und der Code soll nicht geil aussehen er soll funktionieren xD
Wenn du eine extra Funktion hast dann kannst du auch deren Rückgabewert in einer Abfrage nutzen, dafür ist er da.
Im übrigen würde ich die Funktion von der Rückgabe her aus reiner Logik andersherum benennen, 0 also false wird zurückgegeben wenn es nicht geklappt hat...

KillPlayer(playerid)
{
if(ImKrankenhaus(playerid))) return 0;
SetPlayerHealth(playerid, 0.0);
return 1;
}

ocmd:kill(playerid, params[])
{
//...
if(KillPlayer(playerid)) return SendClientMessage(playerid, -0, "Spieler erfolgreich gekillt!");
else return SendClientMessage(playerid, -0, "Diesen Spieler kannst du nicht killen!");
}

TheQuadratos · 24. November 2012

Okay, schade dass es keine Möglichkeit gibt.

Nun mal eine weitere Frage:
Was sieht schöner aus bzw. sollte ich eurer Meinung nach verwenden?

Beispiel 1:

Code

if(Pruefung_LoggedIn(playerid)) return 1;
	if(Pruefung_NotAFK(playerid)) return 1;
	if(Pruefung_CommandBlock(playerid)) return 1;

Beispiel 2:

Code

if(Pruefung_LoggedIn(playerid) && Pruefung_NotAFK(playerid) && Pruefung_CommandBlock(playerid)) return 1;

Grüße
Quadratos

InternetInk aka Benjamin · 24. November 2012

Das kommt darauf an, wenn ALLES erfüllt sein muss um 1 zurückzugeben dann kannste auch direkt das nehmen was du bei Beispiel 2 geschrieben hast.
Wenn irgendeines davon erfüllt sein muss kannst du auch das Beispiel 2 nehmen aber dann halt statt && einfach
Wenn du natürlich unterschiedliche SendClientMessages zurückgeben willst, was ja manchmal angebracht ist dann musst du zwangsweise das Beispiel 1 nehmen.

TheQuadratos · 24. November 2012

Die SendClientMessages befinden sich in den aufgerufenen Funktionen (Pruefung_LoggedIn, etc.).

Jede Prüfung sollte 1 zurückgeben. Also der Spieler soll eingeloggt sein, nicht AFK sein und kein Command Block haben.

Bin momentan noch am Überlegen ob ich 0er oder wie du es mir vorgeschlagen hast oder 1er returnen lasse.

Campbell · 24. November 2012

Returne 1 bzw. true bei Erfolg und 0 bzw. false falls es nicht gelungen ist. Das ist das Prinzip, das die meisten verfolgen (sollten). Demnach ist klar, ob es erfolgreich oder nicht erfolgreich war ohne jeglichen Schnick-Schnack.

breadfish.de 11. März 2022