[PHP][MySQL] Czy to jest dobry licznik? Opcje

[killer99977]

Witam zrobiłem licznik odwiedzin w PHP wszystko działa tak jak powinno tylko czy to jest dobry i czy nie za prosty licznik a oto kod PHP:

[PHP] pobierz, plaintext 
<?php
 
     mysql_connect("$host", "$user", "$password") or die( mysql_error());
     mysql_select_db("$db") or die( mysql_error());
 
     setcookie('licznik', '1', time()+86400);
 
     if($_COOKIE['licznik'] != 1){
 
     $old_status = mysql_fetch_assoc(mysql_query("SELECT * FROM licznik LIMIT 1;")) or die( mysql_error());
     $new_status = $old_status['odwiedzilo'] + 1;
     mysql_query("UPDATE licznik SET odwiedzilo = '{$new_status}' WHERE id = '1';") or die( mysql_error());
     $status = mysql_fetch_assoc(mysql_query("SELECT * FROM licznik LIMIT 1;")) or die( mysql_error());
     echo $status['odwiedzilo'];
 
     } else {
 
     $status = mysql_fetch_assoc(mysql_query("SELECT * FROM licznik LIMIT 1;")) or die( mysql_error());
     echo $status['odwiedzilo'];
 
     }
 
 
?>
[PHP] pobierz, plaintext 

Dopiero uczę sie PHP i prosze i wszelkie inforacje co jest zle itp

[piku235]

Licznik jest dobry, ale zamiast tych 3 linijek mogłeś dać tylko 1:

[PHP] pobierz, plaintext 
$old_status = mysql_fetch_assoc(mysql_query("SELECT * FROM licznik LIMIT 1;")) or die( mysql_error());
$new_status = $old_status['odwiedzilo'] + 1;
mysql_query("UPDATE licznik SET odwiedzilo = '{$new_status}' WHERE id = '1';") or die( mysql_error());
[PHP] pobierz, plaintext 

na:

[PHP] pobierz, plaintext 
mysql_query("UPDATE licznik SET odwiedzilo = odwiedzilo + 1 WHERE id = 1") or die( mysql_error());
[PHP] pobierz, plaintext