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

Threaded Mode
/admins
JXF
Big Clucker
***
Posts: 120
Threads: 31
Joined: Feb 2016
Reputation: 0
#1

27.05.2016, 20:04
Код:
C:\Users\Marius\Desktop\GM\gamemodes\GM.pwn(448) : warning 217: loose indentation
C:\Users\Marius\Desktop\GM\gamemodes\GM.pwn(463) : error 030: compound statement not closed at the end of file (started at line 400)
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


1 Error.
Код:
CMD:admins(playerid, params[])
{
	new sendername[MAX_PLAYER_NAME], string[128];

	if(IsPlayerConnected(playerid))
	{
	SCM(pid, -1, "{FF6600}Online {FFFFFF}Admins");
	for(new i = 0; i < MAX_PLAYERS; i++)
	{
	    if(IsPlayerConnected(i))
	    {
	    
	        if(Player[i][pAdmin] == 1338)
	        {
	        GetPlayerName(i, sendername, sizeof(sendername));
	        format(string, sizeof(string), "Server Founder: {FF0000}%s", sendername);
	        SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 1337)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 5)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Mapper: {0080FF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 4)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Leader Coordinator: {6633CC}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 3)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Guardian: {A173FF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 2)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Moderator: {5CFEADFF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 1)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
			
		}
		else
		{
		    if(!IsPlayerConnected(i))
		    {
				format(string, sizeof(string), "{08FC9A}There are no admins online.");
				SCM(pid, -1, string);
			}
 			return 1;
		}
	}
}
What's the problem?
Find
Reply
izeatfishz
Huge Clucker
***
Posts: 230
Threads: 67
Joined: Jun 2012
Reputation: 0
#2

27.05.2016, 20:11
Код:
CMD:admins(playerid, params[])
{
	new sendername[MAX_PLAYER_NAME], string[128];

	if(IsPlayerConnected(playerid))
	{
	SCM(pid, -1, "{FF6600}Online {FFFFFF}Admins");
	for(new i = 0; i < MAX_PLAYERS; i++)
	{
	    if(IsPlayerConnected(i))
	    {
	    
	        if(Player[i][pAdmin] == 1338)
	        {
	        	GetPlayerName(i, sendername, sizeof(sendername));
	        	format(string, sizeof(string), "Server Founder: {FF0000}%s", sendername);
	        	SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 1337)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 5)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Mapper: {0080FF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 4)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Leader Coordinator: {6633CC}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 3)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Guardian: {A173FF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 2)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Moderator: {5CFEADFF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 1)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
		}
		else
		{
			if(!IsPlayerConnected(i))
		        {
				format(string, sizeof(string), "{08FC9A}There are no admins online.");
				SCM(pid, -1, string);
		  	}
 			return 1;
		}
	}
}
Find
Reply
JXF
Big Clucker
***
Posts: 120
Threads: 31
Joined: Feb 2016
Reputation: 0
#3

27.05.2016, 20:15
Quote:
Originally Posted by izeatfishz
Посмотреть сообщение
Код:
CMD:admins(playerid, params[])
{
	new sendername[MAX_PLAYER_NAME], string[128];

	if(IsPlayerConnected(playerid))
	{
	SCM(pid, -1, "{FF6600}Online {FFFFFF}Admins");
	for(new i = 0; i < MAX_PLAYERS; i++)
	{
	    if(IsPlayerConnected(i))
	    {
	    
	        if(Player[i][pAdmin] == 1338)
	        {
	        	GetPlayerName(i, sendername, sizeof(sendername));
	        	format(string, sizeof(string), "Server Founder: {FF0000}%s", sendername);
	        	SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 1337)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 5)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Mapper: {0080FF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 4)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Leader Coordinator: {6633CC}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 3)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Guardian: {A173FF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 2)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Moderator: {5CFEADFF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
		}
		if(Player[i][pAdmin] == 1)
		{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
		}
		else
		{
			if(!IsPlayerConnected(i))
		        {
				format(string, sizeof(string), "{08FC9A}There are no admins online.");
				SCM(pid, -1, string);
		  	}
 			return 1;
		}
	}
}
I appreciate that you helped to space my script out, but it doesn't fix anything, lol.
Find
Reply
oMa37
Gangsta
*****
Posts: 928
Threads: 13
Joined: Feb 2016
Reputation: 0
#4

