Bezoekersteller

Wat nou als mijn huisgenoot hetzelfde IP adres heeft ? :P

Dan heeft de huisgenoot pech:P XD... Nou.. heel simpel: Dan staan jullie onder 1 ip en word er dus maar 1 bezoeker bijgeteld.

Hmm...
Gaat dit script wel werken?

vind mysql dat wel leuk?

dit is geen insert statement...
Waarom steeds alles selecteren als je net zo goed steeds één ding kan selecteren?
Als ik vandaag meerdere keren op je site komt, dan tel je dat dus als één keer?

Edit: Waarom plaats je hier niet gewoon de code?

Jij bent dus zo'n figuur die het ziet dat een bezoeker op jou website vereert zou moeten zijn om op jou website te komen...

Een nettere oplossing is een UNIQUE contraint op het IP veld. Dan voegt ie 'm automatisch niet toe en kan je de eerste query skippen. Ook mis ik wat foutafhandeling...

PS
Waarom post je je script niet gewoon hier?

@Karl

Het werkt uitstekend.;)

Als je vanadaag meerdere keren op de site komt telt dat voor 1. Als je de volgende dag weer komt staat die van de eerste dag er ook nog in, dus word je nooit geen 2e keer meegeteld.

Hier niet dan?

Dat het werkt hoeft niet te betekenen dat het goed is. Misschien kun je ook wel met vierkante wielen fietsen, maar of het nou goed of handig is, tja...
En van mijn part mist dit script het script.
Bovendien kan elke beginner dit niveau wel evenaren...

Wtf, waarom alle data selecteren en dan PHP laten berekenen hoeveel rijën er zijn? Een simepele COUNT() is al genoeg. En die eerste query is nergens voor nodig, als je een goed model hebt tenminste. Probeer unique te gebruiken.

En aub plaats je code gewoon hier en dan niet populair doen met je pastebin. Het is hier niet voor niets zo gemaakt dat je hier code kunt posten, doh. Als je zo moeilijk gaat doen, ga gewoon weg.

Dan mijn final conclusion: dit is zo simpel dat een aap zonder armen het nog kan maken, serieus, en je kunt het niet eens fatsoenlijk en netjes maken, dus plaats het dan niet.

Jawel. Dit soort simpele scripts moet gewoon alles bevatten. Jij krijgt het voor elkaar om in 27 regels code nog 13 regels (ofzoiets) te vergeten.

En dan ook nog eens een na-aap pastebin...


En een aantal verkeerd.

En dan kom je op 0 goeie lijnen uit :)

werkt ook.

Verder wil ik je d'r even op wijzen dat 'het werkt' een drogrede is, namelijk onthouden van bewijslast.
Ik heb verder je reacite niet gelezen, omdat ie al zo onzinnig begint, voor de rest neem ik de moeite niet.

Edit:

Jah, het werkt, maar of het goed werkt. Oke, misschien kan je niet met vierkante wielen fietsen (wel autorijden), maar met een achthoek of ovaal wel fietsen. Het punt is, het gaat niet goed, het gaat brak, het kan beter. Zelfde met dit script, het werkt misschien, maar of het goed is...


Dat zou ik maar niet te hard roepen. En dat ging over die pastebin, niet over je script.

En waarom moeten nu overal comments (#) tussen staan? En kan je niet inspringen?

@Victor: Jij plaatst je script hier, dus jij wilt dat wij daarop reageren. Doen we dat, zeggen we dat het niet goed is, wordt je boos?

Dat het werkt wil nog niet zeggen dat het goed is. Iets kan nog werken, maar nog lang niet zijn waar het eigenlijk moet zijn. Ik bedoel: als jij in een supermarkt vakken vult, dan werk je ook. Maar of je je werk goed doet, dat is een ander verhaal. Laat die apen en fietsen dus maar buiten beschouwing.

Aan de ene kant zou het misschien korter kunnen. Maar die praat dan over de logica, ik doelde meer op foutafhandeling e.d., als je daar wat aan doet heb je zo een stuk of 13 regels meer, en ook al een heel stuk beter script.

Als het script er maar 26 heeft, plaats er dan ook maar 26 :-).

log gewoon alle bezoeken en vraag de unieke ips op met behulp van een group by of distinct.

sla evt ook wat browser info op. de gegevens zijn natuurlijk niet altijd juist, maar je kan er van uit gaan dat een groot deel is wat de standaard van de specifieke browser is, en er hoeft ook niks van afhankelijk te zijn behalve wat duidelijkheid als beheerder.

gewoon een ideetje.


ps: het zou leuk zijn als je ff een script in de jusite syntax neer zet.

Spaties neergezet achter de * :D

Nu de insert query nog.

MySql vindt het niet erg als je INSERT INTO table SET gebruikt, het geeft geen enkele fout en doet gewoon wat gevraagd wordt...

bron: http://milov.nl/2836

Maar ga eens voor je zelf na, wie heeft nou wat aan dit script? En dan gaan smart-ass-comments want daar worden we allemaal niet beter van. Ik probeer ook maar iets duidelijk te maken.

Dus omdat mysql het goed vind, moet is het goed? Dat lijkt me niet, sql is al dertig jaar oud, dus je doet maar lekker zoals de regels zeggen dat je een query moet doen. Je gaat toch bij een postgredatabase ook geen backticks doen, dat is ziek, alleen maar omdat die lijpo's van mysql zoiets gaan bedenken. Die zich niet aan de standaarden willen houden. Die query klopt niet, zie ook zelf effe wat die mensen zeggen.
Het valt me op dat mensen altijd over IE zitten te zeiken, dat die zich niet aan de standaarden houd, maar moet je eens naar mysql kijken.

