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

Threaded Mode
Trying to create a faction for a command.
Thanks
Huge Clucker
****
Posts: 284
Threads: 86
Joined: Nov 2016
Reputation: 0
#1

23.08.2017, 00:58
I'm trying to create a faction for a command but its not work why?

PHP код:
CMD:cmds(playerid, params[])
{
    if(faction1[playerid] == 1)
    {
    TextDrawShowForPlayer(playerid, Textdraw66);
    TextDrawShowForPlayer(playerid, Textdraw67);
    TextDrawShowForPlayer(playerid, Textdraw68);
    TextDrawShowForPlayer(playerid, Textdraw69);
    TextDrawShowForPlayer(playerid, Textdraw70);
    TextDrawShowForPlayer(playerid, Textdraw71);
    TextDrawShowForPlayer(playerid, Textdraw72);
    TextDrawShowForPlayer(playerid, Textdraw73);
    TextDrawShowForPlayer(playerid, Textdraw74);
    TextDrawShowForPlayer(playerid, Textdraw75);
    TextDrawShowForPlayer(playerid, Textdraw76);
    TextDrawShowForPlayer(playerid, Textdraw77);
    TextDrawShowForPlayer(playerid, Textdraw78);
    TextDrawShowForPlayer(playerid, Textdraw79);
    TextDrawShowForPlayer(playerid, Textdraw80);
    TextDrawShowForPlayer(playerid, Textdraw81);
    TextDrawShowForPlayer(playerid, Textdraw82);
    TextDrawShowForPlayer(playerid, Textdraw83);
    TextDrawShowForPlayer(playerid, Textdraw84);
    TextDrawShowForPlayer(playerid, Textdraw85);
    TextDrawShowForPlayer(playerid, Textdraw86);
    TextDrawShowForPlayer(playerid, Textdraw87);
    TextDrawShowForPlayer(playerid, Textdraw88);
    TextDrawShowForPlayer(playerid, Textdraw89);
    TextDrawShowForPlayer(playerid, Textdraw90);
    TextDrawShowForPlayer(playerid, Textdraw91);
    TextDrawShowForPlayer(playerid, Textdraw92);
    TextDrawShowForPlayer(playerid, Textdraw93);
    TextDrawShowForPlayer(playerid, Textdraw94);
    TextDrawShowForPlayer(playerid, Textdraw95);
    TextDrawShowForPlayer(playerid, Textdraw96);
    TextDrawShowForPlayer(playerid, Textdraw97);
    TextDrawShowForPlayer(playerid, Textdraw98);
    TextDrawShowForPlayer(playerid, Textdraw99);
    TextDrawShowForPlayer(playerid, Textdraw100);
    TextDrawShowForPlayer(playerid, Textdraw101);
    TextDrawShowForPlayer(playerid, Textdraw102);
    TextDrawShowForPlayer(playerid, Textdraw103);
    return 1;
    }
    return 0;
} 
Find
Reply
darkhunter332
Huge Clucker
****
Posts: 274
Threads: 45
Joined: Aug 2013
Reputation: 0
#2

23.08.2017, 01:12
What does it say in the error list?
Find
Reply
Thanks
Huge Clucker
****
Posts: 284
Threads: 86
Joined: Nov 2016
Reputation: 0
#3

23.08.2017, 01:27
they didn't give me error but when i press /cmds given me Unkown Command. & i have include zcmd.
Find
Reply
Thanks
Huge Clucker
****
Posts: 284
Threads: 86
Joined: Nov 2016
Reputation: 0
#4

23.08.2017, 04:16
any help
Find
Reply
thegamer355
Big Clucker
***
Posts: 142
Threads: 10
Joined: May 2013
Reputation: 0
#5

23.08.2017, 05:57
You have a return 0; at the bottom of your code. Simply remove it. And move the return 1; to the end, so were the return 0; was.

Edited code below:

