Gratis berichtenbalk

Gestart doorJeffreyy16 reacties60 weergaven

17 berichten · pagina 2 van 2

Berichten
6.722
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Kun je ook een previeuw laten zien?
Dan weet ik wat het is yummie
0
Jeffreyy

Lid · Topicstarter

Berichten
6.144
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Marijnnn:
if($info[admin] == 1) {

kun je beter schrijven als

if($info[admin] == 1) {

om getallen hoeven eenmaal geen tekens, het is zelfs sneller zonder tekens.

if($aantal <= 0) {

hoe kunnen het aantal berichten minder dan 0 zijn?

$get = mysql_query("SELECT * FROM berichten") or die(mysql_error());
$aantal = mysql_num_rows($get);

kun je beter schrijven:

$get = mysql_query("SELECT COUNT(1) AS aantal FROM berichten") or die(mysql_error());
$fetch = mysql_fetch_assoc($get);
$aantal = $fetch[aantal];

wanneer je via post iets in de database zet, moet je deze gegevens beveiligen via: "mysql_real_escape_string", zodat sql injection niet meer mogelijk is.

query komt er dan als volgt uit te zien:

mysql_query("
                INSERT INTO 
                    berichten 
                (
                    naam,
                    bericht,
                    ip
                ) 
                VALUES 
                (
                    ".mysql_real_escape_string($_POST[naam]).",
                    ".mysql_real_escape_string($_POST[bericht]).",
                    ".$_SERVER[REMOTE_ADDR]."
                )
            ") or die(mysql_error());


while($info = mysql_fetch_array($getinfo)) {

je kunt beter mysql_fetch_assoc gebruiken dit werkt namelijk sneller

while($info = mysql_fetch_assoc($getinfo)) {

Hoop dat je er iets mee kunt.

Groetjes,
Marijn

Bedankt, en natuurlijk kan ik er iets mee!
0

Jouw reactie

Deel je idee, stel een vraag of help een ander verder.

Editor laden…

Praat mee met de One2xs-community.

Inloggen om te reageren