/makeadmin hilfe

*facepalm* Falls du nicht so ein guter Scripter bist, so verwende bitte einen anderen Command-Erzeuger, in meinem Falle wäre das Ocmd, weil Ocmd schneller als die anderen sind (strcmp, dcmd, zcmd, rcmd). Bitte downloade dir das und dazu noch das sscanf stock, findest du im offizielen englischen sa-mp wiki.


Um zum Thema zu kommen:

ocmd:makeadmin(playerid,params[])//Ocmd ist der erzeuger des Befehls, playerid ist die ID und params sind die parameter
{
new pID, rang, string[128];
if(sscanf(params,"ui",pID,rang))return SendClientMessage(playerid,COLOR_RED,"Fehler: /makeadmin [playerid] [rang]");
format(string,sizeof(string),"%s wurde von %s zum Administrator Level %i ernannt. Herzlichen Glückwunsch!",GetSpielerName(pID),GetSpielerName(playerid),rang);
if(IsPlayerConnected(pID))
{
PlayerInfo[playerid][Admin] = rang;
SendClientMessageToAll(string);

}
return 1;
}

Irgendwie lässt du dir jeden Befehl machen oder?
Nunja.. ich habs jz nur in dcmd..

cmd_makeadmin(playerid,params[])
{
if(GetPVarInt(playerid,"Adminlevel") == 5)
{
new pID, rank, string[128], string2[128];
if(sscanf(params,"ui",pID,rank))return SendClientMessage(playerid,COLOR_HELLBLAU,"Benutze: /makeadmin [ID] [Rank]");
if(rank<0||rank>5)return SendClientMessage(playerid,COLOR_DUNKELROT,"Bitte einen gültigen Rang angeben.");
if(IsPlayerConnected(pID))
{
format(string,sizeof(string),"Sie haben (%d)%s auf Rank %i befördert!",pID,SpielerName(pID), rank);
SendClientMessage(playerid, COLOR_GRÜN, string);
format(string2, sizeof(string2),"Herlichen Glückwunsch! (%d)%s hat Ihnen Adminrank %i gegeben!",playerid,SpielerName(playerid), rank);
SendClientMessage(pID, COLOR_GRÜN, string2);
SetPVarInt(pID,"Adminlevel",rank);
SpielerSpeichern(pID);
}
}
return 1;
}

wen dann richtig

ocmd:makeadmin(playerid,params[])//Ocmd ist der erzeuger des Befehls, playerid ist die ID und params sind die parameter
{
new pID, rang, string[128];
if(Spieler[playerid][Adminlevel] == 2)
{
if(sscanf(params,"ui",pID,rang))return SendClientMessage(playerid,COLOR_RED,"Fehler: /makeadmin [playerid] [rang]");
format(string,sizeof(string),"%s wurde von %s zum Administrator Level %i ernannt. Herzlichen Glückwunsch!",GetSpielerName(pID),GetSpielerName(playerid),rang);
if(IsPlayerConnected(pID))
{
PlayerInfo[playerid][Admin] = rang;
SendClientMessageToAll(string);

}
}
else
{
SendClientMessage(playerid,COLOR_RED,"Nicht das Adminlevel oder so");
}
return 1;
}

dann hier

if(strcmp("/makeadmin",cmdtext,true)==0)
{
{
new pID, rang, string[128];
if(Spieler[playerid][Adminlevel] == 2)
{
if(sscanf(params,"ui",pID,rang))return SendClientMessage(playerid,COLOR_RED,"Fehler: /makeadmin [playerid] [rang]");
format(string,sizeof(string),"%s wurde von %s zum Administrator Level %i ernannt. Herzlichen Glückwunsch!",GetSpielerName(pID),GetSpielerName(playerid),rang);
if(IsPlayerConnected(pID))
{
PlayerInfo[playerid][Admin] = rang;
SendClientMessageToAll(string);

}
}
else
{
SendClientMessage(playerid,COLOR_RED,"Nicht das Adminlevel oder so");
}
}
return 1;
}

Weis nicht ob's geht

aha, wenn du scripten könntest, wieso amchste dann enn makeadmin nciht selbst ??
aber um beim thema zu bleiben...
falls du selber schonmal cmds mit parametern gemacht hast, wirst du feststellen dass es um einiges einfacher wäre, wenn du dir einfach mal dcmd, zcmd, ocmd oder sowas runterladen würdest.

und nein das von raphael kann nicht gehen...
mit strcmp kann man nicht einfach "params" verwenden... genau das is das problem dabei

Zitat von D.Fence

alsolangsam fühle ich michverarcht wieoft muss man den hier sage das es kein ocmd oder sonstwas sein soll sonderngenau so wie ich es oben gepostet habe

Du musst einfach nur das was hier im Befehl drin ist in deinem Befehl einfügen da gibt es nicht viel zu verarschen! bisschen mit denken

dann brauchst du die OCMD Include

Hey, pass mal auf! Du schreibst uns keine Fehler, also können wir das wohl kaum wissen, was falsch ist!
Aber bitte, oben ist der Ausgang...

Naya also erstens brauchst du nicht erwarten, dass dir jemand den ganzen Code macht,

Und im allgemeinen musst du garnicht dein komplettes Script umstellen, du kannst auch strcmp.für cmds ohne Parameter hernehmen und ocmd für Befehle mit.

Dafür musst du nicht alles umstellen

D.Fence:

Ganz oben:
#define dcmd(%1,%2,%3) if (!strcmp((%3)[1], #%1, true, (%2)) && ((((%3)[(%2) + 1] == '\0') && (dcmd_%1(playerid, ""))) || (((%3)[(%2) + 1] == ' ') && (dcmd_%1(playerid, (%3)[(%2) + 2]))))) return 1

Unter OnPlayerCommandText:
dcmd(makeadmin, 9, cmdtext);

Irgendwo in keinem Public:
dcmd_makeadmin(playerid,params[])//Ocmd ist der erzeuger des Befehls, playerid ist die ID und params sind die parameter
{
new pID, rang, string[128];
if(sscanf(params,"ui",pID,rang))return SendClientMessage(playerid,COLOR_RED,"Fehler: /makeadmin [playerid] [rang]");
format(string,sizeof(string),"%s wurde von %s zum Administrator Level %i ernannt. Herzlichen Glückwunsch!",GetSpielerName(pID),GetSpielerName(playerid),rang);
if(IsPlayerConnected(pID))
{
PlayerInfo[pID][Admin] = rang;
SendClientMessageToAll(string);

}
return 1;
}

Bitte, danke ...

Hier noch die Namensfunktion
forward GetSpielerName(playerid);
public GetSpielerName(playerid)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, sizeof(name));
return name;
}

Ja du musst noch den sscanf stock einfügen.

(irgendwo im Script am besten unten weil du den nicht mehr bearbeiten brauchst und der lang ist)