[RESOLVED] Submitting scores [Archive]

Click to See Complete Forum and Search --> : [RESOLVED] Submitting scores

sgtPepper

Nov 7th, 2009, 02:21 PM

Hi,
I want to submit game scores to different MySql tables (tab1-tab2) according to the type of the game (1-4).
The PHP code reads the score, some more data and the type of game, stored in $gametype. then it goes through elseif statements to choose the right table.
However, something is not working in this code.
Can someone see a syntax error that I'm not able to see?
Thanks!

<?

$username="******";
$password="******";
$database="******";
$gametype=0

// Connect to database
mysql_connect ("***********.com" , $username, $password) or die( "cannot connect" );
mysql_select_db($database) or die ("cannot open" );

//if(isset($_GET['name'], $_GET['score']))
if(isset($_GET['name'], $_GET['city'], $_GET['country'], $_GET['score'], $_GET['date'], $_GET['gametype']))
{

$name = mysql_real_escape_string($_GET['name']);
$city = mysql_real_escape_string($_GET['city']);
$country = mysql_real_escape_string($_GET['country']);
$score = (int) $_GET['score'];
$date = mysql_real_escape_string($_GET['date']);
$gametype = (int) $_GET['gametype']);
print ($gametype);

if ($gametype == 1) {
$result = mysql_query ("INSERT INTO tab1 (Name,City,Country,Score,Date) VALUES ('$name','$city','$country','$score','$date')");
} elseif ($gametype == 2) {
$result = mysql_query ("INSERT INTO tab2 (Name,City,Country,Score,Date) VALUES ('$name','$city','$country','$score','$date')");
} elseif ($gametype == 3) {
$result = mysql_query ("INSERT INTO tab3 (Name,City,Country,Score,Date) VALUES ('$name','$city','$country','$score','$date')");
} elseif ($gametype == 4) {
$result = mysql_query ("INSERT INTO tab4 (Name,City,Country,Score,Date) VALUES ('$name','$city','$country','$score','$date')");

}

if($result)
echo 'score submitted!';

mysql_close();

?>

kows

Nov 7th, 2009, 02:31 PM

you have an extra bracket in your code on the end of the following line:
$gametype = (int) $_GET['gametype']);

in the future, please post any and all errors you get. make a call to error_reporting(E_ALL) at the beginning of your script if you're not getting any.

anyway, you could clean up your code a little bit -- this seems like something a little better to maintain:
switch($gametype){
/* our supported game types */
case 1:
case 2:
case 3:
case 4:
$table = "tab" . $gametype;
break;
default:
/* unsupported/invalid game type */
echo "Unsupported game type!";
}

if(isset($table)){
$result = mysql_query ("INSERT INTO {$table} (Name,City,Country,Score,Date) VALUES ('$name','$city','$country','$score','$date')");
}

if($result){
echo "Score submitted!";
}
now, if you end up having more "game types," you only need to add a case to the switch statement, rather than add another if-else structure.

and, lastly, you need not use mysql_close(). PHP will close your MySQL connection for you.

edit: I just noticed that you also have a missing terminator (semi-colon, ";") on the end of this line at the beginning of your script:
$gametype=0

sgtPepper

Nov 8th, 2009, 02:59 AM

kows, thanks a lot for you help!

sgtPepper

Nov 9th, 2009, 06:10 AM

OK, yet another problem. Now I'm trying to displaing the MySQL table data on an HTML page, where Name, City, Country, Score and Date are table entries, and name, city, country, score, date are variables:

<html>
<body>
<?php
$username="******";
$password="******";
$database="******";

mysql_connect("*********.com" ,$username,$password);
@mysql_select_db($database) or die( "Unable to select database");
$query="SELECT * FROM tab1";
$result=mysql_query($query);

$num=mysql_numrows($result);

mysql_close();
?>
<table border="1" cellspacing="2" cellpadding="2">
<tr>
<th>Name</th>
<th>City</th>
<th>Country</th>
<th>Score</th>
<th>Date</th>
</tr>

<?php
$i=0;
while ($i < $num) {

$name=mysql_result($result,$i,"Name");
$city=mysql_result($result,$i,"City");
$country=mysql_result($result,$i,"Country");
$score=mysql_result($result,$i,"Score");
$date=mysql_result($result,$i,"Date");
?>

<tr>
<td><?php echo $name; ?></td>
<td><?php echo $city; ?></td>
<td><?php echo $country; ?></td>
<td><?php echo $score; ?></td>
<td><?php echo $date; ?></td>
</tr>

<?php
$i++;
}
?>
</body>
</html>

I only gate a page with an empty table and I can't figure out what's wrong with the script.
You help is appreciated!

sgtPepper

Nov 9th, 2009, 10:02 AM

False alarm. :blush:
I forgot to upload the file to the server... Sorry!