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

Threaded Mode
Openbay(gate) 2 commands into 1
Cubie
Big Clucker
***
Posts: 54
Threads: 21
Joined: Sep 2017
Reputation: 0
#1

26.09.2017, 14:28
I've got two commands, one to open the bay (openbay) and one to close 'm (closebay)
How would I go about merging these two commands into one?
Thanks in advance.

PHP код:
CMD:openbay(playerid,params[])
{
        if((PlayerInfo[playerid][pFaction] != 2) && (PlayerInfo[playerid][pFaction] != 3))
        return SCM(playerid, COLOR_LIGHTRED, "You're not a Faction member.");
        if(IsPlayerInRangeOfPoint(playerid, 8.0,-2534.17944, 2347.45361, 5.94538))
        {
             MoveObject(fdgate1,-2534.1794, 2348.4136, 7.8590,0.50,0.00000, 90.0000, 90.0000);
        }
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2526.27173, 2347.45361, 5.94540))
        {
            MoveObject(fdgate2,-2526.2717, 2348.4136, 7.8590,0.50,0, 90, 90);
        }
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2518.36084, 2347.45361, 5.94540))
        {
            MoveObject(fdgate3,-2518.3608, 2348.4136, 7.8590,0.50,0, 90, 90);
        }
        return 1;
} 
PHP код:
CMD:closebay(playerid,params[])
{
        if((PlayerInfo[playerid][pFaction] != 2) && (PlayerInfo[playerid][pFaction] != 3))
        return SCM(playerid, COLOR_LIGHTRED, "You're not a Faction member.");
        if(IsPlayerInRangeOfPoint(playerid, 8.0,-2534.17944, 2347.45361, 5.94538))
        {
             MoveObject(fdgate1,-2534.17944, 2347.45361, 5.94538,0.50,0.00000, 0.0000, 90.0000);
        }
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2526.27173, 2347.45361, 5.94540))
        {
            MoveObject(fdgate2,-2526.27173, 2347.45361, 5.94540,0.50,0, 0, 90);
        }
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2518.36084, 2347.45361, 5.94540))
        {
            MoveObject(fdgate3,-2518.36084, 2347.45361, 5.94540,0.50,0, 0, 90);
        }
        return 1;
} 
Find
Reply
10MIN
Banned
Posts: 325
Threads: 5
Joined: Jun 2017
#2

26.09.2017, 14:43
With something like:
PHP код:
CMD:bay(playerid,params[])
{
if((PlayerInfo[playerid][pFaction] != 2) && (PlayerInfo[playerid][pFaction] != 3))
return SCM(playerid, COLOR_LIGHTRED, "You're not a Faction member.");
if(!strcmp(params,"open",true)) //the true means that OpEn will be the same with OPEn etc. (ignore uppercase:true)
{
        if(IsPlayerInRangeOfPoint(playerid, 8.0,-2534.17944, 2347.45361, 5.94538))
        {
             MoveObject(fdgate1,-2534.1794, 2348.4136, 7.8590,0.50,0.00000, 90.0000, 90.0000);
        }
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2526.27173, 2347.45361, 5.94540))
        {
            MoveObject(fdgate2,-2526.2717, 2348.4136, 7.8590,0.50,0, 90, 90);
        }
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2518.36084, 2347.45361, 5.94540))
        {
            MoveObject(fdgate3,-2518.3608, 2348.4136, 7.8590,0.50,0, 90, 90);
        }
        return 1; 
}
else if(!strcmp(params,"close",true))
{
        if(IsPlayerInRangeOfPoint(playerid, 8.0,-2534.17944, 2347.45361, 5.94538))
        {
             MoveObject(fdgate1,-2534.17944, 2347.45361, 5.94538,0.50,0.00000, 0.0000, 90.0000);
        }
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2526.27173, 2347.45361, 5.94540))
        {
            MoveObject(fdgate2,-2526.27173, 2347.45361, 5.94540,0.50,0, 0, 90);
        }
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2518.36084, 2347.45361, 5.94540))
        {
            MoveObject(fdgate3,-2518.36084, 2347.45361, 5.94540,0.50,0, 0, 90);
        }
        return 1; 
}
else return SCM(playerid,-1,"Use the command /bay like: /bay open or /bay close");
} 
PS: Sorry for the identation.
Find
Reply
ToiletDuck
Huge Clucker
****
Posts: 400
Threads: 52
Joined: Apr 2012
Reputation: 0
#3

26.09.2017, 14:46
Simple but there are many ways but this is one way, use sscanf of ******

