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

Threaded Mode
What's wrong with this command?
Tnt12340
Big Clucker
***
Posts: 61
Threads: 11
Joined: Mar 2007
Reputation: 0
#1

13.06.2010, 13:01
pawn Код:
if(strcmp(cmd, "/pddoor", true) == 0)
    {
      if(IsPlayerConnected(playerid))
      {
            GetPlayerName(playerid, sendername, sizeof(sendername));
            if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (!pddoor))
            {
                pddoor = 1;
                gate = 4;
                MoveDynamicObject(3, 248.880111, 72.445343, 1002.560546,5);
                MoveDynamicObject(4, 243.894638, 72.445343, 1002.560546,5);
                format(string, sizeof(string), "* %s takes out a key, and unlocks the door.", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
                format(string, sizeof(string), "* %s opens the door.", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
            }
            GetPlayerName(playerid, sendername, sizeof(sendername));
            else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor))
            {
                pddoor = 0;
                MoveDynamicObject(3, 247.880111, 72.445343, 1002.560546,5);
                MoveDynamicObject(4, 244.894638, 72.445343, 1002.560546,5);
                format(string, sizeof(string), "* %s closes the door. *It Auto Locks*", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
            }
            else
            {
                SendClientMessage(playerid, COLOR_GRAD1, "You are not a police officer!");
            }
        }
        return 1;
    }
When i use it, it comes up with these errors :

Quote:

C:\Documents and Settings\Eric\Desktop\NH-RP2\NHRP Server\gamemodes\nhrp.pwn(21639) : error 029: invalid expression, assumed zero
C:\Documents and Settings\Eric\Desktop\NH-RP2\NHRP Server\gamemodes\nhrp.pwn(21639) : warning 215: expression has no effect
C:\Documents and Settings\Eric\Desktop\NH-RP2\NHRP Server\gamemodes\nhrp.pwn(21639) : error 001: expected token: ";", but found "if"
Any help is greatly appreciated!

- Eric
Find
Reply
Flashy
Huge Clucker
****
Posts: 297
Threads: 22
Joined: Apr 2010
Reputation: 0
#2

13.06.2010, 13:09
What is the line 21639?
Find
Reply
Tnt12340
Big Clucker
***
Posts: 61
Threads: 11
Joined: Mar 2007
Reputation: 0
#3

13.06.2010, 13:12
pawn Код:
else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor))
That one; The second one, where its shutting the door.
Find
Reply
Flashy
Huge Clucker
****
Posts: 297
Threads: 22
Joined: Apr 2010
Reputation: 0
#4

13.06.2010, 13:13
Quote:
Originally Posted by Tnt12340
pawn Код:
else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor))
That one; The second one, where its shutting the door.
pawn Код:
else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor));
Copy&past this line into your old line.

You forgot to add a ";"
Find
Reply
Sergei
High-roller
*****
Posts: 2,862
Threads: 11
Joined: Mar 2008
Reputation: 0
#5

13.06.2010, 13:14
@Flashy, You fail so much.

