Need help with admin view.
#1

I wan't to create a /team commands to view online admin and helpers in one command in one time.
Finally i ended up with 6 errors.Can anyone help me fix this commands?

pawn Код:
CMD:team(playerid, params[])
{
    if(IsPlayerConnected(playerid))
    {
        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Admin Team] {a7c2e6}- Administrators:");
        for(new i = 0; i < MAX_PLAYERS; i++)
        {
            if(IsPlayerConnected(i))
            {
                if(PlayerInfo[i][pAdmin] >= 1 && PlayerInfo[i][pAdmin] <= 6 )
                {
                    new adminfo[64], sendername[MAX_PLAYER_NAME], string[128], helperinfo[64];
                    if(PlayerInfo[i][pAdmin] == 5) { adminfo = "Owner"; }
                    else if(PlayerInfo[i][pAdmin] == 4) { adminfo = "Head Of Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 3) { adminfo = "Lead Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 2) { adminfo = "Basic Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 1) { adminfo = "Trial Admin"; }
                    else if(PlayerInfo[i][pHelper] == 1) { helperinfo = "Helper"; }
                    else { adminfo = "Admin"; }
                    GetPlayerName(i, sendername, sizeof(sendername));
                    if(AdminDuty[i] == 0)
                    {
                        format(string, 128, "{e1e3e6} - %s %s", adminfo, sendername);
                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                    }
                    else
                    {
                        format(string, 128, "{7bd25c} - %s %s [ADMIN DUTY]", adminfo, sendername);
                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                    }
                    else
                    {
                        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Helper Team] {a7c2e6}- Helpers:");
                        for(new i = 0; i < MAX_PLAYERS; i++)
                        {
                            if(IsPlayerConnected(i))
                            {
                                if(PlayerInfo[i][pHelper] >= 1 && PlayerInfo[i][pHelper] <= 2)
                                {
                                    new sendername[MAX_PLAYER_NAME], string[128], helperinfo[64];
                                    if(PlayerInfo[i][pHelper] == 1) { adminfo = "Helper"; }
                                    else { helperinfo = "Helper"; }
                                    GetPlayerName(i, sendername, sizeof(sendername));
                                    if(AdminDuty[i] == 0)
                                    {
                                        format(string, 128, "{e1e3e6} - %s %s", helperinfo, sendername);
                                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                                    }
                                    else
                                    {
                                        format(string, 128, "{7bd25c} - %s %s [HELPER DUTY]", helperinfo, sendername);
                                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    return 1;
}
And error code

Код:
 
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(1612) : error 017: undefined symbol "HeadshotCheck"
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(1611) : warning 203: symbol is never used: "blahz"
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(1611) : warning 203: symbol is never used: "blahy"
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(1611) : warning 203: symbol is never used: "blahx"
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3190) : error 029: invalid expression, assumed zero
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3193) : warning 219: local variable "i" shadows a variable at a preceding level
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3199) : warning 219: local variable "sendername" shadows a variable at a preceding level
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3199) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3199) : warning 219: local variable "helperinfo" shadows a variable at a preceding level
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3220) : warning 217: loose indentation
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3224) : warning 217: loose indentation
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3224) : error 029: invalid expression, assumed zero
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3224) : error 017: undefined symbol "cmd_report"
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3224) : error 029: invalid expression, assumed zero
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(3224) : fatal error 107: too many error messages on one line

Compilation aborted.Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


6 Errors.
Reply
#2

