Problem bei /aduty

Simon2202 · 6. April 2014

Guten Tag, Community!
Ich habe einen Aduty Befehl der soweit auch funktioniert.
Jetzt wollte ich einfügen, dass angezeigt wird welcher Supporter Level adminduty ist (Supporter,Moderator etc.)

Der Code sieht so aus:

Spoiler anzeigen

Nur bekomme ich diese Errors:

Spoiler anzeigen

Jetzt bin ich mir unsicher. Muss ich das nicht mit if angeben ? Ich hoffe ihr könnt mir helfen.
Und vorallem bitte erklärt mir was ich falsch gemacht habe. Ich versuche gerade das Scripten zu lernen und es würde mir sehr helfen, wenn ihr es mir erklärt danke im voraus!

Mustang96 · 6. April 2014

du hast bei sämtlichen abfragen immer nur ein "=" anstatt zwei drinnen; muss also folgendermaßen aussehen:
PlayerInfo[playerid][pAdmin] == XX

Kürzer und übersichtlicher:
if(strcmp(cmdtext,"/aduty", true) == 0)
{
if(PlayerInfo[playerid][pAdmin] >= 1)
{
if(ADuty[playerid] == 0)
{
ADuty[playerid] = 1;
SetPlayerHealth(playerid,1000);
SetPlayerArmour(playerid,10000);
SetPlayerColor(playerid,0x9A0000FF);
SetPlayerSkin(playerid,217);
SendClientMessage(playerid,COLOR_RED,"** Sie sind nun AD. Tippe /adoff um wieder OffDuty zu gehen. **");
GetPlayerName(playerid, sendername, sizeof(sendername));
if(PlayerInfo[playerid][pAdmin] = 1)
{
format(string, sizeof(string), "ProbeSupporter %s ist nun OnDuty.",sendername);
}
else if(PlayerInfo[playerid][pAdmin] = 2)
{
format(string,sizeof(string), "Supporter %s ist nun OnDuty.",sendername);
}
else if(PlayerInfo[playerid][pAdmin] = 3)
{
format(string,sizeof(string), "Moderator %s ist nun OnDuty.",sendername);
}
else if(PlayerInfo[playerid][pAdmin] = 4)
{
format(string,sizeof(string), "Admin %s ist nun OnDuty.",sendername);
}
else if(PlayerInfo[playerid][pAdmin] = 5)
{
format(string,sizeof(string), "Manager %s ist nun OnDuty.",sendername);
}
else if(PlayerInfo[playerid][pAdmin] = 6)
{
format(string,sizeof(string), "ServerOwner %s ist nun OnDuty.",sendername);
}
if(PlayerInfo[playerid[pAdmin] > 0)
{
SendClientMessageToAll(COLOR_RED,string);
}
}
else
{
ADuty[playerid] = 0;
SetPlayerHealth(playerid,100);
SetPlayerArmour(playerid,0);
SetPlayerColor(playerid,COLOR_WHITE);
SendClientMessage(playerid,COLOR_RED,"** Sie sind nun nichtmehr AD. **");
GetPlayerName(playerid, sendername, sizeof(sendername));
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar]);
}
}
return 1;
}

Simon2202 · 8. April 2014

Ich habe den Code jetzt so, bekomme keine Errors. aber ingame wird kein Text gesendet:

Spoiler anzeigen

JOK3R · 8. April 2014

Mach es doch in einem case

ocmd:aduty(playerid,params[])
{
if(PlayerInfo[playerid][pAdmin] >= 1)
new string[128];
if(!ADuty[playerid])
{
switch(PlayerInfo[playerid][pAdmin])
{
case 1: format(string, sizeof(string),"Probesupporter: %s ist nun im Admindienst!", sendername(playerid));
case 2: format(string, sizeof(string),"Supporter: %s ist nun im Admindienst!", sendername(playerid));
case 3: format(string, sizeof(string),"Moderator: %s ist nun im Admindienst!", sendername(playerid));
case 4: format(string, sizeof(string),"Admin: %s ist nun im Admindienst!", sendername(playerid));
case 5: format(string, sizeof(string),"Manager: %s ist nun im Admindienst!", sendername(playerid));
case 6: format(string, sizeof(string),"ServerOwner: %s ist nun im Admindienst!", sendername(playerid));
}
SendClientMessageToAll(COLOR_DGREEN,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun im Admindienst **");
SetPlayerHealth(playerid,1000),SetPlayerArmour(playerid,10000);
SetPlayerSkin(playerid,217);
return AdminDuty[playerid]=1;
}
format(string, sizeof(string),"** Supporter/Admin: %s hat sich vom Admindienst Abgemeldet! **", sencername(playerid));
SendClientMessageToAll(COLOR_RED,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun nicht mehr im Admindienst **");
SetPlayerHealth(playerid,100),SetPlayerArmour(playerid,0);
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar]);
ADuty[playerid]=0;
return 1;
}

