Bekijk alle posts

Probeer een keer input type="submit" in plaats van button type="submit".

Het zou sowieso al sneller moeten gaan als je eerst het type nagaat en pas daarna de MATCH AGAINST doet.

Wel oppassen met gelinkte records, dan...

Je kan ook iets doen in deze aard:

$deletesql = "DELETE FROM online WHERE NOW()-datum > 'x'";

Hierbij stelt x het aantal seconden voor dat een gebruiker maximaal inactief mag zijn.

Misschien de file even posten waarin deze fout zich voordoet?

Maar hetgene je wil laten doen door een cronjob moet sowieso worden uitgevoerd. Dan kan je het beter één maal per minuut doen, dan bij elke hit na te gaan of het script al dan niet moet worden uitgevoerd om het uiteindelijk ook om de minuut uit te voeren.
Hoe zwaar de query is maakt in mijn ogen dan ook niet veel uit.

Je kan overigens best eerst controleren of de gegevens correct zijn ingevuld, en dan pas overgaan naar het controleren of de gebruikersnaam al in gebruik is.

Dat zijn twee dingen die los van elkaar staan. Mijn opmerking maakt het een heel klein beetje efficiënter, het andere is een beveiligingslek waardoor men, door een speciale gebruikersnaam op te geven, een query op jouw database kan gaan uitvoeren. Je hebt een functie 'schoon' gedefinieerd in Functions.inc.php, maar je gebruikt die niet op de data die je in je query gebruikt.

Zo bedoel ik het inderdaad, schoon() ook gebruiken bij het nagaan of er al een user bestaat met die gebruikersnaam en/of e-mailadres. Je kan die check trouwens beter met één query afhandelen door een OR te gebruiken.

Er zijn natuurlijk meerdere wegen naar Rome en de code die hierboven staat zou nog een stuk beter kunnen maar ik vind dat hij al een aardig eind is niet? En ik denk dat hij zo wel weer vooruit kan

Het is maar iets kleins, doch heel makkelijk om jezelf standaard aan te leren. Zo oogt het erg onprofessioneel, en ik denk dat de TS dit topic net gestart heeft om dit soort dingen bij te leren.