Код:
CMD:bay(playerid,params[]) 
{ 
    if((PlayerInfo[playerid][pFaction] != 2) && (PlayerInfo[playerid][pFaction] != 3)) 
    	return SCM(playerid, COLOR_LIGHTRED, "You're not a Faction member."); 
    new szType[5];
    if(sscanf(params, "s[5]", szType)) return SCM(playerid, -1, "USAGE: /bay [open/close]");
    if(strcmp(szType, "open", true) == 0)
    {

        if(IsPlayerInRangeOfPoint(playerid, 8.0,-2534.17944, 2347.45361, 5.94538)) 
        	MoveObject(fdgate1,-2534.1794, 2348.4136, 7.8590,0.50,0.00000, 90.0000, 90.0000); 
   
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2526.27173, 2347.45361, 5.94540)) 
        	MoveObject(fdgate2,-2526.2717, 2348.4136, 7.8590,0.50,0, 90, 90); 
        
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2518.36084, 2347.45361, 5.94540)) 
        	MoveObject(fdgate3,-2518.3608, 2348.4136, 7.8590,0.50,0, 90, 90); 
        
    }
    else if(strcmp(szType, "close", true) == 0)
    {
    	if(IsPlayerInRangeOfPoint(playerid, 8.0,-2534.17944, 2347.45361, 5.94538)) 
    		MoveObject(fdgate1,-2534.17944, 2347.45361, 5.94538,0.50,0.00000, 0.0000, 90.0000); 
        
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2526.27173, 2347.45361, 5.94540)) 
        	MoveObject(fdgate2,-2526.27173, 2347.45361, 5.94540,0.50,0, 0, 90); 
         
        else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2518.36084, 2347.45361, 5.94540)) 
        	MoveObject(fdgate3,-2518.36084, 2347.45361, 5.94540,0.50,0, 0, 90); 
        
    }
    else SCM(playerid, -1, "Invalid Choice!");
    return 1; 
}
@EDIT: use ^ much simplier lol.
Find
Reply
Cubie
Big Clucker
***
Posts: 54
Threads: 21
Joined: Sep 2017
Reputation: 0
#4

26.09.2017, 15:47
Thanks guys this'll work.
However is there also a way to make it like this:
/bay
>When the bay is closed it opens
/bay
>When the bay is open it closes
Find
Reply
ToiletDuck
Huge Clucker
****
Posts: 400
Threads: 52
Joined: Apr 2012
Reputation: 0
#5

26.09.2017, 15:50
first you must define iif gate is open or close
fdgate1 < dis one make another global variable fdgatestatus. much better if enum I guess?

after that if fdgatestatus= 0 > meaning it's close
if(!fdgatestatus)
{
// open the gate
fdgatestatus = 1; //set it to open
}
else
{
//close the gate
fdgatestatus = 0; // set it to close
}
Find
Reply
Cubie
Big Clucker
***
Posts: 54
Threads: 21
Joined: Sep 2017
Reputation: 0
#6

27.09.2017, 10:50
Quote:
Originally Posted by ToiletDuck
Посмотреть сообщение
first you must define iif gate is open or close
fdgate1 < dis one make another global variable fdgatestatus. much better if enum I guess?

after that if fdgatestatus= 0 > meaning it's close
if(!fdgatestatus)
{
// open the gate
fdgatestatus = 1; //set it to open
}
else
{
//close the gate
fdgatestatus = 0; // set it to close
}
I get what you mean but I have no clue on how to execute it. Could you maybe give a example?
Find
Reply
MEW273
Big Clucker
***
Posts: 150
Threads: 1
Joined: Jan 2015
Reputation: 0
#7

27.09.2017, 11:35
Quote:
Originally Posted by Cubie
Посмотреть сообщение
I get what you mean but I have no clue on how to execute it. Could you maybe give a example?
This below code should give you an idea of how it can be done, keep in mind this is a somewhat basic piece of code for what you're trying to do, there are many optimizations that could be done.

Код:
new fdgate1_status = 0; // 0 = closed, 1 = open
new fdgate2_status = 0; // 0 = closed, 1 = open
new fdgate3_status = 0; // 0 = closed, 1 = open


