názor na kód

Zdravím,

prosím, koukněte mi na tenhle kódík. Je to moje prvotina, všechno funguje, ale jelikož
si programuju jen tak doma, nemám žádnou kontrolu, jestli jsem to tzv. "vzal za správný konec". Je to počítadlo přístupů, přístupy z jední IP adresy v rámci 1 dne zvyšují "celkové přístupy" ale nezvyšují "unikátní přístupy". Budu Vám vděčný za komentáře.

<?php
class CSpojeni
{
public function __construct()
{
MySQL_Connect("localhost","","")
or exit("Chybka: nepodařilo se připojit k DB serveru.");
MySQL_Select_DB("pocitadlo")
or exit("Chybka: nepodařilo se připojit k DB.");
}
}

class CPocitadlo
{
private $ip_uzivatele;
private $datum;
private $cas;
private $predchozi_pristup;
private $c_pocet; //celkové přístupy
private $u_pocet; //unikátní přístupy, stejná IP druhý den je unikátní přístup

public function __construct()
{
$this->ip_uzivatele = $_SERVER["REMOTE_ADDR"];
$this->datum = Date("Y-m-d");
$this->cas = Date("H:i:s");
self::over();
self::vypis();
}
private function vypis()
{
echo "Tvoje IP adresa: $this->ip_uzivatele<br>\n
Dnešní datum: $this->datum.<br>\n
Čas přístupu: $this->cas<br>\n
Celkový počet přístupů: $this->c_pocet<br>\n
Počet unikátních přistupů: $this->u_pocet<br>\n";
}
private function over()
{
$spoj = new CSpojeni();
$vysledek = MySQL_Query("select max(datum) as 'stare' from pocitadlo where ip = '$this->ip_uzivatele';");
$zaznam = MySQL_Fetch_Array($vysledek);
if($zaznam["stare"]== NULL) //tzn. tato IP adresa není v dbs
{
$this->predchozi_pristup = 0;
self::zapis();
}
else
{
$this->predchozi_pristup = $zaznam["stare"];
if($this->predchozi_pristup == $this->datum) //tzn. tato IP adresa již dnes byla uložena do dbs
{
self::zapis();
}
else // tzn. tato IP adresa je v dbs, ale ne z dnešního dne = nový unik.přístup
{
$this->predchozi_pristup = 0;
self::zapis();
}
}
}
private function zapis()
{
$spoj = new CSpojeni();
$vysledek = MySQL_Query("select max(c_pocet) as 'max_c', max(u_pocet) as 'max_u' from pocitadlo;");
$zaznam = MySQL_Fetch_Array($vysledek);
$this->c_pocet = $zaznam["max_c"];
$this->u_pocet = $zaznam["max_u"];
if($this->predchozi_pristup == 0)
{
$this->c_pocet++;
$this->u_pocet++;
MySQL_Query("insert into pocitadlo (c_pocet,ip,datum,cas) values ($this->c_pocet,'$this->ip_uzivatele', '$this->datum','$this->cas');");
}
else
{
$this->c_pocet++;
MySQL_Query("update pocitadlo set c_pocet = $this->c_pocet where u_pocet=$this->u_pocet; ");
}
}
}


$pocitadlo = new CPocitadlo();


?>

Díky Marty,

asi máš pravdu, bude to přehlednější. Co třeba obecně ten způsob počítání unikátních přístupů: stejná IP adresa každý den nový zápočet?

Jinak jsem čekal víc kritiky ;)

kk..

Díky za tip.

kk..

Mila:
oops, jsem odhalen :o) ... sorry, fakt to nebylo úplně přesně, už jsem dřív něco v PHP3 dělal ( uložit do mysql, vybrat z mysql), ale nebral jsem to jako "programování". Myslel jsem prvotina v PHP5. Kdybych si myslel, že se mám čím vytáhnout, tak bych něco vytáhnul a asi ne tenhle "kódík".

Díky za rady, díky za kritiku. Mohl(a) bys, prosím, ještě upřesnit?

re: - bude mít nějakou metodu, která na něj provede query
......tomuhle nerozumím
re: - Že si bude pamatovat svůj link na databázi
..... jako, že místo "localhost" bude ... taky nerozumím
re: - že bude mít metodu, která spojení zruší
....myslel jsem, že spojení se zruší automaticky po skončení skriptu

kk..