Ot takie cosik mi się napisało w kilka minutek. Zadanie: Zliczanie odwiedzin na stronie dla najpopularniejszych przeglądarek, odświerzanie tylko dla nowych IP, możliwość rozbudowy. Chciałbym wiedzieć, czy można zrobić lepiej. (aha. nie ma cookies, ale to zamierzone)

[PHP] pobierz, plaintext 
<?php
 
mysql_connect (&#092;"localhost\",\"root\",\"haslelko\") or die(\"nie można się połączyć z MySQL\");
mysql_select_db (&#092;"print\") or die(\"Nie można znaleźć bazy szukanej\");
 
$userip=$_SERVER[REMOTE_ADDR];
$wydostan=mysql_query(&#092;"SELECT * FROM pr_licz;\");
while($rekord=mysql_fetch_array($wydostan))
{
$wejsc=$rekord[1];
$ie=$rekord[2];
$opera=$rekord[3];
$firefox=$rekord[4];
}
$checkip=mysql_query(&#092;"SELECT * FROM pr_ipcheck WHERE ip='$userip'\");
$isin=mysql_num_rows($checkip);
if ($isin<1)
{
$wejsc++;
$dodaj=mysql_query(&#092;"INSERT INTO pr_ipcheck (nr, ip, banned, data) VALUES ('', '$userip', '1', now())\");
 
//EXPLORER
if (eregi(&#092;"MSIE\", $_SERVER[HTTP_USER_AGENT]) && !(eregi(\"Opera\", $_SERVER[HTTP_USER_AGENT])))
{
$ie++;
$query=mysql_query(&#092;"UPDATE pr_licz SET wejsc='$wejsc', ie='$ie';\");
}
 
//OPERA
if (eregi(&#092;"Opera\", $_SERVER[HTTP_USER_AGENT]))
{
$opera++;
$query=mysql_query(&#092;"UPDATE pr_licz SET wejsc='$wejsc', opera='$opera';\");
}
 
//FIREFOX
if (eregi(&#092;"firefox\", $_SERVER[HTTP_USER_AGENT]))
{
$firefox++;
$query=mysql_query(&#092;"UPDATE pr_licz SET wejsc='$wejsc', mozilla='$firefox';\");
}
}
 
?>
[PHP] pobierz, plaintext 

ja bym wywalil zapytanie do bazy $wydostan i zrobil tak:

[PHP] pobierz, plaintext 
<?php
//np dla MSIE
$query=mysql_query(&#092;"UPDATE pr_licz SET wejsc=wejsc+1, ie=ie+1;\");
// itd
 
?>
[PHP] pobierz, plaintext 

poza tym dorzucilbym w tabele pr_licz kolumnę o nazwie 'inne' (i odpowiedni kod do niej - pod else - na wypadek jak przegladarka nie wysle HTTP_USER_AGENT)

Pozdrawiam

Dzieki. Faktycznie bedzie prosciej

funkcje inne mam, ale byla na dole skryptu i ni zypomnialem wkleic

Najpierw sugerowalbym zapoznanie sie z jakimis stanardami kodowania i zastosowanie sie do nich. W miedzyczasie mozesz sie dowiedziec dlaczego przy podawaniu indeksow w postaci stringa do tablic powinno sie zapisywac je otoczone apostrofami/cudzyslowami.

sprawdz funkcje get_browser() - zwraca do tablicy duzo fajnych rzeczy

Ja mam kilka uwag:

- Tak jak pisal FiDO standard kodowania, w php.pl jest artukuł poświecony kodowaniu w php
- OOP
- komentarze
- to co napisałeś wykorzystuje tylko MySQL, dodanie innych.

Dzięki za uwagi. Wykorzystam.

Aha. Nie będę tego robił dla innych baz, bo na serwerze mam Mysql'a i raczej nie puszcze tego nigdzie indziej. Po prostu wydajność przedewszystkim Nie zmienia to faktu, że uwaga jest słuszna

Wazne jest to ze jesli ktos po Tobie ten kod przejmie zeby wiedziel co gdzie jest.

czasami warto zrobic kaszane...a rozpisac sobie komentarze na kartce:)
= antyhakierowy syf:)

Jak zrobisz kaszane z kodem to osoba ktora wspolpracuje z Toba nic z tego nie bedzie wiedziec i bardzo duzo czasu spedzi nad tym zeby zrozumiec co gdzie jest. Bo kilku tygodniach ten antyhakerowy syf bedzie tez utrudnieniem dla Ciebie.

o tym to ja wiem.....pracuje w ekipie...co prawda nie w php
ale innym jezyku:)

No to troszeczke sie dziwuje ze takie cos napisales.

nie ma sie co dziwic skoro moj team php=me alone