[Thrarod]

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!

[paweltbg94]

Код:

if (vehicle = 453) >>>>> if (vehicle == 453)
if (fishrnd = x) >>>>> if (fishrnd == x)

[Desert]

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);

[Thrarod]

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

[Thrarod]

Desert - GetPlayerVehicleModel is undefined

[Desert]

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

[Thrarod]

Not workin, getting loads of errors

[biltong]

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.

[Thrarod]

Can someone give me a full code pls?