Bron:
http://www.w3schools.com/SQL/sql_insert.asp
http://en.wikipedia.org/wiki/Insert_%28SQL%28
http://en.wikipedia.org/wiki/SQL

Waar is Frank als je 'm nodig hebt....

@ Karl

Als die syntax niet goed is, waarom bestaat die dan nog überhaupt?

Waarom bestaan backticks nog?

@ToySoldier: probeer MySQL niet op te hemelen. Waarom zou je een syntaxis van de standaard af laten wijken? Dan ben je toch überhaupt al niet slim bezig?

Ik heb het script aangepast, rekening gehouden met alle reacties, kijken jullie of het nu goed is?? :D

-edit 1 uur later-

Aub.. Dan doe ik al die moeite wil ik weten of het goed is ja???... :P

Gebruik eens COUNT().

Aangepast, is het nu 100% zoals de real pros het doen? :P

Ik heb wel veel bijgeleerd door jullie kritiek :P

Je houd nog geen rekening met proxy's.

wat moet er dan gebeuren met de proxies? Niet meegetelt worden?

@victor: de "real pros" doen het zo.

Bovendien mis ik nog steeds foutafhandeling.

@Mathijs: gebruik dan gewoon Google Analytics...

'Real pro's' maken zoiets niet, want wat boeit het nou voor een gebruiker om te zien hoeveel mensen er voor hem waren? Das het zelfde als die lelijke buttons van xHTML en CSS valid neerzetten, nergens voor nodig, iedereen verwacht dat dat al zo is, hoef je die buttons niet voor neer te zetten.

Foutafhandeling is er idd nog niet, dus ga daar aan werken en als je stats voor je zelf wilt dan heb je niet veel aan zo'n stukje code.

@Karl over mensen die over IE zeiken: ik zeik ook over IE, maar ik houd me altijd aan de xHTML en CSS standaarden, dus ik denk dat die vergelijking niet echt klopt van je, maar ik snap wat je bedoelt.

Verder @ Victor: je probeert het nu iig wel goed te doen.

Een gebruiker boeit het niets, maar het is altijd wel leuk om als webmaster te zien hoeveel mensen er op je site zijn geweest...

Die vergelijking klopt wel. Dat jij je wel aan de standaart houd heeft er niks mee te maken.

Haa leuk, hoe origineel.. De 3945248419732518732197541894513524ste bezoekersteller op phphulp =)
Bedankt =)

@ paradox,

14 gingen dit script al voor ;) maar je zat in de buurt, blijft onzinnig

Het een sluit het andere niet uit.

@Midas: Dat kan wel zo zijn, maar waarom een boel downloaden en installeren en configureren als je je zo kunt aanmelden, klein stukje code plaatsen, en dan precies dezelfde gegevens hebt?

Omdat google analytics op de google server werkt.

En misschien wil de persoon waarvoor je het maakt, dat soort informatie niet op een google server hebben staan. (Rechten etc.)

Dus zo'n aparte 'systeem' heeft een aanvulling in deze wereld :)

Ff ontopic allemaal, voeg zoals iemand anders al zij eens een $_SERVER['HTTP_USER_AGENT'] toe voor wat meer info. Verder weet ik niet wie er proxy server riep maar wat wou je daar mee gaan doen? filteren?

hey ja

Jezpur


idd frank ook al behoorlijke tijd geen commentaar meer zien geven...
sorry was even offtopic..

@Yorick
is iemand die jou website bezoek met en proxy geen bezoeker?

Ik heb een soort foutcontrole toegevoegd bij de SELECT query, bij de eerste was ik niet zeker of ik het wel moest doen, aangezien ik denk dat als je dan al in de tabel staat je een foutmelding krijgt (kan zijn van niet).

Ook heb ik nu een manier erbij gezet om de bezoeken per dag weer te geven.

En lees nu eens wat UNIQUE is: http://www.1keydata.com/sql/sql-unique.html

Ik denk dat ik zo nu wel al doorheb wat unique is, wat doe ik fout? Krijg je dan toch geen foutmelding? Of gebruik ik het verkeerd bij maken tabel?

Als je dat wat Karl je geeft nu eens leest, weet je wat je fout doet...

Edit:

Zo doen we dat niet. Geef COUNT(id) een alias en gebruik die als key in de array.

Ok, COUNT(id) heeft een alias, wil je nu zeggen wat er mis is met de unique? :) ik heb die link bekeken, maar ...

Ik denk dat je bedoelt dat je unique niet mag gebruiken voor een combinatie van rijen, maar alleen voor enkele kolommen, maar wat is het alternatief?

Waarde 2 is volgens mij beetje onzin, 1 of true, geen 2.


Moet je voor de grap eens op dat woord in de code klikken, zie je dat het overbodig is connectie zelf te sluiten ;)

Bedankt voor het melden, weet jij misschien wat er mis is met mijn UNIQUE :P

Weet iemand wat er mis is met mijn UNIQUE?? =P

Je UNIQUE staat niet op alle kolommen, terwijl je dat wel wil.

Niemand mag 2x met dezelfde useragent,ip-adres,datum etc gelogd worden... daar is een UNIQUE voor.

hij heeft maar 3 kolommen en een UNIQUE op date en ip, dus dat is het probleem niet...

Het onderstaande stukje mysql moet er uit en hij werkt!

Dan snap jij de ballen niet van SQL, laat staan databases.