CMD:bay(playerid,params[])
{
    if((PlayerInfo[playerid][pFaction] != 2) && (PlayerInfo[playerid][pFaction] != 3))
    	return SCM(playerid, COLOR_LIGHTRED, "You're not a Faction member.");
	else if(IsPlayerInRangeOfPoint(playerid, 8.0,-2534.17944, 2347.45361, 5.94538)) // fdgate1
	{
	    if(fdgate1_status == 0) // gate is closed, so let's open it
	    {
	    	MoveObject(fdgate1,-2534.1794, 2348.4136, 7.8590,0.50,0.00000, 90.0000, 90.0000);
			fdgate1_status = 1; // we just opened the gate so let's update our variable
		}
		else // gate is not closed so it must be open, so let's close it
		{
			MoveObject(fdgate1,-2534.17944, 2347.45361, 5.94538,0.50,0.00000, 0.0000, 90.0000);
			fdgate1_status = 0; // we just closed the gate so let's update our variable
		}
	}
	else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2526.27173, 2347.45361, 5.94540)) // fdgate2
	{
	    if(fdgate2_status == 0) // gate is closed, so let's open it
	    {
	    	MoveObject(fdgate2,-2526.2717, 2348.4136, 7.8590,0.50,0, 90, 90);
			fdgate2_status = 1; // we just opened the gate so let's update our variable
		}
		else // gate is not closed so it must be open, so let's close it
		{
			MoveObject(fdgate2,-2526.27173, 2347.45361, 5.94540,0.50,0, 0, 90);
			fdgate2_status = 0; // we just closed the gate so let's update our variable
		}
	}
	else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2518.36084, 2347.45361, 5.94540)) // fdgate3
	{
	    if(fdgate3_status == 0) // gate is closed, so let's open it
	    {
	    	MoveObject(fdgate3,-2518.3608, 2348.4136, 7.8590,0.50,0, 90, 90);
			fdgate3_status = 1; // we just opened the gate so let's update our variable
		}
		else // gate is not closed so it must be open, so let's close it
		{
			MoveObject(fdgate3,-2518.36084, 2347.45361, 5.94540,0.50,0, 0, 90);
			fdgate3_status = 0; // we just closed the gate so let's update our variable
		}
	}
	else SCM(playerid, -1, "Invalid Choice - you aren't close to any gate!");
	return 1;
}
Find
Reply
Cubie
Big Clucker
***
Posts: 54
Threads: 21
Joined: Sep 2017
Reputation: 0
#8

27.09.2017, 12:08
Quote:
Originally Posted by MEW273
Посмотреть сообщение
This below code should give you an idea of how it can be done, keep in mind this is a somewhat basic piece of code for what you're trying to do, there are many optimizations that could be done.

Код:
new fdgate1_status = 0; // 0 = closed, 1 = open
new fdgate2_status = 0; // 0 = closed, 1 = open
new fdgate3_status = 0; // 0 = closed, 1 = open


CMD:bay(playerid,params[])
{
    if((PlayerInfo[playerid][pFaction] != 2) && (PlayerInfo[playerid][pFaction] != 3))
    	return SCM(playerid, COLOR_LIGHTRED, "You're not a Faction member.");
	else if(IsPlayerInRangeOfPoint(playerid, 8.0,-2534.17944, 2347.45361, 5.94538)) // fdgate1
	{
	    if(fdgate1_status == 0) // gate is closed, so let's open it
	    {
	    	MoveObject(fdgate1,-2534.1794, 2348.4136, 7.8590,0.50,0.00000, 90.0000, 90.0000);
			fdgate1_status = 1; // we just opened the gate so let's update our variable
		}
		else // gate is not closed so it must be open, so let's close it
		{
			MoveObject(fdgate1,-2534.17944, 2347.45361, 5.94538,0.50,0.00000, 0.0000, 90.0000);
			fdgate1_status = 0; // we just closed the gate so let's update our variable
		}
	}
	else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2526.27173, 2347.45361, 5.94540)) // fdgate2
	{
	    if(fdgate2_status == 0) // gate is closed, so let's open it
	    {
	    	MoveObject(fdgate2,-2526.2717, 2348.4136, 7.8590,0.50,0, 90, 90);
			fdgate2_status = 1; // we just opened the gate so let's update our variable
		}
		else // gate is not closed so it must be open, so let's close it
		{
			MoveObject(fdgate2,-2526.27173, 2347.45361, 5.94540,0.50,0, 0, 90);
			fdgate2_status = 0; // we just closed the gate so let's update our variable
		}
	}
	else if(IsPlayerInRangeOfPoint(playerid, 7.0,-2518.36084, 2347.45361, 5.94540)) // fdgate3
	{
	    if(fdgate3_status == 0) // gate is closed, so let's open it
	    {
	    	MoveObject(fdgate3,-2518.3608, 2348.4136, 7.8590,0.50,0, 90, 90);
			fdgate3_status = 1; // we just opened the gate so let's update our variable
		}
		else // gate is not closed so it must be open, so let's close it
		{
			MoveObject(fdgate3,-2518.36084, 2347.45361, 5.94540,0.50,0, 0, 90);
			fdgate3_status = 0; // we just closed the gate so let's update our variable
		}
	}
	else SCM(playerid, -1, "Invalid Choice - you aren't close to any gate!");
	return 1;
}
You're amazing. Thanks bud +Rep'd
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


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