27.05.2016, 20:20
PHP код:
CMD:admins(playerid, params[])
{
    new sendername[MAX_PLAYER_NAME], string[128];
    SCM(pid, -1, "{FF6600}Online {FFFFFF}Admins");
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i))
        {
            if(Player[i][pAdmin] == 1338)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Founder: {FF0000}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 1337)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 5)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Mapper: {0080FF}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 4)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Leader Coordinator: {6633CC}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 3)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Guardian: {A173FF}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 2)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Moderator: {5CFEADFF}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 1)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
            }
        }
        else
        {
            if(!IsPlayerConnected(i))
            {
                format(string, sizeof(string), "{08FC9A}There are no admins online.");
                SCM(pid, -1, string);
            }
        }
    }
    return 1;
} 
Find
Reply
F1N4L
High-roller
*****
Posts: 2,723
Threads: 110
Joined: Jun 2013
Reputation: 0
#5

27.05.2016, 20:20
He missed close Brackets

Код:
CMD:admins(playerid, params[])
{
	new sendername[MAX_PLAYER_NAME], string[128];

	if(IsPlayerConnected(playerid))
	{
	SCM(pid, -1, "{FF6600}Online {FFFFFF}Admins");
	for(new i = 0; i < MAX_PLAYERS; i++)
	{
	    if(IsPlayerConnected(i))
	    {
	    
	        if(Player[i][pAdmin] == 1338)
	        {
	        GetPlayerName(i, sendername, sizeof(sendername));
	        format(string, sizeof(string), "Server Founder: {FF0000}%s", sendername);
	        SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 1337)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 5)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Mapper: {0080FF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 4)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Leader Coordinator: {6633CC}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 3)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Guardian: {A173FF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 2)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Server Moderator: {5CFEADFF}%s", sendername);
			SCM(pid, COLOR_WHITE, string);
			}
			if(Player[i][pAdmin] == 1)
			{
			GetPlayerName(i, sendername, sizeof(sendername));
			format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
			
		}
		else
		{
		    if(!IsPlayerConnected(i))
		    {
				format(string, sizeof(string), "{08FC9A}There are no admins online.");
				SCM(pid, -1, string);
			}
 			return 1;
		}
	}
}
}
}
Find
Reply
JXF
Big Clucker
***
Posts: 120
Threads: 31
Joined: Feb 2016
Reputation: 0
#6

27.05.2016, 20:20
Quote:
Originally Posted by oMa37
Посмотреть сообщение
PHP код:
CMD:admins(playerid, params[])
{
    new sendername[MAX_PLAYER_NAME], string[128];
    SCM(pid, -1, "{FF6600}Online {FFFFFF}Admins");
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i))
        {
            if(Player[i][pAdmin] == 1338)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Founder: {FF0000}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 1337)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 5)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Mapper: {0080FF}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 4)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Leader Coordinator: {6633CC}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 3)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Guardian: {A173FF}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 2)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Server Moderator: {5CFEADFF}%s", sendername);
                SCM(pid, COLOR_WHITE, string);
            }
            if(Player[i][pAdmin] == 1)
            {
                GetPlayerName(i, sendername, sizeof(sendername));
                format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
            }
        }
        else
        {
            if(!IsPlayerConnected(i))
            {
                format(string, sizeof(string), "{08FC9A}There are no admins online.");
                SCM(pid, -1, string);
            }
        }
    }
    return 1;
} 
Cheers.
Find
Reply
Konstantinos
Spam Machine
*****
Posts: 11,827
Threads: 33
Joined: Dec 2011
Reputation: 0
#7

27.05.2016, 20:21
Let's optimize it, shall we?

- foreach is recommended for player loops, if you don't want to use it then use atleast GetPlayerPoolSize.
- switch is faster than many if statements
- in order to send a message that no admins are online the loop MUST finish first.
- NEVER use return in loops unless you want to stop them