@Tnt12340,
You can't do it like that. If you use else if it needs to be resume of another if condition.
Код:
			}
			GetPlayerName(playerid, sendername, sizeof(sendername));
			else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor))
			{
Find
Reply
Tnt12340
Big Clucker
***
Posts: 61
Threads: 11
Joined: Mar 2007
Reputation: 0
#6

13.06.2010, 13:18
Sooo err..

Like this? I'm new to scripting btw, so i may seem like a nub :P

pawn Код:
if(strcmp(cmd, "/pddoor", true) == 0)
    {
      if(IsPlayerConnected(playerid))
      {
            GetPlayerName(playerid, sendername, sizeof(sendername));
            if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (!pddoor))
            {
                pddoor = 1;
                gate = 4;
                MoveDynamicObject(3, 248.880111, 72.445343, 1002.560546,5);
                MoveDynamicObject(4, 243.894638, 72.445343, 1002.560546,5);
                format(string, sizeof(string), "* %s takes out a key, and unlocks the door.", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
                format(string, sizeof(string), "* %s opens the door.", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
            }
            }
            GetPlayerName(playerid, sendername, sizeof(sendername));
            else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor))
            {
            {
                pddoor = 0;
                MoveDynamicObject(3, 247.880111, 72.445343, 1002.560546,5);
                MoveDynamicObject(4, 244.894638, 72.445343, 1002.560546,5);
                format(string, sizeof(string), "* %s closes the door. *It Auto Locks*", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
            }
            else
            {
                SendClientMessage(playerid, COLOR_GRAD1, "You are not a police officer!");
            }
        }
        return 1;
    }
Find
Reply
[XST]O_x
High-roller
*****
Posts: 1,954
Threads: 79
Joined: Aug 2009
Reputation: 0
#7

13.06.2010, 13:20
Quote:
Originally Posted by Tnt12340
Sooo err..

Like this? I'm new to scripting btw, so i may seem like a nub :P

pawn Код:
if(strcmp(cmd, "/pddoor", true) == 0)
    {
      if(IsPlayerConnected(playerid))
      {
            GetPlayerName(playerid, sendername, sizeof(sendername));
            if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (!pddoor))
            {
                pddoor = 1;
                gate = 4;
                MoveDynamicObject(3, 248.880111, 72.445343, 1002.560546,5);
                MoveDynamicObject(4, 243.894638, 72.445343, 1002.560546,5);
                format(string, sizeof(string), "* %s takes out a key, and unlocks the door.", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
                format(string, sizeof(string), "* %s opens the door.", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
            }
            }
            GetPlayerName(playerid, sendername, sizeof(sendername));
            else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor))
            {
            {
                pddoor = 0;
                MoveDynamicObject(3, 247.880111, 72.445343, 1002.560546,5);
                MoveDynamicObject(4, 244.894638, 72.445343, 1002.560546,5);
                format(string, sizeof(string), "* %s closes the door. *It Auto Locks*", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
            }
            else
            {
                SendClientMessage(playerid, COLOR_GRAD1, "You are not a police officer!");
            }
        }
        return 1;
    }
Why twice GetPlayerName?
pawn Код:
if(strcmp(cmd, "/pddoor", true) == 0)
    {
      if(IsPlayerConnected(playerid))
      {
            GetPlayerName(playerid, sendername, sizeof(sendername));
            if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (!pddoor))
            {
                pddoor = 1;
                gate = 4;
                MoveDynamicObject(3, 248.880111, 72.445343, 1002.560546,5);
                MoveDynamicObject(4, 243.894638, 72.445343, 1002.560546,5);
                format(string, sizeof(string), "* %s takes out a key, and unlocks the door.", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
                format(string, sizeof(string), "* %s opens the door.", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
            }
            else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor))
            {
                pddoor = 0;
                MoveDynamicObject(3, 247.880111, 72.445343, 1002.560546,5);
                MoveDynamicObject(4, 244.894638, 72.445343, 1002.560546,5);
                format(string, sizeof(string), "* %s closes the door. *It Auto Locks*", sendername);
                ProxDetector(5.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
            }
            else
            {
                SendClientMessage(playerid, COLOR_GRAD1, "You are not a police officer!");
            }
        }
        return 1;
    }
Find
Reply
Flashy
Huge Clucker
****
Posts: 297
Threads: 22
Joined: Apr 2010
Reputation: 0
#8

13.06.2010, 13:23
Quote:
Originally Posted by $ЂЯĢ
@Flashy, You fail so much.

@Tnt12340,
You can't do it like that. If you use else if it needs to be resume of another if condition.
Код:
			}
			GetPlayerName(playerid, sendername, sizeof(sendername));
			else if(IsACop(playerid) && IsPlayerInSphere(playerid,246.3890,72.0390,1003.6406,4) && (pddoor))
			{
I donґt know what you want, but I had the same problem. I fixed it after adding ; so stfu.
Find
Reply
Tnt12340
Big Clucker
***
Posts: 61
Threads: 11
Joined: Mar 2007
Reputation: 0
#9

13.06.2010, 13:24
Works like a charm! Thanks!
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
  4. Help Archive