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!