Haussystem Kommand Problem /createh

lollipop · 21. Mai 2012

Guten Tag,
ich arbeite derzeit an meinem Haussystem für mein Dorfreallifeprojekt,
nun habe ich ein Problem beim erstellen:
if(strcmp(cmd, "/createh", true) == 0)
{
if(IsPlayerConnected(playerid))
{
if(pInfo[playerid][pAdminlevel] < 4)
{
SendClientMessage(playerid, COLOR_RED, "Du bist nicht befugt diesen Befehl zu benutzten.");
return 1;
}
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
return 1;
}
new priceh;
priceh = strval(tmp);
if(priceh < 1) { SendClientMessage(playerid, COLOR_GREY, "Das Haus kann nicht weniger als 1$ kosten."); return 1; }
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
return 1;
}
new inter;
inter = strval(tmp);
if(inter < 0 || inter > 15) { SendClientMessage(playerid, COLOR_GREY, "Die Interior-ID kann nicht unter 0 und über 15 sein."); return 1; }
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
return 1;
}
new Float:X,Float:Y,Float:Z;
GetPlayerPos(playerid, X,Y,Z);
format(string, sizeof(string), "Du hast ein Haus welches %d kostet mit dem Interrior: %d erstellt.",priceh,inter);
SendClientMessage(playerid, COLOR_GREY, string);
CreateHaus(X,Y,Z,priceh,"Keiner",inter);
}
return 1;
}
Wenn ich nun InGame /createh 5 0 eingebe, dann wird mir angezeigt:
"Benutzung: /createh [Preis] [Interrior]"
Ich hoffe mir kann jemand helfen.

Mit freundlichen Grüßen
Sven

//edit Überschrift
//edit Rechtschreibfehler

beyti · 21. Mai 2012

SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]"); du must das wegmachen dan komt es nichtmehr

lollipop · 21. Mai 2012

Intelligentsbolzen dann funktioniert das Command trotzdem nicht...

Acrio · 21. Mai 2012

Vielleicht klappt es wenn du dieses hier if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
return 1;
}
mit sscanf machst. Bin mir aber nicht sicher.

PoTTii · 21. Mai 2012

Mal ganz ehrlich.
Wie benutzt du nicht einfach sscanf? Es ist nicht sonderlich schwer, und es verkleinert deinen Befehl enorm wodurch du eine bessere Übersicht behälst.

Zu deinem Problem kann ich dir aber leider nicht weiterhelfen.

lollipop · 21. Mai 2012

Ich behalt damit eher den Überblick ;D
Naja wie gehts mit sscanf fange grade praktisch von 0 an sry

BigBenLPs · 21. Mai 2012

Warum so komplex mach es so :

if(strcmp(cmd, "/createh", true) == 0)
{
if(IsPlayerConnected(playerid))
{
if(pInfo[playerid][pAdminlevel] != 4) return SendClientMessage(playerid, COLOR_RED, "Du bist nicht befugt diesen Befehl zu benutzten.");
new priceh,inter;
if(sscanf(params,"ii",priceh,inter)) return SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
if(priceh != 1) return SendClientMessage(playerid, COLOR_GREY, "Das Haus kann nicht weniger als 1$ kosten.");
if(inter < 0 || inter > 15) return SendClientMessage(playerid, COLOR_GREY, "Die Interior-ID kann nicht unter 0 und über 15 sein.");
new Float:X,Float:Y,Float:Z;
GetPlayerPos(playerid, X,Y,Z);
format(string, sizeof(string), "Du hast ein Haus welches %d kostet mit dem Interrior: %d erstellt.",priceh,inter);
SendClientMessage(playerid, COLOR_GREY, string);
CreateHaus(X,Y,Z,priceh,"Keiner",inter);
}
return 1;
}

Hier im Editor geschrieben können Fehler vorhanden sein ;)...

PoTTii · 21. Mai 2012

Angenommen du hast Sscanf und OCMD.