PHP код:
CMD:admins(playerid, params[]) 
{
    new sendername[MAX_PLAYER_NAME], string[128], count;
    SCM(pid, -1, "{FF6600}Online {FFFFFF}Admins");
    for (new i = 0, j = GetPlayerPoolSize(); i <= j; i++) // foreach(new i : Player)
    {
        if (!IsPlayerConnected(i)) continue;
        GetPlayerName(i, sendername, sizeof(sendername));
        switch (Player[i][pAdmin])
        {
            case 1338: format(string, sizeof(string), "Server Founder: {FF0000}%s", sendername);
            case 1337: format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
            case 5: format(string, sizeof(string), "Server Mapper: {0080FF}%s", sendername);
            case 4: format(string, sizeof(string), "Leader Coordinator: {6633CC}%s", sendername);
            case 3: format(string, sizeof(string), "Server Guardian: {A173FF}%s", sendername);
            case 2: format(string, sizeof(string), "Server Moderator: {5CFEADFF}%s", sendername);
            case 1: format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
        }
        SCM(pid, -1, string);
        count++;
    }
    if (!count) SCM(pid, -1, "{08FC9A}There are no admins online.");
    return 1;
} 
Find
Reply
JXF
Big Clucker
***
Posts: 120
Threads: 31
Joined: Feb 2016
Reputation: 0
#8

27.05.2016, 20:47
Quote:
Originally Posted by Konstantinos
Посмотреть сообщение
Let's optimize it, shall we?

- foreach is recommended for player loops, if you don't want to use it then use atleast GetPlayerPoolSize.
- switch is faster than many if statements
- in order to send a message that no admins are online the loop MUST finish first.
- NEVER use return in loops unless you want to stop them

PHP код:
CMD:admins(playerid, params[]) 
{
    new sendername[MAX_PLAYER_NAME], string[128], count;
    SCM(pid, -1, "{FF6600}Online {FFFFFF}Admins");
    for (new i = 0, j = GetPlayerPoolSize(); i <= j; i++) // foreach(new i : Player)
    {
        if (!IsPlayerConnected(i)) continue;
        GetPlayerName(i, sendername, sizeof(sendername));
        switch (Player[i][pAdmin])
        {
            case 1338: format(string, sizeof(string), "Server Founder: {FF0000}%s", sendername);
            case 1337: format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
            case 5: format(string, sizeof(string), "Server Mapper: {0080FF}%s", sendername);
            case 4: format(string, sizeof(string), "Leader Coordinator: {6633CC}%s", sendername);
            case 3: format(string, sizeof(string), "Server Guardian: {A173FF}%s", sendername);
            case 2: format(string, sizeof(string), "Server Moderator: {5CFEADFF}%s", sendername);
            case 1: format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
        }
        SCM(pid, -1, string);
        count++;
    }
    if (!count) SCM(pid, -1, "{08FC9A}There are no admins online.");
    return 1;
} 
The "there are no admins online" message is not shown
has the loop not finished?
also where could I put the message: "There are currently %d admins online"
Find
Reply
Konstantinos
Spam Machine
*****
Posts: 11,827
Threads: 33
Joined: Dec 2011
Reputation: 0
#9

27.05.2016, 21:07
Quote:
Originally Posted by JXF
Посмотреть сообщение
The "there are no admins online" message is not shown
has the loop not finished?
also where could I put the message: "There are currently %d admins online"
It will only be shown if there are no admins online. In your code, it'd be spammed for all the offline players.

Add an alternative of else to the switch just in case:
Код:
...
case 1: format(string, sizeof(string), "Trial Admin: {94FEC9FF}%s", sendername);
default: continue;
and about how many admins:
pawn Код:
if (!count) SCM(pid, -1, "{08FC9A}There are no admins online.");
else
{
    format(string, sizeof(string), "There are currently %d admins online", count); 
    SCM(pid, -1, string);
}
Find
Reply
YaQi
Little Clucker
**
Posts: 8
Threads: 1
Joined: May 2016
Reputation: 0
#10

27.05.2016, 22:45
It's nothing, just you must add (count++) in everyline in an case or in every { }

Example:
Код:
if(Player[i][pAdmin] == 1337)
	{
		GetPlayerName(i, sendername, sizeof(sendername));
		format(string, sizeof(string), "Server Manager: {A31818}%s", sendername);
		SCM(pid, COLOR_WHITE, string);
                count++
	}
Good Luck!
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