Simon2202 · 8. April 2014

Ok, danke. Wenn ich den 3d Text über dem Namen noch hinzu fügen möchte, dann einfach hinter das format oda ?

qLx · 8. April 2014

Einfach noch nen 3DTextLabel für die playerid erstellen.

Simon2202 · 8. April 2014

Zitat von DerJonny.[47]

ocmd:aduty(playerid,params[])
{
if(PlayerInfo[playerid][pAdmin] >= 1)
new string[128];
if(!ADuty[playerid])
{
switch(PlayerInfo[playerid][pAdmin])
{
case 1: format(string, sizeof(string),"Probesupporter: %s ist nun im Admindienst!", sendername(playerid));
case 2: format(string, sizeof(string),"Supporter: %s ist nun im Admindienst!", sendername(playerid));
case 3: format(string, sizeof(string),"Moderator: %s ist nun im Admindienst!", sendername(playerid));
case 4: format(string, sizeof(string),"Admin: %s ist nun im Admindienst!", sendername(playerid));
case 5: format(string, sizeof(string),"Manager: %s ist nun im Admindienst!", sendername(playerid));
case 6: format(string, sizeof(string),"ServerOwner: %s ist nun im Admindienst!", sendername(playerid));
}
SendClientMessageToAll(COLOR_DGREEN,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun im Admindienst **");
SetPlayerHealth(playerid,1000),SetPlayerArmour(playerid,10000);
SetPlayerSkin(playerid,217);
return AdminDuty[playerid]=1;
}
format(string, sizeof(string),"** Supporter/Admin: %s hat sich vom Admindienst Abgemeldet! **", sencername(playerid));
SendClientMessageToAll(COLOR_RED,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun nicht mehr im Admindienst **");
SetPlayerHealth(playerid,100),SetPlayerArmour(playerid,0);
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar]);
ADuty[playerid]=0;
return 1;
}

Alles anzeigen

Wenn ich das einfüge hängt sich mein ganzer Kompiler auf....

YoSohai · 8. April 2014

Überprüf mal den code auf klammer fehler

AirM4X · 8. April 2014

ocmd:aduty(playerid,params[])
{
new string[128];
if(PlayerInfo[playerid][pAdmin] >= 1)
if(!ADuty[playerid]){
switch(PlayerInfo[playerid][pAdmin]){
case 1:format(string,128,"Probesupporter: %s ist nun im Admindienst!",sendername(playerid));
case 2:format(string,128,"Supporter: %s ist nun im Admindienst!",sendername(playerid));
case 3:format(string,128,"Moderator: %s ist nun im Admindienst!",sendername(playerid));
case 4:format(string,128,"Admin: %s ist nun im Admindienst!",sendername(playerid));
case 5:format(string,128,"Manager: %s ist nun im Admindienst!",sendername(playerid));
case 6:format(string,128,"ServerOwner: %s ist nun im Admindienst!",sendername(playerid));
}
SendClientMessageToAll(COLOR_DGREEN,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun im Admindienst **");
SetPlyerHealth(playerid,1000),SetPlayerArmour(playerid,1000);
SetPlayerSkin(playerid,217);
return ADuty[playerid] = 1;
}
else if(ADuty[playerid] == 1){
format(string,128,"** Supporter/Admin: %s hat sich vom Admindienst Abgemeldet! **",sendername(playerid));
SendClientMessageToAll(COLOR_RED,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun nicht mehr im Admindienst! **");
SetPlayerHealth(playerid,100),SetPlayerArmour(playerid,0);
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar]);
return ADuty[playerid] = 0;
}
return 1;
}

Mach das mal so
Du hattest da paar Fehler drinne

