zapis informacji o userze, Czy to jest ok, a może jakieś sugestie? Opcje

[Cysiaczek]

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 

[Darti]

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

[Cysiaczek]

Dzieki. Faktycznie bedzie prosciej (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

funkcje inne mam, ale byla na dole skryptu i ni zypomnialem wkleic (IMG:http://forum.php.pl/style_emoticons/default/rolleyes.gif)

[FiDO]

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.

[Darti]

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

[SongoQ]

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.

[Cysiaczek]

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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Nie zmienia to faktu, że uwaga jest słuszna (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[SongoQ]

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

[vala]

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

[SongoQ]

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.

[vala]

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

[SongoQ]

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.

[vala]

nie ma sie co dziwic skoro moj team php=me alone (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)