Mehrfache nutzung eines Befehls

Ich hätte den Befehl anders aufgebaut, nur ein Vorschlag:

ocmd:blitzer(playerid, params[])
{
new speed, method[12];
if(sscanf(params, "si", method, speed)) return SendClientMessage(playerid, Rot, "Benutzung: /blitzer [add] [Geschwindigkeit]");
if(method == "add") return SendClientMessage(playerid, Gelb, "Moin 1");
else if(method == "remove") return SendClientMessage(playerid, Gelb, "Moin 2");
return SendClientMessage(playerid, Rot, "Es gibt nur add oder remove.");
}

Ungetestet

Ich habe die Klammern nur weggelassen weil ich dachte dann sparst du dir Zeilen.
Aber wenn du dich damit nicht so gut zurecht findest hier bitte mit klammern:

ocmd:blitzer(playerid, params[])
{
new speed, method[12];
if(sscanf(params, "s[12]i", method, speed)) return SendClientMessage(playerid, Rot, "Benutzung: /blitzer [add] [Geschwindigkeit]");
if(method == "add") return
{
SendClientMessage(playerid, Gelb, "Moin 1");
if(speed > 29 && speed < 151)
{
AddBlitzer(playerid, speed, 25);
}
}
else if(method == "remove")
{
return SendClientMessage(playerid, Gelb, "Moin 2");
}
return SendClientMessage(playerid, Rot, "Es gibt nur add oder remove.");
}

Zitat von [BE]axi92

if(method == "add")

In Pawn muss man Strings mit strcmp vergleichen

Sprich

if(!strcmp(method,"add",true))

Danke für die Korrektur, stimmt sind ja auch nur Arrays^^
Aber bei den ganzen Programmiersprachen kommt man auch mal durcheinander

ocmd:blitzer(playerid, params[])
{
new speed, method[12];
if(sscanf(params, "s[12]i", method, speed)) return SendClientMessage(playerid, Rot, "Benutzung: /blitzer [add] [Geschwindigkeit]");
if(!strcmp(method, "add", true))
{
SendClientMessage(playerid, Gelb, "Moin 1");
if(speed > 29 && speed < 151)
{
AddBlitzer(playerid, speed, 25);
}
}
else if(!strcmp(method, "remove", true))
{
return SendClientMessage(playerid, Gelb, "Moin 2");
}
return SendClientMessage(playerid, Rot, "Es gibt nur add oder remove.");
}

ocmd:blitzer(playerid, params[])
{
if(!isPlayerInFrakt(playerid, 2))return SendClientMessage(playerid, Rot, "Du bist nicht berechtigt einen Blitzer aufzubauen.");
if(isPlayerFRang(playerid, 3))return SendClientMessage(playerid, Rot, "Du musst mindestens Geselle sein um einen Blitzer aufzubauen.");
new speed, funktion[16], string[128];
if(sscanf(params, "s", funktion)) return SendClientMessage(playerid, Rot, "Benutzung: /blitzer [add] [Geschwindigkeit] / [remove] / [removeall]");
if(!strcmp(funktion, "add", true))
{
if(sscanf(params, "si", funktion, speed)) return SendClientMessage(playerid, Rot, "Benutzung: /blitzer [add] [Geschwindigkeit]");
if(speed > 29 && speed < 151)
{
AddBlitzer(playerid, speed, 25);
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(isPlayerInFrakt(i, 1) || isPlayerInFrakt(i, 2))
{
format(string, sizeof(string), "HQ: An alle Einheiten: Das System meldet:");
SendClientMessage(i, Meldung, string);
format(string, sizeof(string), "HQ: Ordnungshüter %s hat einen Blitzer aufgebaut. Geschwindigkeit: %dkm/h.", SpielerName(playerid), speed);
SendClientMessage(i, Meldung, string);
}
}
return 1;
}
return SendClientMessage(playerid, Rot, "Die Geschwindigkeit muss zwischen 30 und 150 km/h sein.");
}
else if(!strcmp(funktion, "remove", true))
{
SendClientMessage(playerid, Gelb, "Moin hier alle");
return 1;
}
return SendClientMessage(playerid, Rot, "Die Funktionen für den Blitzer sind [add] / [remove] / [removeall]");
}

Versuch es mal so

Schreib das mal so:

ocmd:blitzer(playerid, params[])
{
if(!isPlayerInFrakt(playerid, 2)) return SendClientMessage(playerid, Rot, "Du bist nicht berechtigt einen Blitzer aufzubauen.");
if(isPlayerFRang(playerid, 3)) return SendClientMessage(playerid, Rot, "Du musst mindestens Geselle sein um einen Blitzer aufzubauen.");
new speed, funktion[16];
if(sscanf(params, "s[16]D(0)", funktion, speed)) return SendClientMessage(playerid, Rot, "Benutzung: /blitzer [add] [Geschwindigkeit] / [remove] / [removeall]");
if(!strcmp(funktion, "add", true))
{
if(!speed) return SendClientMessage(playerid, Rot, "Benutzung: /blitzer [add] [Geschwindigkeit] / [remove] / [removeall]");
if(speed <= 29 || speed > 151) return SendClientMessage(playerid, Rot, "Die Geschwindigkeit muss zwischen 30 und 150 km/h sein.");
AddBlitzer(playerid, speed, 25);
new string[64],string1[128];
format(string1, sizeof(string1), "HQ: Ordnungshüter %s hat einen Blitzer aufgebaut. Geschwindigkeit: %dkm/h.", SpielerName(playerid), speed);
format(string, sizeof(string), "HQ: An alle Einheiten: Das System meldet:");
for(new i; i < MAX_PLAYERS; i++) {
if(isPlayerInFrakt(i, 1) || isPlayerInFrakt(i, 2)) {
SendClientMessage(i, Meldung, string),SendClientMessage(i, Meldung, string1);
}
}
return 1;
} else if(!strcmp(funktion, "remove", true)) {
SendClientMessage(playerid, Gelb, "Moin hier alle");
return 1;
}
return SendClientMessage(playerid, Rot, "Die Funktionen für den Blitzer sind [add] / [remove] / [removeall]");
}

mfg.

Zitat von Kaliber

D(0)

Das habe ich bei sscanf noch nicht gesehen was sagt das genau aus?

[BE]axi92: Das ist ein optionaler Integer Parameter mit dem default Wert 0.

Zitat von Jeffry

[BE]axi92: Das ist ein optionaler Integer Parameter mit dem default Wert 0.

Wie kennzeichnet man das, dass er optional ist? Großbuchstabe?

Lösung:
War mit der Annahme richtig unterwegs hier zu finden in der Doku: https://github.com/Y-Less/sscanf/wiki/Optional-Specifiers

Was bewirkt dieses D(0)
In diesem Beispiel?
Das d nur beim add zählt?

Nein in Zeile 6 kannst du im Chat "/blitzer add" schreiben somit hat er den speed parameter nicht da er ja nur optional ist gibt er da auch noch keine Meldung aus.

Kommst du dann aber zu Zeile 9 überprüft er ja das speed nochmal da aber nichts vorhanden ist gibt er dir das return aus.