Simon2202 · 8. April 2014

Mhm... erstmal danke für die vielen, raschen antworten! Und vorallem danke für eure Hilfe!
Pawn ich habe jetzt deinen Code genommen..
Jedoch ist mir der Compiler auch da abgestürzt..
gezwungener maßen habe ich jeden Case mit einer Klammer ausstatten müssen:
case 1:{format(string,128,"Probesupporter: %s ist nun im Admindienst!",sendername(playerid));}

Da mir Pawno es sonst überhaupt nicht kompilieren will..
Jedoch zeigt er mir jetzt diese Errors:
Zeile (26589) : Ungültiger Befehl
Zeile (26589) : Unbekannte Variable : "ocmd_aduty"
Zeile (26589) : Ungültiger Befehl
Zeile (26589) : Zuviele Fehler in einer Zeile

Alle in der Zeile
ocmd:aduty(playerid,params[])

Weiß jemand warum? Include von ocmd ist drinnen, andere befehle mit ocmd funktionieren auch..

Bullsmith · 9. April 2014

Ich habe mal was besseres für dich entworfen ;D

/////////////////////////////////////////////////////////////////////////////////////////////////////////////
ocmd:aduty(playerid)
{
new string[128];
if(PlayerInfo[playerid][pAdmin]<1)return SendClientMessage(playerid,0xFFFFFFFF,"Du hast nicht den nötigen Adminrang");
if(ADuty[playerid]==0)
{
format(string,sizeof(string),"%s: %s ist nun im Admindienst!",Adminname(playerid),SpielerName(playerid));
SendClientMessageToAll(COLOR_DGREEN,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun im Admindienst **");
SetPlayerHealth(playerid,0x7F800000);//Godmode//
SetPlayerArmour(playerid,100);
SetPlayerSkin(playerid,217);
ADuty[playerid]=1;
return 1;
}
if(ADuty[playerid]==1)
{
format(string,sizeof(string),"%s: %s hat sich vom Admindienst Abgemeldet!",Adminname(playerid),SpielerName(playerid));
SendClientMessageToAll(COLOR_DGREEN,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun nicht mehr im Admindienst **");
SetPlayerHealth(playerid,100);//Godmode aus//
SetPlayerArmour(playerid,100);
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar])
ADuty[playerid]=0;
return 1;
}
}
///////////////////////////////////////////////////////////////////////////////////////////////////////////////
stock Adminname(playerid)
{
new rang[64];
switch(PlayerInfo[playerid][pAdmin])
{
case 0:{rang="Spieler";}
case 1:{rang="Probesupporter";}
case 2:{rang="Supporter";}
case 3:{rang="Moderator";}
case 4:{rang="Admin";}
case 5:{rang="Manager";}
case 6:{rang="ServerOwner";}
default:{rang="Fehler";}
}
return rang;
}
////////////////////////////////////////////////////////////////////////////////////////////////////////////////
stock SpielerName(playerid)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
return name;
}
////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
public OnPlayerConnect(playerid)
{
SetTimerEx("OnSpielerUpdate",5000,true,"u",playerid);//Das Hier bei OnPlayerConnect einbauen :D
return 1;
}
/////////////////////////////////////////////////////////////////////////////////////////////////////////////
forward OnSpielerUpdate(playerid);
///////////////////////////////////////////////////////////////////////////////////////////////////////////////
public OnSpielerUpdate(playerid)
{
if(ADuty[playerid]==1)
{
new kopf_string[64],kopf_color,kopf_weite;
kopf_string="";
format(kopf_string,sizeof(kopf_string),"%s",Adminname(playerid));
kopf_color=0xFF1E00FF;//Rot
kopf_weite=40;
SetPlayerChatBubble(playerid,kopf_string,kopf_color,kopf_weite,6000);
}
return 1;
}
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////
// Ich persönlich finde mit SetPlayerChatBubble sieht das schöneraus//
// Wenn es Errors gibt melde dich bei mir///
// Oder bei Fragen ///
//////////////////////

RunAway · 9. April 2014

Zitat von Bullsmith

