Il contatore non incrementa in PHP/MySQL

Question

Devo creare un contatore per la sezione membri (contare il numero di volte che un utente ha effettuato il login).

Ho il seguente script (counter.php):

<?php 
    $conn = mysql_connect("localhost", "myuser", "mypass"); 
    mysql_select_db("test"); 

    $sql = "SELECT views FROM members WHERE mid = " . $_GET['mid'];  
    $result = mysql_query($sql); 
    if (!$result) 
     { 
     mail(ADMIN, 'Cannot Get: ' . mysql_error(), mysql_error()); 
     } 
    while ($row = mysql_fetch_assoc($result)) 
     { 
     $count = $row['views']++; 
     } 
    $query = "UPDATE members SET views = '$count' WHERE mid = " . $_GET['mid']; 
    mysql_query($query); 
    mysql_close($conn); 

    // show the logo using header() and readfile(); // that part work 
?> 

DB:

CREATE TABLE `members` (
    `mid` int(11) NOT NULL AUTO_INCREMENT, 
    `views` int(11) DEFAULT '0', 
    /* etc...*/ 
    PRIMARY KEY (`mid`) 
) ENGINE=MyISAM DEFAULT CHARSET=latin1; 

Ora, quello che faccio nel mio file .htaccess è:

RewriteEngine On 
RewriteRule ^img/logo([0-9]+).jpg$ /counter.php?mid=$1 [L] 

ma per qualche ragione il mio contatore non conta correttamente. Cosa mi manca?

Answer 1

Si potrebbe probabilmente solo semplificare e procedere come segue:

$query = "UPDATE members SET views = views + 1 WHERE mid = " . $_GET['mid']; 
mysql_query($query); 

if (mysql_affected_rows() == 0) { 
    mail(ADMIN, 'Cannot Get: ' . mysql_error(), mysql_error()); 
} 

mysql_close($conn); 

Non c'è bisogno di fare il check iniziale.

Answer 2

uso questo

$count = $row['views'] + 1; 

o

$count = ++$row['views']; 

o

$query = "UPDATE members SET views = views + 1 WHERE mid = " . $_GET['mid']; 

sintassi:

$x = 1; 
$count = $x++; 
// $count = 1 

$x = 1; 
$count = ++$x; 
// $count = 2 

Answer 3

Il problema è nella linea

$count = $row['views']++; 

Questo dice in realtà:
- Assegnare il valore di vista a $ contare
- viste incremento.

ma si desidera:

$count = ++$row['views']; 

che dice:
- viste incremento.
- Assegna il valore (incrementato) della vista a $ count

Una sottile differenza. : ~)