Hilfe sscanf

ocmd:heal(playerid, params[])
{
if(IsPlayerConnected(playerid))
{
new pID;
if(sscanf(playerid,"u",pID)) return SendClientMessage(playerid,ROT,"Benutze: /heal [Spielername/Spielerid]");
SetPlayerHealth(pID, 100);
}
return 1;
}

ocmd:heal(playerid, params[])
{
if(IsPlayerConnected(playerid))
{
new pID;
if(sscanf(playerid, "u", pID)) SendClientMessage(playerid,ROT,"Benutze: /heal [Spielername/Spielerid]");
SetPlayerHealth(pID, 100.0);
return 1;
}
return 1;
}
Wenn es so auch nicht richtig ist,
kann ich es Dir auch nicht sagen.

Ich arbeite nicht mit ocmd.

kann mir bitte einer sagen warum sehr viele dass machen if(IsPlayerConnected(playerid))
wenn er nicht auf dem Server wäre dann kann der doch auch kein Command benutzen

Piet du hast Recht, das ist sowas wie eine unnötige Zeile im Script.

//Der Befehl ist schon richtig von 4#Future, du hast vielleicht "sscanf" in deinen plugin ordner nicht getan oder die include vergessen.
OMG Natürlich, es heißt "if(sscanf(params..." nicht "if(sscanf(playerid..."
Peinlich Peinlich

// Edit peinlichen Fehler ausgebessert.

ich bin ebenfalls nicht der ocmd oder sscanf fan, jedoch finde ich es komisch, wenn man versucht
aus dem "string" "playerid" etwas zu lesen.

Syntax:

string[] - String to extract parameters from.
	format[] - Parameter types to get.
	{Float,_}:... - Data return variables.

Ihr lest also aus playerid, macht nicht "params" da etwas mehr sinn ?

Falls es (noch) nicht klar ist:

ocmd:heal(playerid,params[])
{
if(IsPlayerConnected(playerid))
{
new pID;
if(sscanf(params,"u",pID)) return SendClientMessage(playerid,ROT,"Benutze: /heal [Spielername/Spielerid]");
SetPlayerHealth(pID, 100);
}
return 1;
}

Jetzt lass doch die blöde Connected abfrage des Spielers selbst weg.
Wenn schon kann man diese für den betreffenden Spieler machen, aber doch nicht für den ausführenden Spieler.

ocmd:heal(playerid,params[])
{
new pID;
if(sscanf(params,"u",pID)) return SendClientMessage(playerid, 0xFFFFFF,"Benutze: /heal [Spielername/SpielerID]");
if(IsPlayerConnected(pID)) return SendClientMessage(playerid, 0xFFFFFF,"Sorry, der Spieler ist nicht online!");
SetPlayerHealth(pID,100);

return 1;
}

Wie immer um die Zeit, keine Garantie

Zitat von 4#Future

ocmd:heal(playerid, params[])
{
if(IsPlayerConnected(playerid))
{
new pID;
if(sscanf(playerid, "u", pID)) SendClientMessage(playerid,ROT,"Benutze: /heal [Spielername/Spielerid]");
SetPlayerHealth(pID, 100.0);
return 1;
}
return 1;
}
Wenn es so auch nicht richtig ist,
kann ich es Dir auch nicht sagen.

Ich arbeite nicht mit ocmd.

Alles anzeigen

als erstes ist hier die benutzung von sscanf falsch
also if(sscanf(playerid, "u", pID))

zu
if(sscanf(params,"u",pID)) da die params vergessen wurden

zu dem ist das doppel returnen sinnlos und es reicht auch 1 return
es würde dir auch reichen SendClientMessage zu returnen
sprich
if(sscanf(playerid, "u", pID)) return SendClientMessage(playerid,ROT,"Benutze: /heal [Spielername/Spielerid]");
. Zu dem ist die abfrage ob der Spielerconnectet ist sinnlos wenn da wäre die einzigste sinnvolle
abfrage vllt ob der spieler eingeloggt

Da warst du jetzt aber ein bisschen langsam Prototype