[RESOLVED] Update problems...

[Drac0]

I want people to have money on a game I'm making...
There is also a way to get more money, but the code doesn't work...

I don't get any errors... And it says what it is supposed to say, but it doesn't update the players money.

<?php
include("include/session.php");

   /* Class constructor */
   function MySQLDB(){
      /* Make connection to database */
      $this->connection = mysql_connect(localhost, root ) or die(mysql_error());
      mysql_select_db(test, $this->connection) or die(mysql_error());}

$getal      = rand(10,100);

if($getal <$p6+1){
$geld       = rand(625,1250);
mysql_query("UPDATE users SET `money`=`money`+'$geld' WHERE 'username'='{$_session->username}'")
or die(mysql_error());  

print "Successful! You got   \$$geld money.";
exit;
}
else{
$geld       = rand(625,1250);
mysql_query("UPDATE users SET `money`=`money`+'$geld' WHERE 'username'='{$_session->username}'")
or die(mysql_error());  

print "Successful! You got   \$$geld money.";
exit;
}
?>

I made it twice for my own reasons...
Ehm... Well it says when i go on the page: Successful! You got $1250 money. (Or less ofc - its random).

But it doesn't update my money even though im online.

This is some info:
Tablename:
users

Field names:
username
userid
money
timestamp

(There is more, but i see those as the most important).

Is there anything im doing wrong?

(Sorry for my english, and thanks in advance!)

[sheephat]

This is probably a bad way to go about it but my method would be to retrieve the amount of money the user has in their account, say £1200, then set it to variable I dunno $playermon = $playercashmysqlquery;

Then do the random thing, then add the generated value to the amount in their account, so:

$playernewmoney = $playermon + $geld;

Then update the player’s money with the new value using the UPDATE statement.

Not exactly ideal, but that’s how I would go about it as a starting point.

[Drac0]

I did understand what you were meaning, but not really how to set it up 😛

Can you give me some ideas? :S

Any other suggestions to?

[bradgrafelman]

@: Why do 2 queries instead of 1? His method in itself is fine - let MySQL do the addition as part of the UPDATE query, no need to lengthen the process without reason.

@: I see you have the standard "or die()" error messages, but you say there's no error message being outputted. This must mean that the query succeeded. This might mean that your query wasn't formed correctly. As such, you should echo out your query. This is why I usually store the query in a variable before executing it - that way you can add a simple echo() statement to echo out the query that was executed.

[Drac0]

Thanks! Problem solved 🙂

[sheephat]

bradgrafelman wrote:

@: Why do 2 queries instead of 1? His method in itself is fine - let MySQL do the addition as part of the UPDATE query, no need to lengthen the process without reason.

@: I see you have the standard "or die()" error messages, but you say there's no error message being outputted. This must mean that the query succeeded. This might mean that your query wasn't formed correctly. As such, you should echo out your query. This is why I usually store the query in a variable before executing it - that way you can add a simple echo() statement to echo out the query that was executed.

I did say it wasnt practicle, and thats the only way i know how to do it, i dont know mysql that much.