Stringlänge bei Befehlen mit String als Parameter zu kurz

Zeig am besten mal den Befehl.

Ohne moß nichts los.

Zitat von Antony_Morvai

Warum mit ocmd?

Ich persönlich finde es so leichter:

//----------------------------------[ooc]-----------------------------------------------
if(strcmp(cmd, "/ooc", true) == 0 || strcmp(cmd, "/o", true) == 0)
{
if(IsPlayerConnected(playerid))
{
if(gPlayerLogged[playerid] == 0)
{
SendClientMessage(playerid, COLOR_RED, "Du bist nicht eingelogt.");
return 1;
}
if ((noooc) && PlayerInfo[playerid][pAdmin] < 1)
{
SendClientMessage(playerid, COLOR_GREEN, "Der OOC ist deaktiviert.");
return 1;
}
if(PlayerInfo[playerid][pMuted] == 1)
{
SendClientMessage(playerid, COLOR_RED, "Sie sind gemutet.");
return 1;
}
GetPlayerName(playerid, sendername, sizeof(sendername));
new length = strlen(cmdtext);
while ((idx < length) && (cmdtext[idx] <= ' '))
{
idx++;
}
new offset = idx;
new result[200];
while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
{
result[idx - offset] = cmdtext[idx];
idx++;
}
result[idx - offset] = EOS;
if(!strlen(result))
{
SendClientMessage(playerid, COLOR_LIGHTBLUE, "Benutze: (/o)oc [Text]");
return 1;
}
format(string, sizeof(string), "(( %s: %s ))", sendername, result);
OOCOff(COLOR_OOC,string);
printf("%s", string);
}
return 1;
}
if(strcmp(cmd, "/noooc", true) == 0)
{
GetPlayerName(playerid, sendername, sizeof(sendername));
if(IsPlayerConnected(playerid))
{
if (PlayerInfo[playerid][pAdmin] >= 1 && (!noooc))
{
noooc = 1;
format(string, sizeof(string), "Der OOC wurde von Admin %s(%d) deaktiviert.", sendername,playerid);
BroadCast(COLOR_LIGHTBLUE,string);
}
else if (PlayerInfo[playerid][pAdmin] >= 1 && (noooc))
{
noooc = 0;
format(string, sizeof(string), "Der OOC wurde von Admin %s(%d) aktiviert.", sendername,playerid);
BroadCast(COLOR_LIGHTBLUE, string);
}
}
return 1;
}

Alles anzeigen

Genau, warum 19 Zeilen wenn man auch 65 haben kann

@Antony_Morvai, hast du ein an der Waffel ?
Aus dem GF Kopiert und es ist nicht mal leichter.
Keine Ahnung aber mitreden wollen...

Versuch es so:

ocmd:o(playerid,params[])
{
new msg;
if(sscanf(params,"s[128]",msg))return SendClientMessage(playerid,ROT,"Benutzung: /o(oc) [Nachricht]");
if(ochat == true) {
format(msg,sizeof msg,"(( %s: %s ))",SpielerName(playerid),msg);
SendClientMessageToAll(WEISS,msg);
}
else {
if(IstSpielerAdmin(playerid,1)) {
format(msg,sizeof msg,"(( %s: %s ))",SpielerName(playerid),msg);
SendClientMessageToAll(WEISS,msg);
}
else {
SendClientMessage(playerid,ROT,"Der O-Chat ist derzeit geschlossen. Nur Leute aus dem Adminteam können ihn benutzen.");
}
}
return true;
}

benutze einfach den sscanf stock und nicht plugin ;d
den stock findest du auf samp wiki

damit du keine probleme hast?...

Das wäre jetzt schnell aus dem Kopf

if(!strcmp(cmdtext, "/ooc", true,4)){
if(!strlen(cmdtext[4])) SendClientMessage(playerid, 0xffffffff, "Benutzung: /ooc [text]");
else {
new tmp[192];
GetPlayerName(playerid,tmp,24);
format(tmp,192,"((OOC - %s: %s))",tmp,cmdtext[5]);
SendClientMessageToAll(0xFFFFFFAA,tmp);
print(tmp);
}
return 1;
}

Kann ich dir damit irgendwie helfen?

Stimmt, lol.

Du würdest ja auch nicht den 100€ Schein nehmen, wenn du einen 200-er kriegen könntest, obwohl er dreckig ist. Diesen würdest du lieber reinigen lassen.