You're missing a closed bracket above "return 1;".
pawn Код:
CMD:team(playerid, params[])
{
    if(IsPlayerConnected(playerid))
    {
        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Admin Team] {a7c2e6}- Administrators:");
        for(new i = 0; i < MAX_PLAYERS; i++)
        {
            if(IsPlayerConnected(i))
            {
                if(PlayerInfo[i][pAdmin] >= 1 && PlayerInfo[i][pAdmin] <= 6 && PlayerInfo[i][pHelper] >= 1 && PlayerInfo[i][pHelper] <= 2)
                {
                    new adminfo[64], sendername[MAX_PLAYER_NAME], string[128], helperinfo[64];
                    if(PlayerInfo[i][pAdmin] == 5) { adminfo = "Owner"; }
                    else if(PlayerInfo[i][pAdmin] == 4) { adminfo = "Head Of Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 3) { adminfo = "Lead Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 2) { adminfo = "Basic Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 1) { adminfo = "Trial Admin"; }
                    else if(PlayerInfo[i][pHelper] == 1) { helperinfo = "Helper"; }
                    else { adminfo = "Admin"; }
                    GetPlayerName(i, sendername, sizeof(sendername));
                    if(AdminDuty[i] == 0)
                    {
                        format(string, 128, "{e1e3e6} - %s %s", adminfo, sendername);
                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                    }
                    else
                    {
                        format(string, 128, "{7bd25c} - %s %s [ADMIN DUTY]", adminfo, sendername);
                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                    }
                    else
                    {
                        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Helper Team] {a7c2e6}- Helpers:");
                        for(new i = 0; i < MAX_PLAYERS; i++)
                        {
                            if(IsPlayerConnected(i))
                            {
                                if(PlayerInfo[i][pHelper] >= 1 && PlayerInfo[i][pHelper] <= 2)
                                {
                                    new sendername[MAX_PLAYER_NAME], string[128], helperinfo[64];
                                    if(PlayerInfo[i][pHelper] == 1) { adminfo = "Helper"; }
                                    else { helperinfo = "Helper"; }
                                    GetPlayerName(i, sendername, sizeof(sendername));
                                    if(AdminDuty[i] == 0)
                                    {
                                        format(string, 128, "{e1e3e6} - %s %s", helperinfo, sendername);
                                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                                    }
                                    else
                                    {
                                        format(string, 128, "{7bd25c} - %s %s [HELPER DUTY]", helperinfo, sendername);
                                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }
    return 1;
}
Reply
#3

Quote:
Originally Posted by Dwane
Посмотреть сообщение
You're missing a closed bracket above "return 1;".
pawn Код:
CMD:team(playerid, params[])
{
    if(IsPlayerConnected(playerid))
    {
        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Admin Team] {a7c2e6}- Administrators:");
        for(new i = 0; i < MAX_PLAYERS; i++)
        {
            if(IsPlayerConnected(i))
            {
                if(PlayerInfo[i][pAdmin] >= 1 && PlayerInfo[i][pAdmin] <= 6 && PlayerInfo[i][pHelper] >= 1 && PlayerInfo[i][pHelper] <= 2)
                {
                    new adminfo[64], sendername[MAX_PLAYER_NAME], string[128], helperinfo[64];
                    if(PlayerInfo[i][pAdmin] == 5) { adminfo = "Owner"; }
                    else if(PlayerInfo[i][pAdmin] == 4) { adminfo = "Head Of Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 3) { adminfo = "Lead Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 2) { adminfo = "Basic Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 1) { adminfo = "Trial Admin"; }
                    else if(PlayerInfo[i][pHelper] == 1) { helperinfo = "Helper"; }
                    else { adminfo = "Admin"; }
                    GetPlayerName(i, sendername, sizeof(sendername));
                    if(AdminDuty[i] == 0)
                    {
                        format(string, 128, "{e1e3e6} - %s %s", adminfo, sendername);
                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                    }
                    else
                    {
                        format(string, 128, "{7bd25c} - %s %s [ADMIN DUTY]", adminfo, sendername);
                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                    }
                    else
                    {
                        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Helper Team] {a7c2e6}- Helpers:");
                        for(new i = 0; i < MAX_PLAYERS; i++)
                        {
                            if(IsPlayerConnected(i))
                            {
                                if(PlayerInfo[i][pHelper] >= 1 && PlayerInfo[i][pHelper] <= 2)
                                {
                                    new sendername[MAX_PLAYER_NAME], string[128], helperinfo[64];
                                    if(PlayerInfo[i][pHelper] == 1) { adminfo = "Helper"; }
                                    else { helperinfo = "Helper"; }
                                    GetPlayerName(i, sendername, sizeof(sendername));
                                    if(AdminDuty[i] == 0)
                                    {
                                        format(string, 128, "{e1e3e6} - %s %s", helperinfo, sendername);
                                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                                    }
                                    else
                                    {
                                        format(string, 128, "{7bd25c} - %s %s [HELPER DUTY]", helperinfo, sendername);
                                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }
    return 1;
}
After i compile i get this error

Код:
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(2409) : error 029: invalid expression, assumed zero
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(2412) : warning 219: local variable "i" shadows a variable at a preceding level
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(2418) : warning 219: local variable "sendername" shadows a variable at a preceding level
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(2418) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(2418) : warning 219: local variable "helperinfo" shadows a variable at a preceding level
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


1 Error.
Reply
#4

Change
pawn Код:
// From line 2412:
new i
// To
i
pawn Код:
// From line 2418:
new adminfo[64], sendername[MAX_PLAYER_NAME], string[128], helperinfo[64];
// To
new adminfo[64];
Reply
#5

Its tell that the problem is in the line 2408 which is
pawn Код:
}
                    else   //Line 2408  //the full new arranged commands can get at down.The problem start from helper
                    {
                        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Helper Team] {a7c2e6}- Helpers:");
                        for(i = 0; i < MAX_PLAYERS; i++)
and the line 2417 is the line 2418 that you asked me to change
that
Код:
 new adminfo[64];
is actually
Код:
 new helperinfo[64]
because that admin info i already set at line 2390

This is the new one which you asked me to arrange

pawn Код:
CMD:team(playerid, params[])
{
    if(IsPlayerConnected(playerid))
    {
        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Admin Team] {a7c2e6}- Administrators:");
        for(new i = 0; i < MAX_PLAYERS; i++)
        {
            if(IsPlayerConnected(i))
            {
                if(PlayerInfo[i][pAdmin] >= 1 && PlayerInfo[i][pAdmin] <= 6)
                {
                    new adminfo[64], sendername[MAX_PLAYER_NAME], string[128];
                    if(PlayerInfo[i][pAdmin] == 5) { adminfo = "Owner"; }
                    else if(PlayerInfo[i][pAdmin] == 4) { adminfo = "Head Of Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 3) { adminfo = "Lead Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 2) { adminfo = "Basic Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 1) { adminfo = "Trial Admin"; }
                    else { adminfo = "Admin"; }
                    GetPlayerName(i, sendername, sizeof(sendername));
                    if(AdminDuty[i] == 0)
                    {
                        format(string, 128, "{e1e3e6} - %s %s", adminfo, sendername);
                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                    }
                    else
                    {
                        format(string, 128, "{7bd25c} - %s %s [ADMIN DUTY]", adminfo, sendername);
                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                    }
                    else
                    {
                        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Helper Team] {a7c2e6}- Helpers:");
                        for(i = 0; i < MAX_PLAYERS; i++)
                        {
                            if(IsPlayerConnected(i))
                            {
                                if(PlayerInfo[i][pHelper] >= 1 && PlayerInfo[i][pHelper] <= 2)
                                {
                                    new helperinfo[64];
                                    if(PlayerInfo[i][pHelper] == 1) { helperinfo = "Helper"; }
                                    else { helperinfo = "Helper"; }
                                    GetPlayerName(i, sendername, sizeof(sendername));
                                    if(AdminDuty[i] == 0)
                                    {
                                        format(string, 128, "{e1e3e6} - %s %s", helperinfo, sendername);
                                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                                    }
                                    else
                                    {
                                        format(string, 128, "{7bd25c} - %s %s [HELPER DUTY]", helperinfo, sendername);
                                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }
    return 1;
}
and there is still an erro

Код:
C:\Users\Admin\Documents\Server\The Andreas\gamemodes\TheAndreasife.pwn(2408) : error 029: invalid expression, assumed zero
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


1 Error.
Reply
#6

You're using the "else" two times.
pawn Код:
// An example
new
    x = 1,
    y = 2
;

if( x > y )
{  
    y = x;
}
else
{  
    y = x;
}
// It should stop here, although you're using another else which is wrong.
else
{  
    y = x;
}
I'm not sure what exactly did you want to do there, but I think you wanted to continue outside the if/else statement.
pawn Код:
CMD:team(playerid, params[])
{
    if(IsPlayerConnected(playerid))
    {
        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Admin Team] {a7c2e6}- Administrators:");
        for(new i = 0; i < MAX_PLAYERS; i++)
        {
            if(IsPlayerConnected(i))
            {
                if(PlayerInfo[i][pAdmin] >= 1 && PlayerInfo[i][pAdmin] <= 6)
                {
                    new adminfo[64], sendername[MAX_PLAYER_NAME], string[128];
                    if(PlayerInfo[i][pAdmin] == 5) { adminfo = "Owner"; }
                    else if(PlayerInfo[i][pAdmin] == 4) { adminfo = "Head Of Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 3) { adminfo = "Lead Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 2) { adminfo = "Basic Admin"; }
                    else if(PlayerInfo[i][pAdmin] == 1) { adminfo = "Trial Admin"; }
                    else { adminfo = "Admin"; }
                    GetPlayerName(i, sendername, sizeof(sendername));
                    if(AdminDuty[i] == 0)
                    {
                        format(string, 128, "{e1e3e6} - %s %s", adminfo, sendername);
                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                    }
                    else
                    {
                        format(string, 128, "{7bd25c} - %s %s [ADMIN DUTY]", adminfo, sendername);
                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                    }
                    SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Helper Team] {a7c2e6}- Helpers:");
                    for(i = 0; i < MAX_PLAYERS; i++)
                    {
                        if(IsPlayerConnected(i))
                        {
                            if(PlayerInfo[i][pHelper] >= 1 && PlayerInfo[i][pHelper] <= 2)
                            {
                                new helperinfo[64];
                                if(PlayerInfo[i][pHelper] == 1) { helperinfo = "Helper"; }
                                else { helperinfo = "Helper"; }
                                GetPlayerName(i, sendername, sizeof(sendername));
                                if(AdminDuty[i] == 0)
                                {
                                    format(string, 128, "{e1e3e6} - %s %s", helperinfo, sendername);
                                    SendClientMessage(playerid, random(0xFFFFFFFF), string);
                                }
                                else
                                {
                                    format(string, 128, "{7bd25c} - %s %s [HELPER DUTY]", helperinfo, sendername);
                                    SendClientMessage(playerid, 0xFAAFBEFF, string);
                                }
                            }
                        }
                    }
                }
            }
        }
    }
    return 1;
}
Reply
#7

I know this doesn't answer the problem, but I strongly recommend you use a 'switch' statement, it's faster, and more neater IMO. Modified version of Dwane's code:

pawn Код:
CMD:team(playerid, params[])
{
    if(IsPlayerConnected(playerid))
    {
        SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Admin Team] {a7c2e6}- Administrators:");
        for(new i = 0; i < MAX_PLAYERS; i++)
        {
            if(IsPlayerConnected(i))
            {
                if(PlayerInfo[i][pAdmin] >= 1 && PlayerInfo[i][pAdmin] <= 6)
                {
                    new adminfo[64], sendername[MAX_PLAYER_NAME], string[128];
                    switch(PlayerInfo[i][pAdmin])
                    {
                        case 1: adminfo ="Level 1 Rank Here";
                        case 2: adminfo ="Level 2 Rank Here";
                        case 3: adminfo ="Level 3 Rank Here";
                        case 4: adminfo ="Level 4 Rank Here";
                        case 5: adminfo ="Level 5 Rank Here";
                        case 6: adminfo ="Level 6 Rank Here";
                    }
                    GetPlayerName(i, sendername, sizeof(sendername));
                    if(AdminDuty[i] == 0)
                    {
                        format(string, 128, "{e1e3e6} - %s %s", adminfo, sendername);
                        SendClientMessage(playerid, random(0xFFFFFFFF), string);
                    }
                    else
                    {
                        format(string, 128, "{7bd25c} - %s %s [ADMIN DUTY]", adminfo, sendername);
                        SendClientMessage(playerid, 0xFAAFBEFF, string);
                    }
                    SendClientMessage(playerid, COLOR_GREY, "{4985d7}[Helper Team] {a7c2e6}- Helpers:");
                    for(i = 0; i < MAX_PLAYERS; i++)
                    {
                        if(IsPlayerConnected(i))
                        {
                            if(PlayerInfo[i][pHelper] >= 1 && PlayerInfo[i][pHelper] <= 2)
                            {
                                new helperinfo[64];
                                if(PlayerInfo[i][pHelper] == 1) { helperinfo = "Helper"; }
                                else { helperinfo = "Helper"; }
                                GetPlayerName(i, sendername, sizeof(sendername));
                                if(AdminDuty[i] == 0)
                                {
                                    format(string, 128, "{e1e3e6} - %s %s", helperinfo, sendername);
                                    SendClientMessage(playerid, random(0xFFFFFFFF), string);
                                }
                                else
                                {
                                    format(string, 128, "{7bd25c} - %s %s [HELPER DUTY]", helperinfo, sendername);
                                    SendClientMessage(playerid, 0xFAAFBEFF, string);
                                }
                            }
                        }
                    }
                }
            }
        }
    }
    return 1;
}
Reply
#8

Wow works nice and i tested it.Nothing problem found.Thank you very much for help
I'll add your credits in the pawn script
Reply
#9

Wow kinda nice the "switch" i'll add and thank you for the help.
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)