pawn Код:
CMD:cmds(playerid, params[]) 
{ 
    if(faction1[playerid] == 1) 
    { 
    TextDrawShowForPlayer(playerid, Textdraw66); 
    TextDrawShowForPlayer(playerid, Textdraw67); 
    TextDrawShowForPlayer(playerid, Textdraw68); 
    TextDrawShowForPlayer(playerid, Textdraw69); 
    TextDrawShowForPlayer(playerid, Textdraw70); 
    TextDrawShowForPlayer(playerid, Textdraw71); 
    TextDrawShowForPlayer(playerid, Textdraw72); 
    TextDrawShowForPlayer(playerid, Textdraw73); 
    TextDrawShowForPlayer(playerid, Textdraw74); 
    TextDrawShowForPlayer(playerid, Textdraw75); 
    TextDrawShowForPlayer(playerid, Textdraw76); 
    TextDrawShowForPlayer(playerid, Textdraw77); 
    TextDrawShowForPlayer(playerid, Textdraw78); 
    TextDrawShowForPlayer(playerid, Textdraw79); 
    TextDrawShowForPlayer(playerid, Textdraw80); 
    TextDrawShowForPlayer(playerid, Textdraw81); 
    TextDrawShowForPlayer(playerid, Textdraw82); 
    TextDrawShowForPlayer(playerid, Textdraw83); 
    TextDrawShowForPlayer(playerid, Textdraw84); 
    TextDrawShowForPlayer(playerid, Textdraw85); 
    TextDrawShowForPlayer(playerid, Textdraw86); 
    TextDrawShowForPlayer(playerid, Textdraw87); 
    TextDrawShowForPlayer(playerid, Textdraw88); 
    TextDrawShowForPlayer(playerid, Textdraw89); 
    TextDrawShowForPlayer(playerid, Textdraw90); 
    TextDrawShowForPlayer(playerid, Textdraw91); 
    TextDrawShowForPlayer(playerid, Textdraw92); 
    TextDrawShowForPlayer(playerid, Textdraw93); 
    TextDrawShowForPlayer(playerid, Textdraw94); 
    TextDrawShowForPlayer(playerid, Textdraw95); 
    TextDrawShowForPlayer(playerid, Textdraw96); 
    TextDrawShowForPlayer(playerid, Textdraw97); 
    TextDrawShowForPlayer(playerid, Textdraw98); 
    TextDrawShowForPlayer(playerid, Textdraw99); 
    TextDrawShowForPlayer(playerid, Textdraw100); 
    TextDrawShowForPlayer(playerid, Textdraw101); 
    TextDrawShowForPlayer(playerid, Textdraw102); 
    TextDrawShowForPlayer(playerid, Textdraw103); 
    } 
    return 1; 
}
Find
Reply
Thanks
Huge Clucker
****
Posts: 284
Threads: 86
Joined: Nov 2016
Reputation: 0
#6

23.08.2017, 16:58
EDIT: Im trying to make like this