public OnSpielerUpdate(playerid) { if(ADuty[playerid]==1) { new kopf_string[64],kopf_color,kopf_weite; kopf_string=""; format(kopf_string,sizeof(kopf_string),"%s",Adminname(playerid)); kopf_color=0xFF1E00FF;//Rot kopf_weite=40; SetPlayerChatBubble(playerid,kopf_string,kopf_color,kopf_weite,6000); } return 1; } ////////////

Zitat von Bullsmith

Ich habe mal was besseres für dich entworfen ;D

/////////////////////////////////////////////////////////////////////////////////////////////////////////////
ocmd:aduty(playerid)
{
new string[128];
if(PlayerInfo[playerid][pAdmin]<1)return SendClientMessage(playerid,0xFFFFFFFF,"Du hast nicht den nötigen Adminrang");
if(ADuty[playerid]==0)
{
format(string,sizeof(string),"%s: %s ist nun im Admindienst!",Adminname(playerid),SpielerName(playerid));
SendClientMessageToAll(COLOR_DGREEN,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun im Admindienst **");
SetPlayerHealth(playerid,0x7F800000);//Godmode//
SetPlayerArmour(playerid,100);
SetPlayerSkin(playerid,217);
ADuty[playerid]=1;
return 1;
}
if(ADuty[playerid]==1)
{
format(string,sizeof(string),"%s: %s hat sich vom Admindienst Abgemeldet!",Adminname(playerid),SpielerName(playerid));
SendClientMessageToAll(COLOR_DGREEN,string);
SendClientMessage(playerid,COLOR_YELLOW,"** Du bist nun nicht mehr im Admindienst **");
SetPlayerHealth(playerid,100);//Godmode aus//
SetPlayerArmour(playerid,100);
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar])
ADuty[playerid]=0;
return 1;
}
}
///////////////////////////////////////////////////////////////////////////////////////////////////////////////
stock Adminname(playerid)
{
new rang[64];
switch(PlayerInfo[playerid][pAdmin])
{
case 0:{rang="Spieler";}
case 1:{rang="Probesupporter";}
case 2:{rang="Supporter";}
case 3:{rang="Moderator";}
case 4:{rang="Admin";}
case 5:{rang="Manager";}
case 6:{rang="ServerOwner";}
default:{rang="Fehler";}
}
return rang;
}
////////////////////////////////////////////////////////////////////////////////////////////////////////////////
stock SpielerName(playerid)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
return name;
}
////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
public OnPlayerConnect(playerid)
{
SetTimerEx("OnSpielerUpdate",5000,true,"u",playerid);//Das Hier bei OnPlayerConnect einbauen :D
return 1;
}
/////////////////////////////////////////////////////////////////////////////////////////////////////////////
forward OnSpielerUpdate(playerid);
///////////////////////////////////////////////////////////////////////////////////////////////////////////////
public OnSpielerUpdate(playerid)
{
if(ADuty[playerid]==1)
{
new kopf_string[64],kopf_color,kopf_weite;
kopf_string="";
format(kopf_string,sizeof(kopf_string),"%s",Adminname(playerid));
kopf_color=0xFF1E00FF;//Rot
kopf_weite=40;
SetPlayerChatBubble(playerid,kopf_string,kopf_color,kopf_weite,6000);
}
return 1;
}
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////
// Ich persönlich finde mit SetPlayerChatBubble sieht das schöneraus//
// Wenn es Errors gibt melde dich bei mir///
// Oder bei Fragen ///
//////////////////////

Alles anzeigen

Dir ist bewusst, das die Chatbubble Variante einfach unnötige Ressourcen frisst?

Bullsmith · 9. April 2014

Zitat von RunAway

Dir ist bewusst, das die Chatbubble Variante einfach unnötige Ressourcen frisst?

Und was ist wenn der Server unnötige Resscourcen frisst ?
Was passiert dann ? Kommt es zu Lags auf dem Server, oder wiese siehst du von der Variante ab ?
Ist es so schlimm, resscourcen fressend zu scripten ? Wenn ich so einige GF Server anschaue, dann sehe ich doch auch bei denen, das einige Dinge darin unnötig sind. Lags sind mir dennoch nicht bei GF Server aufgefallen.

RunAway · 9. April 2014

Spoiler anzeigen

Zitat von Bullsmith

Spoiler anzeigen

