SA-MP Forums Archive
Staement isnt closed? How? This is weird - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+---- Forum: Help Archive (https://sampforum.blast.hk/forumdisplay.php?fid=89)
+---- Thread: Staement isnt closed? How? This is weird (/showthread.php?tid=271308)



Staement isnt closed? How? This is weird - Shockey HD - 23.07.2011

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)



Re: Staement isnt closed? How? This is weird - HydraX - 23.07.2011

Your ending the script with 2 returns. Add only one. In other words, remove return 1;


Re: Staement isnt closed? How? This is weird - Shockey HD - 23.07.2011

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...


Re: Staement isnt closed? How? This is weird - =WoR=Varth - 24.07.2011

You missed closing bracket "}"


Re: Staement isnt closed? How? This is weird - Skylar Paul - 24.07.2011

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..


Re: Staement isnt closed? How? This is weird - MoroDan - 24.07.2011

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"50006);
        }
        else if(
listitem == 1)
        {
            if(
IsPlayerVipType(playerid1))
            {
                
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"50006);
        }
        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;




Re: Staement isnt closed? How? This is weird - Mean - 24.07.2011

You need non of those returns.
It probably ain't the problem though. You've forgot a closing bracket somewhere.


Re: Staement isnt closed? How? This is weird - HuSs3n - 24.07.2011

Код:
			
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!");
}



Re: Staement isnt closed? How? This is weird - shaikh007 - 24.07.2011

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


Re: Staement isnt closed? How? This is weird - Shockey HD - 25.07.2011

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