Is my syntax correct?

Anon

I put a * by where the problem seems to be. It's not updating the db properly and the credits/tax seems to just dissapear.

if($total_cost<0)

{

$totalprofit = $total_cost - $total_cost - $total_cost;
echo "Tax Rate: " . $playerrace[tax_rate] . "%" . "<BR>";
$taxtopay = $totalprofit / 100 $playerrace[tax_rate];
echo "Tax to pay: " . NUMBER($taxtopay) . "<BR>";
$totalprofit = $totalprofit - NUMBER($taxtopay);
echo "total profit with tax: " . NUMBER($totalprofit) . "<BR>";
echo "giving " . NUMBER($taxtopay) . "credits to $playerrace[leader] <BR>";
$taxtopay=NUMBER($taxtopay);
mysql_query("UPDATE ships SET credits=credits+$taxtopay WHERE character_name=$playerrace[leader]");
$trade_result = mysql_query("UPDATE ships SET turns=turns-1, turns_used=turns_used+1, rating=rating+1, credits=credits+$totalprofit, ship_ore=ship_ore+$trade_ore, ship_organics=ship_organics+$trade_organics, ship_goods=ship_goods+$trade_goods, ship_energy=ship_energy+$trade_energy where ship_id=$playerinfo[ship_id]");

}

yelch, looks ugly here, but basically there seems to be a problem with the $taxtopay in the mysql_query noted with a * by it. Or is it something else?

Anon

mysql_query("UPDATE ships SET credits='".($credits+$taxtopay)."' WHERE character_name='$playerrace[leader]'");

That should do it.

dstockto

You said: mysql_query("UPDATE ships SET credits='".($credits+$taxtopay)."' WHERE character_name='$playerrace[leader]'");

Actually, I think he is trying to do something a little different than what you showed. The "credits" is the value from in the database, and I don't think he has a variable named $credits. Try this:

mysql_query("UPDATE ships SET credits=credits+".$taxtopay." WHERE character_name=$playerrace[leader]");

Another way to do it would be to create a variable like $query that will hold your query as you build it. ie,

$query="UPDATE ships SET credits=credits+";
$query.=$taxtopay;
$query.=" WHERE character_name='";
$query.=$playerrace['leader'];
$query.="'";
$result=mysql_query($query);

This should work, and you can do it in less lines than that too...

dave