Und was ist wenn der Server unnötige Resscourcen frisst ?
Was passiert dann ? Kommt es zu Lags auf dem Server, oder wiese siehst du von der Variante ab ?
Ist es so schlimm, resscourcen fressend zu scripten ? Wenn ich so einige GF Server anschaue, dann sehe ich doch auch bei denen, das einige Dinge darin unnötig sind. Lags sind mir dennoch nicht bei GF Server aufgefallen.

Spoiler anzeigen

Alles was unnötigerweise Ressourcen frisst ist unnötig, grade was in OnPlayerUpdate ist, da diese Dinge ca 3-4 mal pro Sekunde aufgerufen wird.
Bei 20 Spielern ist dies noch nicht gravierend, aber lass es dann mal 100 Spieler seien.
Und dann die Ursache für die Laggs finden und beheben, wenn man schon so unsauber arbeitet, ist dann mühsame Arbeit. Wieso sollte er dann sowas lernen und verwenden.
Dennoch ist es von jedem die seinige Entscheidung.

//edit mir ist grade aufgefallen, das du einen eigenen Timer erstellt hast und nicht OnPlayerUpdate nutzt.
Dennoch ist es schwachsinnig für sowas einen Timer für jeden Spieler zu nutzen. Wieso ncith einfach mit labels arbeiten?

Bullsmith · 9. April 2014

Zitat von RunAway

Spoiler anzeigen

Spoiler anzeigen

Alles was unnötigerweise Ressourcen frisst ist unnötig, grade was in OnPlayerUpdate ist, da diese Dinge ca 3-4 mal pro Sekunde aufgerufen wird.
Bei 20 Spielern ist dies noch nicht gravierend, aber lass es dann mal 100 Spieler seien.
Und dann die Ursache für die Laggs finden und beheben, wenn man schon so unsauber arbeitet, ist dann mühsame Arbeit. Wieso sollte er dann sowas lernen und verwenden.
Dennoch ist es von jedem die seinige Entscheidung.

Wenn du das mal richtig betrachtest, habe ich die Funktion OnPlayerUpdate nicht genommen.
Ich habe meine eigene Funktion genommen die alle 5 Sekunden geupdatet wird.

RunAway · 9. April 2014

Zitat von RunAway

//edit mir ist grade aufgefallen, das du einen eigenen Timer erstellt hast und nicht OnPlayerUpdate nutzt.
Dennoch ist es schwachsinnig für sowas einen Timer für jeden Spieler zu nutzen. Wieso ncith einfach mit labels arbeiten?

Bullsmith · 9. April 2014

Zitat von RunAway

Ich finde das 3dtextlabels einfach nur nicht so gut aussehen, sie sind so schwach, SetPlayerChatBubble siehst man besser. Ich habe bewusst OnPlayerUpdate nicht genommen, weil ich genau weis, das diese Function 50 Mal In der sekunde ausgeführt wird. Ich meide OnPlayerUpdate genauso gerne wie jeder andere, und ich sage hier auch nur meine Meinung dazu und möchte ihn gerne helfen. Man könnte ja auch einen Timer für alles erstellen, der dann alles für einen spieler Updatet

RunAway · 9. April 2014

Zitat von Bullsmith

Man könnte ja auch einen Timer für alles erstellen, der dann alles für einen spieler Updatet

Und da wäre dann wieder das besagte PRoblem
Dann leiber auf die "Schönheit" verzichten, aber dafür etwas ressourcendfreundlicher Arbeiten, denn bei vielen verschiedenen Timern, updates generell kommt es schnell zu Sekundenlaggs

Bullsmith · 9. April 2014

Zitat von RunAway

Und da wäre dann wieder das besagte PRoblem
Dann leiber auf die "Schönheit" verzichten, aber dafür etwas ressourcendfreundlicher Arbeiten, denn bei vielen verschiedenen Timern, updates generell kommt es schnell zu Sekundenlaggs

Jo, ich wollte ihn nur helfen
Ich glaube aber trotzdem, dass es diese Funktion, diese eine Funktion, gerade die Laggs ausführt
Ist ja nur das eine
Andererseits möchte ich mich noch einmal für deine Aufklärung bedanken. Nun kenne ich die Nebenwirkungen
Danke

Simon: Nimm einfach das was dir am meisten entspricht

Schönen Abend noch

Simon2202 · 9. April 2014

Danke, ich werde zuhause schaun ob alles klappt danke!

breadfish.de 12. März 2022