Hilfe bei ocmd Befehl

wie soll die Client Message auch ein format (%s) auslesen?
vielleicht mal formatieren?

Hallo,
du benutzt SendClientMessageToAll falsch .....
Du kannst nicht direkt in der Methode formatieren.
Nutze dazu format

MFG

Zitat von -iClouD

Hallo liebe Community, ich habe ein Problem. Wen ich den Befehl /aduty eingebe, dann restartet sich einfach mein Server neu.

ocmd:aduty(playerid,params[])
{
if(IsPlayerAnAdmin(playerid,1))
{
SendClientMessageToAll(COLOR_HELLBLAU,"Admin %s ist nun im Dienst!");
SetPlayerHealth(playerid,100);
SetPlayerArmour(playerid,100);
SetPlayerSkin(playerid,279);
SetPlayerColor(playerid,COLOR_RED);
return 1;
}
return 1;
}

Alles anzeigen

Du fragst nirgends den Namen vom Admin ab, daher ein string formatieren.
new string[128];
format(string, sizeof(string), "Admin %s ist nun im Dienst", SpielerName(playerid));
SendClientMessageToAll(COLOR_HELLBLAU, string);

Falls du Errors wegen SpielerName(playerid) bekommen solltest, das iwo im Script einfügen:

stock SpielerName(playerid)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
return name;
}

Lies in meinem oberen Beitrag, hatte es eben rein editiert. // Ah ok erledigt

stock SpielerName(playerid)
{
new name[MAX_PLAYERS];
GetPlayerName(playerid,name,sizeof(name));
return name;
}

MAX_PLAYERS???!!!

mach daraus bitte MAX_PLAYER_NAME

stock SpielerName(playerid)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
return name;
}

Zitat von s h a i N

stock SpielerName(playerid)
{
new name[MAX_PLAYERS];
GetPlayerName(playerid,name,sizeof(name));
return name;
}

MAX_PLAYERS???!!!

mach daraus bitte MAX_PLAYER_NAME

stock SpielerName(playerid)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
return name;
}

Alles anzeigen

Hoppala, habs mal editiert, ja jeder macht Fehler

ocmd:aduty(playerid,params[])
{
	if(IsPlayerAnAdmin(playerid,1))
	{
		SendClientMessageToAll(COLOR_HELLBLAU,"Admin %s ist nun im Dienst!");
		SetPlayerHealth(playerid,100);
		SetPlayerArmour(playerid,100);
		SetPlayerSkin(playerid,279);
		SetPlayerColor(playerid,COLOR_RED);
		return 1;
	}
        else if
        {
                SendClientMessageToAll(COLOR_HELLBLAU,"Admin %s ist nun nicht mehr im Dienst!");
		SetPlayerHealth(playerid,100); //Wie viel leben er bekommen soll
		SetPlayerArmour(playerid,100); // wie viel armor er danach bekommen soll
		SetPlayerSkin(playerid,279); //Skin wo er wieder bekommen soll.
		SetPlayerColor(playerid,COLOR_RED); //Farbe wo er wieder bekommen soll
        }
	return 1;
}

Bitte noch dementsprechend ändern!
Also mit deinem string dann halt einfach in beiden umändern

Zitat von Marv_Checker

SQL

ocmd:aduty(playerid,params[])
{
	if(IsPlayerAnAdmin(playerid,1))
	{
		SendClientMessageToAll(COLOR_HELLBLAU,"Admin %s ist nun im Dienst!");
		SetPlayerHealth(playerid,100);
		SetPlayerArmour(playerid,100);
		SetPlayerSkin(playerid,279);
		SetPlayerColor(playerid,COLOR_RED);
		return 1;
	}
        else if
        {
                SendClientMessageToAll(COLOR_HELLBLAU,"Admin %s ist nun nicht mehr im Dienst!");
		SetPlayerHealth(playerid,100); //Wie viel leben er bekommen soll
		SetPlayerArmour(playerid,100); // wie viel armor er danach bekommen soll
		SetPlayerSkin(playerid,279); //Skin wo er wieder bekommen soll.
		SetPlayerColor(playerid,COLOR_RED); //Farbe wo er wieder bekommen soll
	return 1;
}

Alles anzeigen

Bitte noch dementsprechend ändern!
Also mit deinem string dann halt einfach in beiden umändern

Völliger Schwachsin, wenn er Admin ist, geht er in den Dienst, wenn er z.B. Leader ist dann geht er aus dem Dienst? Schwachsinn...

//oben ins Script
new bool:aDuty[MAX_PLAYERS];
ocmd:aduty(playerid,params[])
{
if(IsPlayerAnAdmin(playerid,1))
{
new string[128];
if(!aDuty[playerid]) {
format(string,sizeof(string),"Admin %s ist nun im Dienst!", SpielerName(playerid));
SendClientMessageToAll(COLOR_HELLBLAU,string);
SetPlayerHealth(playerid,100);
SetPlayerArmour(playerid,100);
SetPlayerSkin(playerid,279);
SetPlayerColor(playerid,COLOR_RED);
aDuty[playerid] = true;
} else {
format(string,sizeof(string),"Admin %s ist nun nicht mehr im Dienst!", SpielerName(playerid));
SendClientMessageToAll(COLOR_HELLBLAU,string);
SetPlayerHealth(playerid,100);
SetPlayerArmour(playerid,100);
SetPlayerSkin(playerid,279);
SetPlayerColor(playerid,COLOR_RED);
aDuty[playerid] = false;
}
}
return true;
}

Marv_Checker, also gebe ich /aduty ein, gehe in den Dienst und wenn ich kein Admin bin, gehe ich aus dem Dienst?

Entschuldigung, habe mich verschrieben kann ja mal passieren oder?

Zitat von Marv_Checker

Entschuldigung, habe mich verschrieben kann ja mal passieren oder?

Jap, kann es, deswegen wurdest du drauf hingewießen