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

Threaded Mode
Please close this thread problem done.
saiberfun
High-roller
*****
Posts: 3,312
Threads: 33
Joined: Nov 2008
Reputation: 0
#1

02.07.2009, 16:45
pawn Code:
public OnPlayerPickUpPickup(playerid, pickupid)
{
new str[256];
format(str,256,"You found %d Bombs",AccountInfo[playerid][unlock]);
if(pickupid == p1)
{
if(AccountInfo[playerid][up1] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][unlock] ++;
AccountInfo[playerid][up1] = 1;
}
else
if(AccountInfo[playerid][up1] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}}
when i pickup the pickup it says "You found 0Bombs" eventho AccountInfo[playerid][unlock] already equals 1
so it should be 1Bombs and it shouldnt be You found blah...
because AccountInfo[playerid][up1] equals 1...
could u tell me whats the problem?
Find
Reply
woot
Gangsta
*****
Posts: 785
Threads: 22
Joined: Jun 2007
Reputation: 0
#2

02.07.2009, 16:58
Put your SendClientMessage after AccountInfo[playerid][unlock] ++;
Find
Reply
saiberfun
High-roller
*****
Posts: 3,312
Threads: 33
Joined: Nov 2008
Reputation: 0
#3

02.07.2009, 17:06
didnt really help still the same shit happens
Find
Reply
refshal
High-roller
*****
Posts: 1,070
Threads: 12
Joined: Jun 2009
Reputation: 0
#4

