1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help

Threaded Mode
Need help with admin view.
zionx08
Little Clucker
**
Posts: 27
Threads: 6
Joined: Sep 2012
Reputation: 0
#1

04.12.2012, 14:03
( Последний раз редактировалось zionx08; 04.12.2012 в 14:07. Причина: Add info )
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.
Find
Reply
Konstantinos
Spam Machine
*****
Posts: 11,827
Threads: 33
Joined: Dec 2011
Reputation: 0
#2

04.12.2012, 14:08
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;
}
Find
Reply
zionx08
Little Clucker
**
Posts: 27
Threads: 6
Joined: Sep 2012
Reputation: 0
#3

04.12.2012, 14:30
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.
Find
Reply
Konstantinos
Spam Machine
*****
Posts: 11,827
Threads: 33
Joined: Dec 2011
Reputation: 0
#4

04.12.2012, 14:48
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];
Find
Reply
zionx08
Little Clucker
**
Posts: 27
Threads: 6
Joined: Sep 2012
Reputation: 0
#5

04.12.2012, 15:15
( Последний раз редактировалось zionx08; 04.12.2012 в 15:18. Причина: Add info )
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.
Find
Reply
Konstantinos
Spam Machine
*****
Posts: 11,827
Threads: 33
Joined: Dec 2011
Reputation: 0
#6

04.12.2012, 15:22
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;
}
Find
Reply
Bicentric
Big Clucker
***
Posts: 143
Threads: 5
Joined: Sep 2012
Reputation: 0
#7

04.12.2012, 15:28
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;
}
Find
Reply
zionx08
Little Clucker
**
Posts: 27
Threads: 6
Joined: Sep 2012
Reputation: 0
#8

04.12.2012, 15:30
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
Find
Reply
zionx08
Little Clucker
**
Posts: 27
Threads: 6
Joined: Sep 2012
Reputation: 0
#9

04.12.2012, 15:33
Wow kinda nice the "switch" i'll add and thank you for the help.
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 2 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help