/setadmin
#1

// Fixed.
Reply
#2

Код:
CMD:setadmin(playerid, params[])
{
    new OtherPlayer, Level, Msg[128], Name[24], AdminName[24], OldLevel;
    SendAdminText(playerid, "/setadmin", params);
    if (APlayerData[playerid][LoggedIn] == true)
    {
        if (APlayerData[playerid][PlayerLevel] >= 5)
        {
            if (sscanf(params, "ui", OtherPlayer, Level)) SendClientMessage(playerid, 0xFF0000AA, "Usage: \"/setlevel [id] [level]\"");
            else
            {
                if (IsPlayerConnected(OtherPlayer))
                {
                    OldLevel = APlayerData[OtherPlayer][PlayerLevel];
                    GetPlayerName(playerid, AdminName, sizeof(AdminName));
                    GetPlayerName(OtherPlayer, Name, sizeof(Name));
                    APlayerData[OtherPlayer][PlayerLevel] = Level;
                    }
                else return SendClientMessage(playerid, 0xFF0000FF, "{FF0000}Error: {FFFFFF}That player is not online");
            }
        }
  return 1;
}
try that.
Reply
#3

**REMOVED
Reply
#4

Missed a brace:
pawn Код:
CMD:setadmin(playerid, params[])
{
    new OtherPlayer, Level, Msg[128], Name[24], AdminName[24], OldLevel;
    SendAdminText(playerid, "/setadmin", params);
    if (APlayerData[playerid][LoggedIn] == true)
    {
        if (APlayerData[playerid][PlayerLevel] >= 5)
        {
            if (sscanf(params, "ui", OtherPlayer, Level)) SendClientMessage(playerid, 0xFF0000AA, "Usage: \"/setlevel [id] [level]\"");
            else
            {
                if (IsPlayerConnected(OtherPlayer))
                {
                    OldLevel = APlayerData[OtherPlayer][PlayerLevel];
                    GetPlayerName(playerid, AdminName, sizeof(AdminName));
                    GetPlayerName(OtherPlayer, Name, sizeof(Name));
                    APlayerData[OtherPlayer][PlayerLevel] = Level;
                }
                else return SendClientMessage(playerid, 0xFF0000FF, "{FF0000}Error: {FFFFFF}That player is not online");
            }
        }
    }
    return 1;
}
Show us your APlayerData array.
Reply
#5

// FIxed.
Reply
#6

There doesn't seem anything wrong with the indentation in this code please post the exact line.
For the second and third warning, use it somewhere now or just ignore it till you find a place to use.
Reply
#7

Fixed it, thanks.
Reply
#8

Just a suggestion for you to make commands a bit easier when you have this....

[PAWN] if (APlayerData[playerid][LoggedIn] == true)
{
if (APlayerData[playerid][PlayerLevel] >= 5)
{[/PAWN

Try doing this for readability.

pawn Код:
#define CMDLEVEL(%0) if(APlayerData[playerid][PlayerLevel] < %0) return SendClientMessage(playerid, 0xFFFF0000, "You are not high enough level to use this command")

and

#define LOGGEDIN()  if(!APlayerData[playerid][LoggedIn]) return SendClientMessage(playerid, 0xFFFF0000, "You are not logged in!")

now all you need to do in your commands

cmd:setlevel(playerid, params[])
{
    LOGGEDIN();
    CMDLEVEL(5);
}
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)