PHP and mySQL glitch

[BrentC]

Hi people

Please look at the following...I will not give you all the code involved but I narrowed it down to this...

<?php


$con = mysql_connect("localhost","root","");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }



mysql_select_db("Persons", $con);



$result = mysql_query("SELECT * FROM  hits_on_musicpage");
$pagehit_total = mysql_num_rows($result);


//this echo always reports 2 times the rows that should be in the database

echo $pagehit_total;


echo "<table border='1' align=center>
<tr>
<th>IP</th>
<th>Host</th>
<th>Month</th>
<th>Day</th>
<th>Year</th>
</tr>";


while($row = mysql_fetch_array($result))
  {
  echo "<tr>";
  echo "<td>" . $row['ip'] . "</td>";
  echo "<td>" . $row['host'] . "</td>";
  echo "<td>" . $row['month'] . "</td>";
  echo "<td>" . $row['day'] . "</td>";
  echo "<td>" . $row['year'] . "</td>";
echo "</tr>";
  }
echo "</table>";

mysql_close($con);

?>

Thank you so much for your help!

[jamiemac2005]

Hey, can we see the result of that page?
If it's returning twice the amount of rows in the table then what is in the table and how is it structured?

Does the second echo (the one that echoes the row of data, for each entry) echo twice the amount or is it working fine?

I know it sounds random and nitty-gritty (or OCD) but your sql query has an extra space in it, i'm pretty sure whitespace doesn't matter in mysql but i can't remember so there might be something there.

Cheers,
Jamey

[BrentC]

here is the code that populates the database...

$ip = $_SERVER['REMOTE_ADDR'];
$host = gethostbyaddr($ip);
$mtime = date('m');
$dtime = date('d');
$ytime = date('Y');
$htime = date('H');
$mintime = date('i');
$stime = date('s');

$host_trimmed = substr($host, 0, 16);

mysql_select_db("Persons", $con);


$sql="INSERT INTO hits_on_musicpage (month, day, year, ip, host) VALUES ('$mtime', '$dtime', '$ytime', '$ip', '$host_trimmed')";

The output from all this is just a simple table but with two rows for every hit (Insersion).

Here is the url for the SELECT page

24.67.40.54/select-musicpage-hits.php

[jamiemac2005]

I don't see a problem with any of the code (as long as the population script is cut down there)...

Could you try opening up mysql.exe and selecting all the entries, since they're all repeated there might be something going on there.

Cheers,
Jamey

[BrentC]

I don't know how to open mysql.exe...

The code that follows population code above is some html

[BrentC]

I have found that when someone outside of my network downloads my page the database app works ok. Just when I access the page on my server that it is saved twice.

Also, I have another mySQL app and it works fine either way....

Brent

[Redcore]

This is a pretty bizarre problem. Your code is really simple and appears to be correct. After Googling it a bit, I found something that might help explain...

Browsers will request a page twice for several reason - FF when it applies the visitor's character encoding choice, IE when it requests your favicon.ico file, all browsers if you have a URL rewrite rule that is either missing or has a trailing slash on a folder (I don't remember which)...

There are two common ways to detect and prevent the double database insert -

1) If something about the data is unique so that one of the values will only exist once in the database, make that column a unique key so that the second insert will produce a duplicate key error and won't insert the second time, or

2) Use a session variable to detect that the form processing code has already been executed once. Set some session variable when the query executes and test if this variable is set at the start of the form processing code. Skip the form processing code if the variable is already set.

The reason printing the query does not show you anything in this case is because the page is requested/refreshed and unless you are very fast you won't see the two copies of the page being displayed.

http://codingforums.com/showpost.php...83&postcount=5

This looks like it would be the cause of your issue since you run this INSERT script every time someone accesses that page. The first suggestion is the one you should go with. Create a UNIQUE INDEX with columns (month, day, year, ip) and then any duplicated entries should be ignored.