02.07.2009, 17:09
pawn Code:
public OnPlayerPickUpPickup(playerid, pickupid)
{
new str[256];
format(str,256,"You found %d Bombs",AccountInfo[playerid][unlock]);
if(pickupid == p1)
{
if(AccountInfo[playerid][up1] == 0)
{
AccountInfo[playerid][unlock] ++;
AccountInfo[playerid][up1] = 1;
SendClientMessage(playerid,LIGHTGREEN,str);
}
else
if(AccountInfo[playerid][up1] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
}
Find
Reply
saiberfun
High-roller
*****
Posts: 3,312
Threads: 33
Joined: Nov 2008
Reputation: 0
#5

02.07.2009, 17:13
still the same problems
imma send the whole OnPlayerPickupPickup maybe u see an error then..
pawn Code:
public OnPlayerPickUpPickup(playerid, pickupid)
{
new str[256];
format(str,256,"You found %d Bombs",AccountInfo[playerid][unlock]);
if(pickupid == 1)
{
SetPlayerHealth(playerid,100);
}
if(pickupid == 2)
{
SetPlayerArmour(playerid,50);
}
if(pickupid == 3)
{
SetPlayerArmour(playerid,50);
}
if(pickupid == 4)
{
SetPlayerHealth(playerid,100);
}
if(pickupid == 5)
{
SetPlayerHealth(playerid,100);
}
if(pickupid == p1)
{
if(AccountInfo[playerid][up1] == 0)
{
AccountInfo[playerid][unlock] ++;
AccountInfo[playerid][up1] = 1;
SendClientMessage(playerid,LIGHTGREEN,str);
}
else
if(AccountInfo[playerid][up1] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
if(pickupid == p2)
{
if(AccountInfo[playerid][up2] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][up2] = 1;
AccountInfo[playerid][unlock] ++;
}
else
if(AccountInfo[playerid][up2] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
if(pickupid == p3)
{
if(AccountInfo[playerid][up3] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][up3] = 1;
AccountInfo[playerid][unlock] ++;
}
else
if(AccountInfo[playerid][up3] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
if(pickupid == p4)
{
if(AccountInfo[playerid][up4] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][up4] = 1;
AccountInfo[playerid][unlock] ++;
}
else
if(AccountInfo[playerid][up4] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
if(pickupid == p5)
{
if(AccountInfo[playerid][up5] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][up5] = 1;
AccountInfo[playerid][unlock] ++;
}
else
if(AccountInfo[playerid][up5] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
if(pickupid == p6)
{
if(AccountInfo[playerid][up6] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][up6] = 1;
AccountInfo[playerid][unlock] ++;
}
else
if(AccountInfo[playerid][up6] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
if(pickupid == p7)
{
if(AccountInfo[playerid][up7] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][up7] = 1;
AccountInfo[playerid][unlock] ++;
}
else
if(AccountInfo[playerid][up7] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
if(pickupid == p8)
{
if(AccountInfo[playerid][up8] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][up8] = 1;
AccountInfo[playerid][unlock] ++;
}
else
if(AccountInfo[playerid][up8] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
if(pickupid == p9)
{
if(AccountInfo[playerid][up9] == 0)
{
SendClientMessage(playerid,LIGHTGREEN,str);
AccountInfo[playerid][up9] = 1;
AccountInfo[playerid][unlock] ++;
}
else
if(AccountInfo[playerid][up9] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
}
return 1;
}
Find
Reply
saiberfun
High-roller
*****
Posts: 3,312
Threads: 33
Joined: Nov 2008
Reputation: 0
#6

02.07.2009, 18:21
tried to change the way now
pawn Code:
public OnPlayerPickUpPickup(playerid, pickupid)
{
new str[256];
format(str,256,"You found %d Bombs",AccountInfo[playerid][unlock]);
if(pickupid == p1)
{
if(AccountInfo[playerid][up1] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
else
if(AccountInfo[playerid][up1] == 0)
{
AccountInfo[playerid][unlock] ++;
AccountInfo[playerid][up1] = 1;
SendClientMessage(playerid,LIGHTGREEN,str);
}
}
}
but still it does go around the
pawn Code:
if(AccountInfo[playerid][up1] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
Please Help
Find
Reply
Ignas1337
Gangsta
****
Posts: 546
Threads: 22
Joined: Jul 2007
Reputation: 0
#7

02.07.2009, 18:37
why do you format a message and then change a var and only then send a message? format AFTER variable changes and send message then.
Find
Reply
refshal
High-roller
*****
Posts: 1,070
Threads: 12
Joined: Jun 2009
Reputation: 0
#8

02.07.2009, 18:39
pawn Code:
public OnPlayerPickUpPickup(playerid, pickupid)
{
new str[256];
format(str,256,"You found %d Bombs",AccountInfo[playerid][unlock]);
if(pickupid == p1)
{
if(AccountInfo[playerid][up1] == 1)
{
SendClientMessage(playerid,ADMIN_RED,"You already picked up this pickup.");
}
else
if(AccountInfo[playerid][up1] == 0)
{
AccountInfo[playerid][unlock] ++;
AccountInfo[playerid][up1] = 1;
SendClientMessage(playerid,LIGHTGREEN,str);
return 1;
}
Try that one.
Find
Reply
Nero_3D
High-roller
*****
Posts: 2,856
Threads: 6
Joined: Jun 2007
Reputation: 0
#9

02.07.2009, 18:49
@Coop, I recommend you to read this post again

Quote:
Originally Posted by Izanagi
why do you format a message and then change a var and only then send a message? format AFTER variable changes and send message then.
Find
Reply
saiberfun
High-roller
*****
Posts: 3,312
Threads: 33
Joined: Nov 2008
Reputation: 0
#10

02.07.2009, 18:52
Quote:
Originally Posted by Izanagi
why do you format a message and then change a var and only then send a message? format AFTER variable changes and send message then.
well i fixed that already only thing annoying me atm is that it does go around the if(AccountInfo[playerid][up1] == 1)
like ti doesnt show

and @coop: sry didnt put the return in the pawn code here but in the fs it is
Find
Reply
refshal
High-roller
*****
Posts: 1,070
Threads: 12
Joined: Jun 2009
Reputation: 0
#11

02.07.2009, 18:55
Quote:
Originally Posted by ♣ ⓐⓢⓢ
@Coop, I recommend you to read this post again
It was just a suggestion...
Find
Reply
saiberfun
High-roller
*****
Posts: 3,312
Threads: 33
Joined: Nov 2008
Reputation: 0
#12

02.07.2009, 18:56
Quote:
Originally Posted by cοοp
Quote:
Originally Posted by ♣ ⓐⓢⓢ
@Coop, I recommend you to read this post again
It was just a suggestion...
ye ty for that man^-^

and erm..
dunno how but i fixed the prob on my own.
weird...
Find
Reply
Ignas1337
Gangsta
****
Posts: 546
Threads: 22
Joined: Jul 2007
Reputation: 0
#13

02.07.2009, 19:13
well grats in doing so I go watch a movie now so I will be out of reach. Good luck whith whatever yo're scripting
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