ocmd:createh(playerid, params[]) //wenn /createh geschrieben wird
{
if(pInfo[playerid][pAdminlevel] < 4)
{
SendClientMessage(playerid, COLOR_RED, "Du bist nicht befugt diesen Befehl zu benutzten.");
return 1;
}
new priceh, inter;
if(sscanf(params, "ii", priceh, inter)) // wird priceh oder inter nicht ausgefüllt (zb nur /createh 0 statt /createh 0 1
return SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
if(priceh < 1)
return SendClientMessage(playerid, COLOR_GREY, "Das Haus kann nicht weniger als 1$ kosten.");
if(inter < 0 || inter > 15)
return SendClientMessage(playerid, COLOR_GREY, "Die Interior-ID kann nicht unter 0 und über 15 sein.");
new Float:X,Float:Y,Float:Z;
GetPlayerPos(playerid, X,Y,Z);
format(string, sizeof(string), "Du hast ein Haus welches %d kostet mit dem Interrior: %d erstellt.",priceh,inter);
SendClientMessage(playerid, COLOR_GREY, string);
CreateHaus(X,Y,Z,priceh,"Keiner",inter);
}

lollipop · 21. Mai 2012

Ich bleib lieber bei den standart Commands mit OCMD habe ich nur Probleme ;D
So nun habe ich es eig soweit nur:
C:\Users\Sven\Desktop\0.3e\gamemodes\Selfmade.pwn(540) : error 017: undefined symbol "params"
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.
Dank an: [BJR]Big_Ben

Wie fixe ich das?

PoTTii · 21. Mai 2012

Weil sscanf nicht mit dem standart strcmp geht sondern nur mit ocmd, dcmd etc

Nimm einfach meins.

Achja.

bei deinem format

%d == float (kommazahl)
%i == integer (ganzzahl)

ergo -> benutz %i statt %d

lollipop · 21. Mai 2012

Habs selber gefixt ;D Trotzdem dank an alle ;D
Habs mit ocmd
Hier der Code der passt:
ocmd:hcreate(playerid, params[])
{
if(pInfo[playerid][pAdminlevel] < 4)
{
SendClientMessage(playerid, COLOR_RED, "Du bist nicht befugt diesen Befehl zu benutzten.");
return 1;
}
new priceh, inter,string[100];
if(sscanf(params, "ii", priceh, inter)) return SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
if(priceh < 1) return SendClientMessage(playerid, COLOR_GREY, "Das Haus kann nicht weniger als 1$ kosten.");
if(inter < 0 || inter > 15)return SendClientMessage(playerid, COLOR_GREY, "Die Interior-ID kann nicht unter 0 und über 15 sein.");
new Float:X,Float:Y,Float:Z;
GetPlayerPos(playerid, X,Y,Z);
format(string, sizeof(string), "Du hast ein Haus welches %d kostet mit dem Interrior: %d erstellt.",priceh,inter);
SendClientMessage(playerid, COLOR_GREY, string);
CreateHaus(X,Y,Z,priceh,"Keiner",inter);
return 1;
}

PoTTii · 21. Mai 2012

Zitat von Aaron Hotchner

Weil sscanf nicht mit dem standart strcmp geht sondern nur mit ocmd, dcmd etc

Nimm einfach meins.

Achja.

bei deinem format

%d == float (kommazahl)
%i == integer (ganzzahl)

ergo -> benutz %i statt %d

Alles anzeigen

lollipop · 21. Mai 2012

Alles klar habs geändert danke ;D

BigBenLPs · 21. Mai 2012

Ob Aaron da Recht hat ka...
Ich würde dir ZCMD empfehlen ist nach meinem Wissen die schnellste CMD Variante
dazu einfach die ZCMD Include von hier : http://forum.sa-mp.com/showthread.php?t=91354
in den Includes Ordner und oben ins Script #include <zcmd>
Nun einfach das
if(strcmp(cmd, "/createh", true) == 0)

mit dem ersetzen :

COMMAND:createh(playerid,params[])

Wichtig alle estra CMD Arten wie ZCMD dürfen nicht unter dem Callback
public OnPlayerCommandText(playerid, cmdtext[])
{
return 1;
}

stehen sie müssen immer außerhalb eines Callbacks stehen Beispiel :

public OnPlayerCommandText(playerid, cmdtext[])
{
return 1;
}
COMMAND:createh(playerid,params[])

