Makeleader geht nicht

Tomsen · 20. November 2012

Hey,

ich bin gerade dabei ein Fraktionssystem zu scripten.
Ich hab den Befehl hier aber er wird nicht ausgeführt, sobald alles korrekt ist, wie Adminrang, Schreibweiße etc, wird nicht das ausgeführt was eigentlich ausgeführt werden sollte:

Code

ocmd:makeleader(playerid,params[])
{
	new pID,frakid;
	if(!IsPlayerAnAdmin(playerid,2000)) return SendClientMessage(playerid,rot,"[Fehler]: Du hast nicht das nötige Adminlevel!");
	if(sscanf(params,"ui",pID,frakid)) return SendClientMessage(playerid,rot,"Benutze: /makeleader [ID/Name] [Fraktionsid] {FFFF00} /frakids");
	if(frakid<0 || frakid>1) return SendClientMessage(playerid,rot,"[Fehler]: Fraktionsid kann nicht niedriger als 0 und nicht höher als 1 sein!");
	if(frakid==0) //Zivilisten
	{
		SpielerInfo[pID][pFraktion] = 0;
		SpielerInfo[pID][pFRang] = 0;
		SpielerInfo[pID][pLeader] = 0;
		new adminst[128],playerst[128];
		format(adminst,sizeof(adminst),"[Server]: Du hast den Spieler %s [ID:%d] aus seiner Fraktion geschmissen!",SpielerName(pID),pID);
		format(playerst,sizeof(playerst),"[Server]: Du wurdest von Admin %s aus deiner Fraktion geschmissen!",SpielerName(playerid));
		return 1;
	}
	if(frakid==1) //Ordnungsamt
	{
		SpielerInfo[pID][pFraktion] = 1;
		SpielerInfo[pID][pFRang] = 6;
		SpielerInfo[pID][pFRang] = 1;
		new adminst[128],playerst[128];
		format(adminst,sizeof(adminst),"[Server]: Du hast den Spieler %s [ID:%d] zum Leader der Fraktion Ordnungsamt ernannt!",SpielerName(pID),pID);
		format(playerst,sizeof(playerst),"[Server]: Du wurdest von Admin %s zum Leader des Ordnungsamtes ernannt!",SpielerName(playerid));
		return 1;
	}
	return 1;
}

Alles anzeigen

Aber wieso?

Schonmal danke im Vorraus.

- Tomsen

Ditti · 20. November 2012

Nun, wenn du sagst, dass nicht das richtige ausgeführt wird, kannst du das benennen, was fälschlicherweise ausgeführt wird? Also ab welcher Zeile es sozusagen falsch wird.

Mal als Hinweis am Rande: schau dir mal switch an - ist gut, um eine einzelne Variable auf Werte zu testen: http://wiki.amxmodx.org/Pawn_Tutorial#Switch_Statements

RayJohnson · 20. November 2012

Zitat von Tomsen

if(frakid<0 frakid>1)

Ich würde empfehlen es so zu machen:
~~if(frakid == 0 frakid >=1)~~

~~if(frakid gleich 0 oder frakid größer oder gleich 1)~~

//edit: Quatsch, tut mir leid.
Ich muss mich wohl hinlegen, das was du dort hast ist bereits richtig.

Sag uns am besten, ab welcher Zeile ungefähr oder nachdem du was eingegeben hast nichts mehr kommt/erscheint/passiert.

Tomsen · 21. November 2012

