Player.Cash or Cash Function

[Terror_Styles]

Hey Guys What's going on?

I was trying to Implement my Cash System But Faced a bit problem

I have created Two Functions SetCash & GetCash  Also Create Table in Database

Codes:
Function SetCash, GetCash, Cash

Code Select Expand

function GetCash( player )
{
   local v =  GetSQLColumnData( QuerySQL( DB, "SELECT Cash FROM Cash WHERE Name='" + player.Name+"'" ), 0 );
   if ( v ) return v.tointeger();
   else return 0;
}
function Cash( player )
{
local q = QuerySQL( DB, "SELECT * FROM Cash WHERE Name='" + player.Name+ "'" );
status[ player.ID ].Cash = GetSQLColumnData( q, 1 );
status[ player.ID ].Bank = GetSQLColumnData( q, 2 );
}
function SetCash( player, amount )
{
      status[ player.ID ].Cash = amount.tointeger();
      QuerySQL(DB, "UPDATE Cash SET Cash='" +  amount.tointeger() + "' WHERE Name='" + player.Name + "'");   
}

Database Loaded

Code Select Expand

function CreateTables()
{
QuerySQL( DB, "CREATE TABLE IF NOT EXISTS Cash( Name VARCHAR(32),Cash INT(15), Bank INT(15) )" );
}

I have added this on PlayerDeath

Code Select Expand

SetCash( player, GetCash( player ) -1000);

But When I type /kill it Doesn't Decrease my Cash but do in Database







What I wanted to fix?
I wanted Cash to be increase & decrease with real cash provided in Gta Vice City!

[Xmair]

Code Select Expand

SetCash( player, GetCash( player) - 1000 );

[Terror_Styles]

Code Select Expand

SetCash( player, GetCash( player) - 1000 );

What Bro?

[Xmair]

Code Select Expand

SetCash( player, GetCash( player) - 1000 );

What Bro?

This

Code Select Expand

SetCash( player, GetCash( player ) -1000);and this

Code Select Expand

SetCash( player, GetCash( player) - 1000 );are different.
Just replace that line with the one I gave and try.

[KAKAN]

In hand: -1000
It's correct.

[Xmair]

In hand: -1000
It's correct.

Then replace your SetCash function to this

Code Select Expand

function SetCash( player, amount )
{
      status[ player.ID ].Cash = amount.tointeger();
      QuerySQL(DB, "UPDATE Cash SET Cash='" +  amount.tointeger() + "' WHERE Name='" + player.Name + "'");   
      player.Cash = amount.tointeger( );
}

[Terror_Styles]

Thanks @Xmair
@KAKAN Too! For Reminding Him!

[EK.IceFlake]

Why are you using a string to store cash?

[.]

Why are you using a string to store cash?

Because then it looks similar to INI storage