Pagina 1 van 1

Bezoekers van vandaag

BerichtGeplaatst: 12 mei 2008 09:29
door Stefan
Hey,

Ik heb geprobeert wat te maken voor het bekijken hoeveel bezoekers er vandaag zijn geweest:
Code: Selecteer alles
$query_visit = mysql_query("
   SELECT    ipaddress
   FROM    wm_visitors
   WHERE    ipaddress = '".$_SERVER['REMOTE_ADDR']."'
   AND   datetime - INTERVAL 1 DAY
");
$result_visit = mysql_num_rows($query_visit);

if($result_visit == 0){
mysql_query("
   INSERT INTO   wm_visitors
           (ipaddress, datetime)
   VALUES           ('".$_SERVER['REMOTE_ADDR']."', NOW())
") or die(mysql_error());
}


Ik vroeg me af of dit juist is? Ik krijg voor de rest geen errors...

Re: Bezoekers van vandaag

BerichtGeplaatst: 12 mei 2008 16:16
door ElbertF
't Is niet de meest efficiente manier, als je veel bezoekers hebt zul je merken dat je site na een tijd een stuk trager wordt. Wat je zou kunnen doen is rijen van voorgaande dagen steeds verwijderen.

Als je 't trouwens superveilig wilt doen, kun je $_SERVER['REMOTE_ADDR'] het beste escapen (met mysql_real_escape_string()). Een IP-adres zou je in theorie kunnen spoofen om zo een SQL-injectie uit te voeren. :}

Re: Bezoekers van vandaag

BerichtGeplaatst: 12 mei 2008 16:41
door Stefan
Jah die functie van de rijen verwijderen heb ik inderdaad ook gemaakt, maar in iedergeval bedankt voor je feedback ;)

Re: Bezoekers van vandaag

BerichtGeplaatst: 12 mei 2008 17:18
door ElbertF
Nog een kleine tip, omsluit tabel- en veldnamen met ` (waarschijnlijk linksboven op je toetsenbord). Zo kom je nooit in de problemen als je veldnaam toevallig een MySQL statement is. Niet altijd noodzakelijk maar een goede gewoonte om aan te leren. ;)

Error: SELECT * FROM start;
Geen error: SELECT * FROM `start`;

Re: Bezoekers van vandaag

BerichtGeplaatst: 12 mei 2008 18:56
door frizzy
Dankje voor de tips, vooral dat escapen ga ik toepassen! Heb ik ook nog wat aan ;)

Re: Bezoekers van vandaag

BerichtGeplaatst: 13 mei 2008 12:18
door ikkeikkemij
ElbertF schreef:(waarschijnlijk linksboven op je toetsenbord)

sst! Hij is helderziend .. :x

Re: Bezoekers van vandaag

BerichtGeplaatst: 14 mei 2008 16:29
door libia
iedereen raad juist altijd aan om die backticks (`) weg te laten, omdat het 'afval' is ;) gebruik het alleen als het nodig is, een voordeel is bijvoorbeeld ook dat je dezelfde query dan ook in niet mysql databases kunt gebruiken, de rest ondersteunt backticks namelijk niet.. en het is natuurlijk meer werk :)

Re: Bezoekers van vandaag

BerichtGeplaatst: 14 mei 2008 16:34
door ElbertF
Alleen soms gebruiken is natuurlijk niet consequent, ik heb me in het verleden regelmatig op 't hoofd gekrabt met vreemde errors die op die manier ontstonden.. :}