SendClientMessage mit 2 Strings

Hey,
also wenn ich es richtig verstanden habe sollte es so funktionieren?

ocmd:pn(playerid,params[])
{
new pID,text[128];
if(sscanf(params,"is[128]",pID,text))return SendClientMessage(playerid,Rot,"Befehl: {0080FF}/pn [SpielerID] [Nachricht]");
SendClientMessage(pID,Grün,"Nachricht von ID %i: %s",playerid,text);
return 1;
}

Wenn ich es allerdings falsch verstanden habe würde ich mich freuen wenn du es nochmal genauer erklären könntest.
Da ich nicht genau wusste wie du es meinst

ocmd:pn(playerid,params[])
{
new pID,text[128],Spieler[128];
if(sscanf(params,"is[128]",pID,text))return SendClientMessage(playerid,Rot,"Befehl: {0080FF}/pn [SpielerID] [Nachricht]");
format(Spieler,sizeof(Spieler),"Nachricht von ID %i:%s",playerid, text);
SendClientMessage(pID,Grün,Spieler);
return 1;
}

einfach
format(Spieler,sizeof(Spieler),"Nachricht von ID %i:",playerid);
zu
format(Spieler,sizeof(Spieler),"Nachricht von ID %i:%s",playerid, text);

und die sscanf abfrage vor dem formatieren weil du ja erst abfragen musst ob er das richtige eingegeben hat und dann formatieren

du hast vergessen den Text in die Formatierung einzubinden

Mit Freundlichen Grüßen
Snake ♤

#edit Fehler behoben
#edit nochmal den Fehler richtig erklärt

@snake112:
Deins wird nicht klappen da du erst das format hast und dann die sccanf abfrage.

@[SM]Rasbroth:
Mach es so. Mein erster Versuch ist auch falsch

ocmd:pn(playerid,params[])
{
new pID,text[128],string[128];
if(sscanf(params,"is[128]",pID,text))return SendClientMessage(playerid,Rot,"Befehl: {0080FF}/pn [SpielerID] [Nachricht]");
format(string,sizeof(string),"Nachricht von ID %i: %s",playerid, text);
SendClientMessage(pID,Grün,string);
return 1;
}

Desweiteren sollte der text nicht 128 haben sondern am besten nur 80.
Da eine Nachricht nur 128 Zeichen haben darf und du ja davor schon einiges hast.
sprich so
ocmd:pn(playerid,params[])
{
new pID,text[80],string[128];
if(sscanf(params,"is[80]",pID,text))return SendClientMessage(playerid,Rot,"Befehl: {0080FF}/pn [SpielerID] [Nachricht]");
if(strlen(text) > 80)return SendClientMessage(playerid,ROT,"Der text darf nur 80 Zeichen beinhalten");
format(string,sizeof(string),"Nachricht von ID %i: %s",playerid, text);
SendClientMessage(pID,Grün,string);
return 1;
}

PermanenT:

uns sry Danke für den Hinweis bin mit Handy on aber misst bloß die 2 mit der 3 tauschen :))
habe das von ihm kopiert und nicht drauf geachtet

Ja, dies geht.
schau dir diese Funktion an
GetPlayerName

Zitat von PermanenT

text[128],string[128]

Willst du nicht noch mehr Arrays erstellen?

Man kann das auch so machen, ohne 256 cells zu verschwenden:

ocmd:pn(playerid,params[])
{
new pID;
if(sscanf(params,"is[128]",pID,params))return SendClientMessage(playerid,Rot,"Befehl: {0080FF}/pn [SpielerID] [Nachricht]");
if(!IsPlayerConnected(pID)) return SendClientMessage(playerid,Rot,"Ein Spieler mit dieser ID ist nicht online!"); //Wäre evtl auch ganz nützlich xD
format(params,145,"Nachricht von ID %i: %s",playerid,params);
SendClientMessage(pID,Grün,params);
return 1;
}

Ich bin nicht 100%ig sicher ob sscanf da mitspielt, aber eigentlich sollte das gehen

Zitat von [SM]Rasbroth

Bitte keinen Code oder so, nur die Frage beantworten (Danke).

Natürlich...GetPlayerName

mfg.

Kaliber:
Ich werde es mir für's nächste mal merken

@[SM]Rasbroth:
Doch du kansnt IsPlayerCOnnected auch ohne Speichersystem nutzen

Zitat von [SM]Rasbroth

Das mit "IsPlayerConnectet" kann ich doch ohne Speichersystem garnicht verwenden oder ?

Bitte was?

Zitat von [SM]Rasbroth

Funktioniert alles, InGame sieht es so aus, wie ich es haben möchte.

Das ist schön Es würde auch funktionieren, wenn du die Arrays auf 500 jeweils setzen würdest, ändert nichts daran, dass man dies nicht muss..

mfg.

Zitat von Kaliber

...Nein snake...nein

xD habs falsch verstanden was er geschrieben hatte aber so wäre einfacher müsste er nicht immer
GetPlayerName(playerid,name,sizeof(name));

machen