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

Threaded Mode
Staement isnt closed? How? This is weird
Shockey HD
Gangsta
****
Posts: 684
Threads: 122
Joined: Jul 2011
Reputation: 0
#1

23.07.2011, 21:32
Heres the code that this error is coming up for.

Код:
	if(dialogid == 941)
	{
		if(response)
		{
			if(listitem == 0)
			{
			GameTextForPlayer(playerid, "Parachutes for VIP", 5000, 6);
 			}
 			if(listitem == 1)
			{
			if(IsPlayerVipType(playerid,1)) { // 3418
			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for admins!");
			}
			if(listitem == 2)
			{
			if(IsPlayerVipType(playerid,1)) {
			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for admins!");
			}
			if(listitem == 3)
			{
			if(IsPlayerVipType(playerid,1)) {
			GivePlayerWeapon(playerid,46,1);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 3 for admins!");
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			}
			if(listitem == 4)
			{
			if(IsPlayerVipType(playerid,1)) {
			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 4 for admins!");
			}
			if(listitem == 5)
			{
   			if(IsPlayerVipType(playerid,1)) {
   			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 5 for admins!");
			}
			if(listitem == 6)
			{
			GameTextForPlayer(playerid, "Parachutes for FreeMembers", 5000, 6);
 			}
			if(listitem == 7)
			{
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			GivePlayerWeapon(playerid,46,1);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for freemembers!");
			 }
			if(listitem == 8)
			{
			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for freemembers!");
 			}
		}
 	return 1;
	}
	return 0;
}
Код:
C:\Documents and Settings\Chris\Desktop\FreeRoam(2).pwn(3472) : error 030: compound statement not closed at the end of file (started at line (started at line 3418)
Find
Reply
HydraX
Gangsta
*****
Posts: 896
Threads: 85
Joined: Mar 2009
Reputation: 0
#2

23.07.2011, 21:39
Your ending the script with 2 returns. Add only one. In other words, remove return 1;
Find
Reply
Shockey HD
Gangsta
****
Posts: 684
Threads: 122
Joined: Jul 2011
Reputation: 0
#3

23.07.2011, 21:50
Quote:
Originally Posted by HydraX
Посмотреть сообщение
Your ending the script with 2 returns. Add only one. In other words, remove return 1;
I removed the VIP code, and it worked. But, my script was to have, One section is for VIP members and the others for regular members...
Find
Reply
=WoR=Varth
High-roller
*****
Posts: 1,988
Threads: 17
Joined: Apr 2011
Reputation: 0
#4

24.07.2011, 00:51
You missed closing bracket "}"
Find
Reply
Skylar Paul
Huge Clucker
****
Posts: 282
Threads: 47
Joined: Feb 2011
Reputation: 0
#5

24.07.2011, 02:54
pawn Код:
if(dialogid == 941)
{
    if(response)
    {
        if(listitem == 0)
        {
            GameTextForPlayer(playerid, "Parachutes for VIP", 5000, 6);
        }
        else if(listitem == 1)
        {
            if(IsPlayerVipType(playerid,1))
            { // 3418
                GivePlayerWeapon(playerid,46,1);
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for admins!");
            }
        }
        else if(listitem == 2)
        {
                if(IsPlayerVipType(playerid,1))
                {
                GivePlayerWeapon(playerid,46,1);
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for admins!");
                }
        }
        else if(listitem == 3)
        {
            if(IsPlayerVipType(playerid,1))
            {
                GivePlayerWeapon(playerid,46,1);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 3 for admins!");
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            }
        }
        else if(listitem == 4)
        {
            if(IsPlayerVipType(playerid,1))
            {
                GivePlayerWeapon(playerid,46,1);
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 4 for admins!");
            }
        }
        else if(listitem == 5)
        {
            if(IsPlayerVipType(playerid,1))
            {
                GivePlayerWeapon(playerid,46,1);
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 5 for admins!");
            }
        }
        else if(listitem == 6)
        {
            GameTextForPlayer(playerid, "Parachutes for FreeMembers", 5000, 6);
        }
        else if(listitem == 7)
        {
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            GivePlayerWeapon(playerid,46,1);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for freemembers!");
        }
        else if(listitem == 8)
        {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for freemembers!");
        }
        return 1;
    }
    return 0;
}
My god.. Please learn to properly use indentation..
Find
Reply
MoroDan
Huge Clucker
****
Posts: 278
Threads: 0
Joined: Jun 2011
Reputation: 0
#6

24.07.2011, 09:46
Quote:
Originally Posted by Shockey HD
Посмотреть сообщение
Heres the code that this error is coming up for.

Код:
	if(dialogid == 941)
	{
		if(response)
		{
			if(listitem == 0)
			{
			GameTextForPlayer(playerid, "Parachutes for VIP", 5000, 6);
 			}
 			if(listitem == 1)
			{
			if(IsPlayerVipType(playerid,1)) { // 3418
			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for admins!");
			}
			if(listitem == 2)
			{
			if(IsPlayerVipType(playerid,1)) {
			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for admins!");
			}
			if(listitem == 3)
			{
			if(IsPlayerVipType(playerid,1)) {
			GivePlayerWeapon(playerid,46,1);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 3 for admins!");
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			}
			if(listitem == 4)
			{
			if(IsPlayerVipType(playerid,1)) {
			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 4 for admins!");
			}
			if(listitem == 5)
			{
   			if(IsPlayerVipType(playerid,1)) {
   			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 5 for admins!");
			}
			if(listitem == 6)
			{
			GameTextForPlayer(playerid, "Parachutes for FreeMembers", 5000, 6);
 			}
			if(listitem == 7)
			{
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			GivePlayerWeapon(playerid,46,1);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for freemembers!");
			 }
			if(listitem == 8)
			{
			GivePlayerWeapon(playerid,46,1);
			SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
			SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for freemembers!");
 			}
		}
 	return 1;
	}
	return 0;
}
Код:
C:\Documents and Settings\Chris\Desktop\FreeRoam(2).pwn(3472) : error 030: compound statement not closed at the end of file (started at line (started at line 3418)
PHP код:
if(dialogid == 941)
{
    if(response)
    {
        if(listitem == 0)
        {
            GameTextForPlayer(playerid, "Parachutes for VIP", 5000, 6);
        }
        else if(listitem == 1)
        {
            if(IsPlayerVipType(playerid, 1))
            {
                GivePlayerWeapon(playerid,46,1);
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for admins!");
            }
        }
        else if(listitem == 2)
        {
            if(IsPlayerVipType(playerid,1))
            {
                GivePlayerWeapon(playerid,46,1);
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for admins!");
            }
        }
        else if(listitem == 3)
        {
            if(IsPlayerVipType(playerid,1))
            {
                GivePlayerWeapon(playerid,46,1);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 3 for admins!");
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            }
        }
        else if(listitem == 4)
        {
            if(IsPlayerVipType(playerid,1))
            {
                GivePlayerWeapon(playerid,46,1);
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 4 for admins!");
            }
        }
        else if(listitem == 5)
        {
            if(IsPlayerVipType(playerid,1))
            {
                GivePlayerWeapon(playerid,46,1);
                SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
                SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 5 for admins!");
            }
        }
        else if(listitem == 6)
        {
            GameTextForPlayer(playerid, "Parachutes for FreeMembers", 5000, 6);
        }
        else if(listitem == 7)
        {
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            GivePlayerWeapon(playerid,46,1);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for freemembers!");
        }
        else if(listitem == 8)
        {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for freemembers!");
        }
        return 1;
    }
    return 0;
} 
Find
Reply
Mean
High-roller
*****
Posts: 2,726
Threads: 85
Joined: Jul 2010
Reputation: 0
#7

24.07.2011, 10:15
You need non of those returns.
It probably ain't the problem though. You've forgot a closing bracket somewhere.
Find
Reply
HuSs3n
Unregistered
 
#8

24.07.2011, 12:34
Код:
			
if(listitem == 2)
{
if(IsPlayerVipType(playerid,1)) {  <============= remove this ,lol
GivePlayerWeapon(playerid,46,1);
SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for admins!");
}
Reply
shaikh007
Big Clucker
***
Posts: 72
Threads: 14
Joined: Jul 2011
Reputation: 0
#9

24.07.2011, 15:03
pawn Код:
if(dialogid == 941)
    {
        if(response)
        {
            if(listitem == 0)
            {
            GameTextForPlayer(playerid, "Parachutes for VIP", 5000, 6);
            }
            if(listitem == 1)
            {
            if(IsPlayerVipType(playerid,1)) { // 3418
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for admins!");
            }
            if(listitem == 2)
            {
            if(IsPlayerVipType(playerid,1)) {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for admins!");
            }
            if(listitem == 3)
            {
            if(IsPlayerVipType(playerid,1)) {
            GivePlayerWeapon(playerid,46,1);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 3 for admins!");
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            }
            }
            if(listitem == 4)
            {
            if(IsPlayerVipType(playerid,1)) {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 4 for admins!");
            }
            }
            if(listitem == 5)
            {
            if(IsPlayerVipType(playerid,1)) {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 5 for admins!");
            }
            }
            if(listitem == 6)
            {
            GameTextForPlayer(playerid, "Parachutes for FreeMembers", 5000, 6);
            }
            if(listitem == 7)
            {
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            GivePlayerWeapon(playerid,46,1);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for freemembers!");
             }
            if(listitem == 8)
            {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for freemembers!");
            }
        }
    }
}
}
    return 1;
}
try above code may be it help
Find
Reply
Shockey HD
Gangsta
****
Posts: 684
Threads: 122
Joined: Jul 2011
Reputation: 0
#10

25.07.2011, 04:26
Quote:
Originally Posted by [RonaldO]
Посмотреть сообщение
Код:
			
if(listitem == 2)
{
if(IsPlayerVipType(playerid,1)) {  <============= remove this ,lol
GivePlayerWeapon(playerid,46,1);
SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for admins!");
}
Come on bro are you really that stupid? That has nothing to do with the problem. LOL! If you dont know what your talking about, dont answer...



Also thanks guys but i got two problems now from it.

Код:
if(dialogid == 941)
    {
        if(response)
        {
            if(listitem == 0)
            {
            GameTextForPlayer(playerid, "Parachutes for VIP", 5000, 6);
            }
            if(listitem == 1)
            {
            if(IsPlayerVipType(playerid,1)) { // 3418
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for admins!");
            }
            }
            if(listitem == 2)
            {
            if(IsPlayerVipType(playerid,1)) {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for admins!");
            }
            }
            if(listitem == 3)
            {
            if(IsPlayerVipType(playerid,1)) {
            GivePlayerWeapon(playerid,46,1);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 3 for admins!");
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            }
            }
            if(listitem == 4)
            {
            if(IsPlayerVipType(playerid,1)) {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 4 for admins!");
            }
            }
            if(listitem == 5)
            {
            if(IsPlayerVipType(playerid,1)) {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 5 for admins!");
            }
            }
            if(listitem == 6)
            {
            GameTextForPlayer(playerid, "Parachutes for FreeMembers", 5000, 6);
            }
            if(listitem == 7)
            {
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            GivePlayerWeapon(playerid,46,1);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 1 for freemembers!");
             }
            if(listitem == 8)
            {
            GivePlayerWeapon(playerid,46,1);
            SetPlayerPos(playerid,2529.000000,-1603.000000,1505.00000);
            SendClientMessage(playerid,COLOR_GREEN,"You have gone to Parachue 2 for freemembers!");
            }
        }
    }
}
    return 1;
}
With the errors of

Код:
C:\Documents and Settings\Chris\Desktop\FreeRoam(2).pwn(3479) : warning 209: function "OnDialogResponse" should return a value
C:\Documents and Settings\Chris\Desktop\FreeRoam(2).pwn(3480) : error 010: invalid function or declaration
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


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