Könnte es ggf. auch am SpielerName liegen? Wie sieht die Funktion da aus? Nicht dass da irgendwo ein \n \0 oder so ist :p
Ansonsten einfach sscanf neu runterladen (inkl. include), in include ordner von pawn reinhauen und neu compilen.

ps: bei mir geht sscanf2 natürlich... @msk

Jetzt wo du's sagst -.-

Falls jemand dasselbe Problem hat:

Benutzung ist:

new string[128]
sscanf(params, "s[128]", string)

ocmd:o(playerid,params[])
{
new spielertext[128],string[128];
if(sscanf(params,"s",spielertext)) { return SendClientMessage(playerid,COLOR_RED,"Benutzung: /o(oc) [Nachricht]"); }
if(ochat == true)
{
format(string,sizeof(string),"(( %s: %s ))",SpielerName(playerid),spielertext);
SendClientMessageToAll(WEISS,string);
}
else {

if(IstSpielerAdmin(playerid,1)) return SendClientMessage(playerid,ROT,"Der O-Chat ist derzeit geschlossen. Nur Leute aus dem Adminteam können ihn benutzen.");//Admin-Afrage anpasssen sodass er abfragt ob er kein Admin ist
format(msg,sizeof msg,"(( %s: %s ))",SpielerName(playerid),msg);
SendClientMessageToAll(WEISS,msg);
}

return 1;
}

Mit freundlichen Grüßen
Sven

//Edit Code

Zitat von lollipop

ocmd:o(playerid,params[])
{
new spielertext[128],string[128];
if(sscanf(params,"s",spielertext)) { return SendClientMessage(playerid,COLOR_RED,"Benutzung: /o(oc) [Nachricht]"); }
if(ochat == true)
{
format(string,sizeof(string),"(( %s: %s ))",SpielerName(playerid),spielertext);
SendClientMessageToAll(WEISS,string);
}
else {

if(IstSpielerAdmin(playerid,1)) return SendClientMessage(playerid,ROT,"Der O-Chat ist derzeit geschlossen. Nur Leute aus dem Adminteam können ihn benutzen.");//Admin-Afrage anpasssen sodass er abfragt ob er kein Admin ist
format(msg,sizeof msg,"(( %s: %s ))",SpielerName(playerid),msg);
SendClientMessageToAll(WEISS,msg);
}

return 1;
}

Mit freundlichen Grüßen
Sven

//Edit Code

Alles anzeigen

Und das gibt bei dir keine Fehlermeldung..?
if(sscanf(params,"s",spielertext)) { return SendClientMessage(playerid,COLOR_RED,"Benutzung: /o(oc) [Nachricht]"); }
zu
if(sscanf(params,"s[128]",spielertext)) { return SendClientMessage(playerid,COLOR_RED,"Benutzung: /o(oc) [Nachricht]"); }
nebenbei, du brauchst nur eine von beiden stringvariablen zu erstellen und kannst jene für beides benutzen.

Möchte mich nicht rechtfertigen war aber jez nach ner langen Pause wieder mal dran Urlaub und sowas.
ALso hier:
ocmd:o(playerid,params[])
{
new spielertext[128],string[128];
if(sscanf(params,"s",spielertext)) return SendClientMessage(playerid,COLOR_RED,"Benutzung: /o(oc) [Nachricht]");
if(ochat == true)
{
format(string,sizeof(string),"(( %s: %s ))",SpielerName(playerid),spielertext);
SendClientMessageToAll(WEISS,string);
}
else {

if(IstSpielerAdmin(playerid,1)) return SendClientMessage(playerid,ROT,"Der O-Chat ist derzeit geschlossen. Nur Leute aus dem Adminteam können ihn benutzen.");//Admin-Afrage anpasssen sodass er abfragt ob er kein Admin ist
format(msg,sizeof msg,"(( %s: %s ))",SpielerName(playerid),msg);
SendClientMessageToAll(WEISS,msg);
}

return 1;
}
Das sollte jetzt richtig sein.
Ich habs nicht compiled sondern nur auf die schnelle geschrieben.
Ich denke mit meinem Codeschnipsel sollte es funktionieren.
Zum eig. problem. Muss man bei sscanf 2 das so machen? Also
"if(sscanf(params,"s[128]",spielertext))"Also das mit den [128]
Musste es nie machen bzw. habs nie probiert. Also mach das weg dann sollte es auch gehn.

Mit freundlichen Grüßen
Sven