SA-MP Forums Archive
[Help] Bugged Fishing - 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: [Help] Bugged Fishing (/showthread.php?tid=133903)


[Help] Bugged Fishing - Thrarod - 14.03.2010

Today I made a new fishing system because couldn't find any, I made it and its the code,
Код:
	if (strcmp("/fish", cmdtext, true) == 0)
	{
		new vehicle;
		new fishrnd;
		new weightrnd;
		vehicle = GetPlayerVehicleID(playerid);
		fishrnd = random(10);
		weightrnd = random(10);
		if (vehicle = 453)
		{
		if (fishrnd = 1)
		{
		GivePlayerMoney(playerid, -10);
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You caught some Shrimps");
		GivePlayerMoney(playerid, (1 * weightrnd));
		}
		if (fishrnd = 2)
		{
		GivePlayerMoney(playerid, -10);
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You caught some Sardine");
		GivePlayerMoney(playerid, (2 * weightrnd));
		}
		if (fishrnd = 3)
		{
		GivePlayerMoney(playerid, -10);
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You caught some Herring");
		GivePlayerMoney(playerid, (3 * weightrnd));
		}
		if (fishrnd = 4)
		{
		GivePlayerMoney(playerid, -10);
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You caught some Mackerel");
		GivePlayerMoney(playerid, (4 * weightrnd));
		}
		if (fishrnd = 5)
		{
		GivePlayerMoney(playerid, -10);
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You caught some Trout");
		GivePlayerMoney(playerid, (5 * weightrnd));
		}
		if (fishrnd = 6)
		{
		GivePlayerMoney(playerid, -10);
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You caught a pair of pants and threw it back");
		}
		if (fishrnd = 7)
		{
		GivePlayerMoney(playerid, -10);
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You caught a used condom and threw it back");
		}
		if (fishrnd = 8)
		{
		GivePlayerMoney(playerid, -10);
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You caught a moneybag, lucky you");
		GivePlayerMoney(playerid, (10 * weightrnd));
		}
		}
		else if (vehicle != 453)
		{
		SendClientMessage(playerid, COLOR_OOC, "[Fishing] You are not in a Reefer");
		}
		return 1;
	}
The problem is, it doesnt check the vehicle and when I say the cmd it looks like all fishes caught but no money, only 8 * -10(bait money), Please fix it!

Re: [Help] Bugged Fishing - paweltbg94 - 14.03.2010

Код:
if (vehicle = 453) >>>>> if (vehicle == 453)
if (fishrnd = x) >>>>> if (fishrnd == x)


Re: [Help] Bugged Fishing - Desert - 14.03.2010

Im not sure on the multiplier couse i have never used anything else than + and -

Anyway you check if the vehicleid is 453 but you should check the vehicle model with GetPlayerVehicleModel(playerid);


So it will be vehicle = GetPlayerVehicleModel(playerid);

Re: [Help] Bugged Fishing - Thrarod - 14.03.2010

paweltbg94 - The = operator automatically doubles itself even we dont see
Desert - Im gonna try it

Re: [Help] Bugged Fishing - Thrarod - 16.03.2010

Desert - GetPlayerVehicleModel is undefined

Re: [Help] Bugged Fishing - Desert - 16.03.2010

new vehicle2 = GetPlayerVehicleID(playerid);
GetVehicleModel(vehicle2);

Re: [Help] Bugged Fishing - Thrarod - 17.03.2010

Not workin, getting loads of errors

Re: [Help] Bugged Fishing - biltong - 17.03.2010

Quote:
Originally Posted by Thrarod
paweltbg94 - The = operator automatically doubles itself even we dont see
No it doesn't. It gives me the possibly unintended assignment warning.

Re: [Help] Bugged Fishing - Thrarod - 08.04.2010

Can someone give me a full code pls?