This code don't send money - Printable Version

This code don't send money - 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)
+--- Thread: This code don't send money (/showthread.php?tid=307462)

This code don't send money - Face9000 - 30.12.2011

Hello,i've this lotto code to sent the jackpot money at the winner,i made some tests in my server and when i pickup the winning number,it shows the winning message but IT DOESN'T GIVE MONEY

pawn Код:

if(winner != -1) //If there was a winner
    {
      for(new i; i<MAX_PLAYERS; i++) //checks through all players
    {
        new Pname[24];
        GetPlayerName(i, Pname, 24);
        new str[128];
        SendClientMessageToAll(0x62FF32FF, "****LOTTO INFORMATION****"); //Lotto info
        format(str, sizeof(str), "WE HAVE A WINNER! %s (Id: %d) won $%d!!!!", Pname, i, Jackpot);
        LottoLog(str);
        GivePlayerMoney(i, Jackpot);
        SendClientMessageToAll(0x62FF32FF, str); //Lotto info
        SendClientMessageToAll(0x62FF32FF, "Make sure you get a ticket for next draw - /lotto [1-30]"); //Lotto info
        Jackpot = LOTTO_START; //Resets the jackpot
    }
    }

What's wrong?

Re: This code don't send money - James Coral - 30.12.2011

Right Code:

Код:

if(strcmp(cmd, "/givemoney", true) == 0)
	{
		new string[128];
		new playermoney;
		new sendername[MAX_PLAYER_NAME];
		new giveplayer[MAX_PLAYER_NAME];
 		new giveplayerid, moneys;
		tmp = strtok(cmdtext, idx);

		if(!strlen(tmp))
		{
			SendClientMessage(playerid, COLOR_RED, "USAGE: /givecash <playerid/playername> <amount>");
			return 1;
		}
		giveplayerid = strval(tmp);

		tmp = strtok(cmdtext, idx);
		if(!strlen(tmp))
		{
			SendClientMessage(playerid, COLOR_RED, "USAGE: /givecash <playerid/playername> <amount>");
			return 1;
		}
		moneys = strval(tmp);

		if (IsPlayerConnected(giveplayerid) && giveplayerid != INVALID_PLAYER_ID && giveplayerid != playerid)
		{
			GetPlayerName(giveplayerid, giveplayer, sizeof(giveplayer));
			GetPlayerName(playerid, sendername, sizeof(sendername));
			playermoney = GetPlayerMoney(playerid);

			if (moneys > 0 && playermoney >= moneys)
			{
				GivePlayerMoney(playerid, (0 - moneys));
				GivePlayerMoney(giveplayerid, moneys);
				format(string, sizeof(string), "You gave %s(%d), Money Amount: $%d.", giveplayer,giveplayerid, moneys);
				SendClientMessage(playerid, COLOR_GREEN, string);
				format(string, sizeof(string), "You got $%d from %s(%d).", moneys, sendername, playerid);
				SendClientMessage(giveplayerid, COLOR_GREEN, string);
				PlayerPlaySound(playerid,1084,0.0,0.0,0.0);
				printf("%s(%d) has transfered %d to %s(%d)",sendername, playerid, moneys, giveplayer, giveplayerid);
				}
			else
			{
				SendClientMessage(playerid, COLOR_RED, "Invalid amount!");
			}
		}
		else
		{
			format(string, sizeof(string), "ERROR: Player Is Not Connected or Its Yourself!");
			SendClientMessage(playerid, COLOR_RED, string);
		}
		return 1;
	}

Re: This code don't send money - James Coral - 30.12.2011

Sorry For Touble Posting But i got that you want /givemoney CODE

Sorry for Code btw!

Re: This code don't send money - Face9000 - 30.12.2011

Emh what is that?I've already my givemoney cmd,i need only to fix that part of code i provided in first post.

Re: This code don't send money - Ash. - 30.12.2011

Debug the code, print out something after every line. It must be silently exiting somewhere.

Re: This code don't send money - James Coral - 30.12.2011

Lol check other FilterScript i remember i have seen Jackpot FS

Re: This code don't send money - James Coral - 30.12.2011

like i find : http://forum.sa-mp.com/showthread.ph...hlight=JackPot

USE NEXT TIME SEARCH!
(You can take there a Code)

Re: This code don't send money - Face9000 - 30.12.2011

Quote:

Originally Posted by funky1234

Debug the code, print out something after every line. It must be silently exiting somewhere.

pawn Код:

GivePlayerMoney(i, Jackpot);

Is not enough? The lotto system works good,the numbers are draw'ed but the only fault is it don't give the jackpot money to the winner.

Quote:

Originally Posted by James Coral

like i find : http://forum.sa-mp.com/showthread.ph...hlight=JackPot

USE NEXT TIME SEARCH!
(You can take there a Code)

Before i made this topic i searched like 5 times.I've coded ALL lotto script and in my gamemode I dont use (and i dont want) others filterscripts,i make my functions so if i have a problem i know where is the fix.

Re: This code don't send money - Ash. - 30.12.2011

Quote:

Originally Posted by James Coral

like i find : http://forum.sa-mp.com/showthread.ph...hlight=JackPot

USE NEXT TIME SEARCH!
(You can take there a Code)

This isn't helpful. They want to script it theirselves, rather than become a plagiarist.

Quote:

Originally Posted by Logitech90

pawn Код:

GivePlayerMoney(i, Jackpot);

Is not enough? The lotto system works good,the numbers are draw'ed but the only fault is it don't give the jackpot money to the winner.

It should be. Print out what "i" and "Jackpot" are. And, like I say, try printing out every line to see if it's silently exerting.

Re: This code don't send money - Face9000 - 30.12.2011

Oh sorry for not explaining.

The "i" is a loop of all players:

pawn Код:

for(new i; i<MAX_PLAYERS; i++) 
    {

the "Jackpot" is a value that store the Jackpot money and gives it to the winner.