{
if(IsPlayerConnected(playerid))
{
if(pInfo[playerid][pAdminlevel] != 4) return SendClientMessage(playerid, COLOR_RED, "Du bist nicht befugt diesen Befehl zu benutzten.");
new priceh,inter;
if(sscanf(params,"ii",priceh,inter)) return SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
if(priceh != 1) return SendClientMessage(playerid, COLOR_GREY, "Das Haus kann nicht weniger als 1$ kosten.");
if(inter < 0 || inter > 15) return SendClientMessage(playerid, COLOR_GREY, "Die Interior-ID kann nicht unter 0 und über 15 sein.");
new Float:X,Float:Y,Float:Z;
GetPlayerPos(playerid, X,Y,Z);
format(string, sizeof(string), "Du hast ein Haus welches %d kostet mit dem Interrior: %d erstellt.",priceh,inter);
SendClientMessage(playerid, COLOR_GREY, string);
CreateHaus(X,Y,Z,priceh,"Keiner",inter);
}
return 1;
}

MFG

lollipop · 21. Mai 2012

I know im not stupid ;D
Wie gesagt muss nur grade bissl neu lernen bzw. erneut lernen ansosnten kenn ich ja alles ;D
Bin hier seit 2010 registriert habe shcon Ahnung ;D

PoTTii · 21. Mai 2012

Zitat von [BJR]Big_Ben

Ob Aaron da Recht hat ka...
Ich würde dir ZCMD empfehlen ist nach meinem Wissen die schnellste CMD Variante
dazu einfach die ZCMD Include von hier : http://forum.sa-mp.com/showthread.php?t=91354
in den Includes Ordner und oben ins Script #include <zcmd>
Nun einfach das
if(strcmp(cmd, "/createh", true) == 0)

mit dem ersetzen :

COMMAND:createh(playerid,params[])

Wichtig alle estra CMD Arten wie ZCMD dürfen nicht unter dem Callback
public OnPlayerCommandText(playerid, cmdtext[])
{
return 1;
}

stehen sie müssen immer außerhalb eines Callbacks stehen Beispiel :

public OnPlayerCommandText(playerid, cmdtext[])
{
return 1;
}
COMMAND:createh(playerid,params[])

{
if(IsPlayerConnected(playerid))
{
if(pInfo[playerid][pAdminlevel] != 4) return SendClientMessage(playerid, COLOR_RED, "Du bist nicht befugt diesen Befehl zu benutzten.");
new priceh,inter;
if(sscanf(params,"ii",priceh,inter)) return SendClientMessage(playerid, COLOR_RED, "Benutzung: /createh [Preis] [Interrior]");
if(priceh != 1) return SendClientMessage(playerid, COLOR_GREY, "Das Haus kann nicht weniger als 1$ kosten.");
if(inter < 0 || inter > 15) return SendClientMessage(playerid, COLOR_GREY, "Die Interior-ID kann nicht unter 0 und über 15 sein.");
new Float:X,Float:Y,Float:Z;
GetPlayerPos(playerid, X,Y,Z);
format(string, sizeof(string), "Du hast ein Haus welches %d kostet mit dem Interrior: %d erstellt.",priceh,inter);
SendClientMessage(playerid, COLOR_GREY, string);
CreateHaus(X,Y,Z,priceh,"Keiner",inter);
}
return 1;
}

MFG

Alles anzeigen

Kollege, gewähn dir mal was an...

Wenn du schreibst
if(pInfo[playerid][pAdminlevel] != 4) return SendClientMessage(playerid, COLOR_RED, "Du bist nicht befugt diesen Befehl zu benutzten.");

Kann NUR Adminrang 4 es ausführen.

Es müsste heissen:

if(pInfo[playerid][pAdminlevel] < 4) return SendClientMessage(playerid, COLOR_RED, "Du bist nicht befugt diesen Befehl zu benutzten.");

Selbiges beim Geld.

Du hast:
if(priceh != 1) return SendClientMessage(playerid, COLOR_GREY, "Das Haus kann nicht weniger als 1$ kosten.");

So KANN das Haus nur 1$ kosten. Bei mehr oder weniger, gibts ne Fehlermeldung.

lollipop · 21. Mai 2012

Also ich habs bei mir richtig wie gesagt ist für mich kein Neuland ;D
Naja ist ja auch egal danke für eure Hilfe.

Mit freundlichen Grüßen
Sven

BigBenLPs · 21. Mai 2012

Ja oh ok sry bin leicht verpeilt und im Halbschlaf...
gehe auch gleich wieder pennen ....
Nagut Bis Dann...

breadfish.de 11. März 2022