Das was bei if(frakid==0 bzw 1 steht wird nicht ausgeführt. Also er erkennt die Frakid nicht oder sowas.

Mit Switch hat ich es auch schon, nur hat das nicht wirklich funktioniert.

- Tom_Reese

Beavis · 21. November 2012

mach mal eine else if abfrage bei dein oamt

Kaliber · 21. November 2012

Schreibe das mal so:

ocmd:makeleader(playerid,params[])
{
new pID,frakid;
if(!IsPlayerAnAdmin(playerid,2000)) return SendClientMessage(playerid,rot,"[Fehler]: Du hast nicht das nötige Adminlevel!");
if(sscanf(params,"ui",pID,frakid)) return SendClientMessage(playerid,rot,"Benutze: /makeleader [ID/Name] [Fraktionsid] {FFFF00} /frakids");
if(frakid<0 || frakid>1) return SendClientMessage(playerid,rot,"[Fehler]: Fraktionsid kann nicht niedriger als 0 und nicht höher als 1 sein!");
if(frakid==0) //Zivilisten
{
SpielerInfo[pID][pFraktion] = 0;
SpielerInfo[pID][pFRang] = 0;
SpielerInfo[pID][pLeader] = 0;
new playerst[128];
format(playerst,sizeof(playerst),"[Server]: Du hast den Spieler %s [ID:%d] aus seiner Fraktion geschmissen!",SpielerName(pID),pID);
SendClientMessage(playerid,rot,playerst);
format(playerst,sizeof(playerst),"[Server]: Du wurdest von Admin %s aus deiner Fraktion geschmissen!",SpielerName(playerid));
SendClientMessage(pID,rot,playerst);
}
else if(frakid==1) //Ordnungsamt
{
SpielerInfo[pID][pFraktion] = 1;
SpielerInfo[pID][pFRang] = 6;
SpielerInfo[pID][pFRang] = 1;
new playerst[128];
format(playerst,sizeof(playerst),"[Server]: Du hast den Spieler %s [ID:%d] zum Leader der Fraktion Ordnungsamt ernannt!",SpielerName(pID),pID);
SendClientMessage(playerid,rot,playerst);
format(playerst,sizeof(playerst),"[Server]: Du wurdest von Admin %s zum Leader des Ordnungsamtes ernannt!",SpielerName(playerid));
SendClientMessage(pID,rot,playerst);
}
return 1;
}

mfg.

Tomsen · 22. November 2012

Bringt auch nichts.

Wenn ich als Frakid 0 oder 1 eingebe passiert garnichts, obwohl es ja dann Zivilist oder Ordnungsamt setzen sollte.
Wenn ich eine falsch Zahl eingebe oder den Command nicht vollständig abschicke kommen die Errors. Das muss irgentwo was mit Frakid zu tun haben, aber ich finde den Fehler nicht.

- Tomsen

//Edit:
Kaliber: Danke ich versuche es

//Edit 2: Ich hab nicht ernsthaft vergessen die Message abzuschicken oder?
Meine Güte ich war wohl zu müde gestern xD
Jetzt funktioniert es. Danke.

//Edit3:

Hab noch eine Frage. Hab eine Funktion gemacht die Abfragt ob man in der Frak ist. Hab diese dann auf eine Schranke angewendet, aber man kann sie immer öffnen. wieso?

Code

stock IsPlayerInFrak(playerid,frakid)
{
	if(mysql_GetInt("accounts","Fraktion","Name",SpielerInfo[playerid][pName])== frakid)return 1;
	return 0;
}

Code

if(strcmp("/tor",cmdtext,true,10) == 0)
	{
		if(IsPlayerInFrak(playerid,1))
		{
			if(IsPlayerInRangeOfPoint(playerid,10,2287.35, -2362.53, 13.16)) 
			{
				if(schranke==0)
				{
					MoveDynamicObject(indusschranke,2287.350000001, -2362.53, 13.16, 0.1, 0, 0, 44.00);
					SendClientMessage(playerid,grau,"Bitte schließe die Schranke auch wieder!");
					schranke=1;
					return 1;
				}
				else
				{
					MoveDynamicObject(indusschranke,2287.350000001, -2362.53, 13.16, 0.1, 0, -90, 44.00);
					schranke=0;
					return 1;
				}
			}
		} else SendClientMessage(playerid,rot,"[Fehler]: Du hast keine Fernbedienung!");
 		return 1;
	}

Alles anzeigen

Dann noch eine andere Sache, die Schranke fährt nicht langsam auf, sondern ich hab es nur so hinbekommen das es Ruckartig geht. Also man gibt den Command ein und die ist oben.
Wie kann man das ändern?

- Tomsen

breadfish.de 11. März 2022