PHP код:
CMD:cmds(playerid, params[])
{
    if (faction1[playerid] == 1)
    TextDrawShowForPlayer(playerid, Textdraw66);
    TextDrawShowForPlayer(playerid, Textdraw67);
    TextDrawShowForPlayer(playerid, Textdraw68);
    TextDrawShowForPlayer(playerid, Textdraw69);
    TextDrawShowForPlayer(playerid, Textdraw70);
    TextDrawShowForPlayer(playerid, Textdraw71);
    TextDrawShowForPlayer(playerid, Textdraw72);
    TextDrawShowForPlayer(playerid, Textdraw73);
    TextDrawShowForPlayer(playerid, Textdraw74);
    TextDrawShowForPlayer(playerid, Textdraw75);
    TextDrawShowForPlayer(playerid, Textdraw76);
    TextDrawShowForPlayer(playerid, Textdraw77);
    TextDrawShowForPlayer(playerid, Textdraw78);
    TextDrawShowForPlayer(playerid, Textdraw79);
    TextDrawShowForPlayer(playerid, Textdraw80);
    TextDrawShowForPlayer(playerid, Textdraw81);
    TextDrawShowForPlayer(playerid, Textdraw82);
    TextDrawShowForPlayer(playerid, Textdraw83);
    TextDrawShowForPlayer(playerid, Textdraw84);
    TextDrawShowForPlayer(playerid, Textdraw85);
    TextDrawShowForPlayer(playerid, Textdraw86);
    TextDrawShowForPlayer(playerid, Textdraw87);
    TextDrawShowForPlayer(playerid, Textdraw88);
    TextDrawShowForPlayer(playerid, Textdraw89);
    TextDrawShowForPlayer(playerid, Textdraw90);
    TextDrawShowForPlayer(playerid, Textdraw91);
    TextDrawShowForPlayer(playerid, Textdraw92);
    TextDrawShowForPlayer(playerid, Textdraw93);
    TextDrawShowForPlayer(playerid, Textdraw94);
    TextDrawShowForPlayer(playerid, Textdraw95);
    TextDrawShowForPlayer(playerid, Textdraw96);
    TextDrawShowForPlayer(playerid, Textdraw97);
    TextDrawShowForPlayer(playerid, Textdraw98);
    TextDrawShowForPlayer(playerid, Textdraw99);
    TextDrawShowForPlayer(playerid, Textdraw100);
    TextDrawShowForPlayer(playerid, Textdraw101);
    TextDrawShowForPlayer(playerid, Textdraw102);
    TextDrawShowForPlayer(playerid, Textdraw103);
    TextDrawShowForPlayer(playerid, Textdraw104);
    TextDrawShowForPlayer(playerid, Textdraw105);
    TextDrawShowForPlayer(playerid, Textdraw106);
    return 1;
} 
But when i press 1 giving me nothing.

PHP код:
if(InMenu[playerid] == 1)
        {
            if(faction1[playerid]==1)
            {
               if(strval(text)==1)
               {
                  SendClientMessage(playerid, -1, "{00FFFF}Coming Soon");
                  InMenu[playerid]=1;
                  return 0;
               } 
Find
Reply
Bingo
High-roller
*****
Posts: 1,690
Threads: 18
Joined: Jul 2012
Reputation: 0
#7

23.08.2017, 17:30
Why are you using return 0?
Find
Reply
thegamer355
Big Clucker
***
Posts: 142
Threads: 10
Joined: May 2013
Reputation: 0
#8

23.08.2017, 17:33
Quote:
Originally Posted by Thanks
Посмотреть сообщение
EDIT: Im trying to make like this

But when i press 1 giving me nothing.

PHP код:
if(InMenu[playerid] == 1)
        {
            if(faction1[playerid]==1)
            {
               if(strval(text)==1)
               {
                  SendClientMessage(playerid, -1, "{00FFFF}Coming Soon");
                  InMenu[playerid]=1;
                  return 0;
               } 
So, you're making that work if InMenu[playerid] == 1)
but in the code itself you're setting it to 1? what's the point of that?
Shouldn't the if(InMenu[playerid] == 1) be if(InMenu[playerid] == 0)?

About the return 0, just use return 1;

Also in the code above, please re-read what i've posted before, you are forgetting some {} this time
Find
Reply
Thanks
Huge Clucker
****
Posts: 284
Threads: 86
Joined: Nov 2016
Reputation: 0
#9

23.08.2017, 17:41
Im using it like that

Код:
	if(InMenu[playerid] == 0)
		{
			if(faction1[playerid]==1)
			{
			   if(strval(text)==1)
			   {
				  SendClientMessage(playerid, -1, "{00FFFF}Coming Soon");
				  InMenu[playerid]=0;
				  return 1;
			   }
			}
			faction1[playerid]=1;
		}
   }
	  return 1;
}
And still not work!
Find
Reply
Thanks
Huge Clucker
****
Posts: 284
Threads: 86
Joined: Nov 2016
Reputation: 0
#10

23.08.2017, 18:02
Im using it like that

Код:
	if(InMenu[playerid] == 0)
		{
			if(faction1[playerid]==1)
			{
			   if(strval(text)==1)
			   {
				  SendClientMessage(playerid, -1, "{00FFFF}Coming Soon");
				  InMenu[playerid]=0;
				  return 1;
			   }
			}
			faction1[playerid]=1;
		}
   }
	  return 1;
}
And still not work!
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