ページビューカウンタ、コードが提供されています、それは最善の方法ですか？

Question

このテーブルはvisitsとし、ページIDはid、ユーザのIPアドレスはipの2つのフィールドがあるとします。

私はこのコードを書いており、それが最良の方法であるかどうか疑問に思っていますか？

$ip=$_SERVER['REMOTE_ADDR']; 
    $id=$_GET['id']; 

    $query = "SELECT 
    (SELECT COUNT(ip) FROM visits WHERE id = '{$id}' AND WHERE ip ='{$ip}') as visited, 
    (SELECT COUNT(ip) FROM visits WHERE id = '{$id}') as pageHits"; 

    $result=mysql_query($query, $connection); 
    $row=mysql_fetch_array($result); 

    $pageHits=$row['pageHits']; 
    $visited=$row['visited']; //it's either 0 or 1; 

    if($visited==0){ 
     $query ="INSERT INTO visits (ip , id) VALUES ('{$ip}', {$id})" 
     $result=mysql_query($query, $connection); 
     $pageHits++; 
    } 

    echo $pageHits; 

Answer 1

私はデータベースの既存のエントリをチェックしません。

（組み合わせ）ipとidにUNIQUEキーを追加し、ちょうどINSERT IGNOREを実行します。追跡しません

$ip=$_SERVER['REMOTE_ADDR']; 
$id=$_GET['id']; 

$query = 'INSERT IGNORE INTO visits (ip, id) VALUES ('.$ip.', '.intval($id).')'; 
mysql_query($query, $connection); 

// and now select the visits: 
$query = 'SELECT COUNT(ip) as pageHits FROM visits WHERE id = '.intval($id); 
$result = mysql_query($query, $connection); 
list($pageHits) = mysql_fetch_array($result); 
echo $pageHits; 

Answer 2

あなたの持っている使用方法によってはかなり異なります。内部ネットワークの簡単なカウンタの場合、これで十分です。

「pageHits」が実際に何を数えているかはっきりしていませんが、私が "pageHits"と呼ぶことのないユニークな訪問者の数と思われます。

誰もが見ることができるウェブサイトについては、アプローチは問題があります。明らかな問題の1つは、 "id"パラメータによるSQLインジェクションに対する脆弱性です。

もちろん、さまざまな程度の複雑さでこのようなカウントを行うライブラリが多数用意されています。詳細や実際のユースケースがなければ、より複雑なものをコード化する必要があるかどうかを判断するのは難しいです。