Problem mit admin duty(Admin)

Flaw · 24. November 2015

Guten Tag!
Ich werde mich hier wie immer kurz fassen...

Wenn ich meinen aduty Command InGame ausführen möchte:
ocmd:aduty(playerid, params[])
{
new string[256];
switch(pInfo[playerid][Adminlevel])
{
case 1:
{
if(Aduty[playerid] == false)
{
Aduty[playerid] = true;
SendClientMessage(playerid,Rot,"<< Du bist nun im Test-Supporter-Dienst");
format(string,sizeof(string),"<< %s ist nun als Test-Supporter OnDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strins(pName,"[WESTSIDE]",0);
SetPlayerName(playerid,pName);
}
else
{
Aduty[playerid] = false;
SendClientMessage(playerid,Rot,"<< Du bist nicht länger als Test-Supporter im Dienst");
format(string,sizeof(string),"<< %s ist nun als Test-Supporter OffDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun nicht mehr für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strdel(pName,0,10);
SetPlayerName(playerid,pName);
}
}
case 2:
{
if(Aduty[playerid] == false)
{
Aduty[playerid] = true;
SendClientMessage(playerid,Rot,"<< Du bist nun im Supporter-Dienst >>");
format(string,sizeof(string),"<< %s ist nun als Supporter OnDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strins(pName,"[WESTSIDE]",0);
SetPlayerName(playerid,pName);
}
else
{
Aduty[playerid] = false;
SendClientMessage(playerid,Rot,"<< Du bist nicht länger als Supporter im Dienst");
format(string,sizeof(string),"<< %s ist nun als Supporter OffDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun nicht mehr für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strdel(pName,0,10);
SetPlayerName(playerid,pName);
}
}
case 3:
{
if(Aduty[playerid] == false)
{
Aduty[playerid] = true;
SendClientMessage(playerid,Rot,"<< Du bist nun im Test-Admin-Dienst >>");
format(string,sizeof(string),"<< %s ist nun als Test-Admin OnDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strins(pName,"[WESTSIDE]",0);
SetPlayerName(playerid,pName);
}
else
{
Aduty[playerid] = false;
SendClientMessage(playerid,Rot,"<< Du bist nicht länger als Test-Admin im Dienst");
format(string,sizeof(string),"<< %s ist nun als Test-Admin OffDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun nicht mehr für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strdel(pName,0,10);
SetPlayerName(playerid,pName);
}
}
case 4:
{
if(Aduty[playerid] == false)
{
Aduty[playerid] = true;
SendClientMessage(playerid,Rot,"<< Du bist nun im Admin-Dienst >>");
format(string,sizeof(string),"<< %s ist nun als Admin OnDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strins(pName,"[WESTSIDE]",0);
SetPlayerName(playerid,pName);
}
else
{
Aduty[playerid] = false;
SendClientMessage(playerid,Rot,"<< Du bist nicht länger als Admin im Dienst");
format(string,sizeof(string),"<< %s ist nun als Admin OffDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun nicht mehr für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strdel(pName,0,10);
SetPlayerName(playerid,pName);
}
}
case 5:
{
if(Aduty[playerid] == false)
{
Aduty[playerid] = true;
SendClientMessage(playerid,Rot,"<< Du bist nun im Head-Admin-Dienst >>");
format(string,sizeof(string),"<< %s ist nun als Head-Admin OnDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strins(pName,"[WESTSIDE]",0);
SetPlayerName(playerid,pName);
}
else
{
Aduty[playerid] = false;
SendClientMessage(playerid,Rot,"<< Du bist nicht länger als Head-Admin im Dienst");
format(string,sizeof(string),"<< %s ist nun als Head-Admin OffDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun nicht mehr für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strdel(pName,0,10);
SetPlayerName(playerid,pName);
}
}
case 6:
{
if(Aduty[playerid] == false)
{
Aduty[playerid] = true;
SendClientMessage(playerid,Rot,"<< Du bist nun im Communitymanager-Dienst >>");
format(string,sizeof(string),"<< %s ist nun als Communitymanager OnDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strins(pName,"[WESTSIDE]",0);
SetPlayerName(playerid,pName);
}
else
{
Aduty[playerid] = false;
SendClientMessage(playerid,Rot,"<< Du bist nicht länger als Communitymanager im Dienst");
format(string,sizeof(string),"<< %s ist nun als Communitymanager OffDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun nicht mehr für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strdel(pName,0,10);
SetPlayerName(playerid,pName);
}
}
case 7:
{
if(Aduty[playerid] == false)
{
Aduty[playerid] = true;
SendClientMessage(playerid,Rot,"<< Du bist nun im Projektleiter-Dienst >>");
format(string,sizeof(string),"<< %s ist nun als Projektleiter OnDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strins(pName,"[WESTSIDE]",0);
SetPlayerName(playerid,pName);
}
else
{
Aduty[playerid] = false;
SendClientMessage(playerid,Rot,"<< Du bist nicht länger als Projektleiter im Dienst");
format(string,sizeof(string),"<< %s ist nun als Projektleiter OffDuty >>",SpielerName(playerid));
SendClientMessageToAll(Rot,string);
SendClientMessageToAll(Rot,"<< Dieser ist nun nicht mehr für Fragen verfügbar >>");
new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
strdel(pName,0,10);
SetPlayerName(playerid,pName);
}
}
}
return 1;
}

kommt GARNICHTS ingame,
Wegen was kommt das?
Defines sind auch schon getätigt worden.

Rexo · 24. November 2015

Auf den ersten Blick tippe ich mal das die Variable "pInfo[playerid][Adminlevel]" 0 ist. Bau sonst noch ein case 0 ein das den Spieler anzeigt "Du bist kein Admin" um auf Nummer sicher zu gehen.

Flaw · 24. November 2015

Aber, überall wo ich case benutzt habe, geht es nicht. Hast du gerade mal Zeit? und könntest ein paar Fehler beheben? Ich weiß ja nicht, ob du dazu bereit wärest ^,.^ skype: L3goduplo

alva · 24. November 2015

Schreib dir doch eine Funktion die den Rangnamen ausgibt:
Wird vielleicht nicht deinen Fehler beheben, jedoch ist dein Code leserlicher und das präventiert Fehler.

C

stock GetAdminRankName(adminlevel)
{
	new returnname[30];
	if(adminlevel == 0)
	{
		format(returnname, 30, "Spieler");
	}
	if(adminlevel == 1)
	{
		format(returnname, 30, "Supporter");
	}
		//[...]
	return returnname;
}
ocmd:aduty(playerid, params[])
{
	#pragma unused params
    new string[256];
	if(!pInfo[playerid][Adminlevel])return SendClientMessage(playerid, -1, "Du bist kein Administrator");
	if(Aduty[playerid]
	{
		Aduty[playerid] = false;
		SendClientMessage(playerid,Rot,"<< Du bist nicht länger als Admin angemeldet.");
		format(string,sizeof(string),"<< %s ist nun als %s OffDuty >>",SpielerName(playerid), GetAdminRankName(pInfo[playerid][Adminlevel]);
		SendClientMessageToAll(Rot,string);
		new pName[MAX_PLAYER_NAME];
		GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
		strdel(pName,0,10);
		SetPlayerName(playerid,pName);
	}
	else
	{
		Aduty[playerid] = true;
		SendClientMessage(playerid,Rot,"<< Du bist nun als Admin angemeldet.");
		format(string,sizeof(string),"<< %s ist nun als %s OnDuty >>",SpielerName(playerid), GetAdminRankName(pInfo[playerid][Adminlevel]);
		SendClientMessageToAll(Rot,string);
		new pName[MAX_PLAYER_NAME];
		GetPlayerName(playerid,pName,MAX_PLAYER_NAME);
		strins(pName,"[WESTSIDE]",0);
		SetPlayerName(playerid,pName);
	}
    return 1;
}

Alles anzeigen

Versuchs mal so in der Art, ansonsten benutz Debugmessages um deine Variablen zu überprüfen, zB. bevor geprüft wird ob jemand Admin ist: printf("[aduty] %s -> Adminrang %d"", SpielerName(playerid, pInfo[playerid][Adminlevel]);

breadfish.de 12. März 2022