En PHP Kode fejl :S

Tags:    php

Hejsa jeg er igang med dette forum her http://www.rngrafik.wep.dk/forums
Men som i måske har set alleredet kan man ikke lave en ny tråd :S..

Kilde kode :

Fold kodeboks ind/udKode 


Til rigtige svar er der 200 UP



5 svar postet i denne tråd vises herunder
4 indlæg har modtaget i alt 12 karma
Sorter efter stemmer Sorter efter dato
Har lidt svært ved at se hvordan din database er bygget op, men er meget sikker på at fejlen er på linie 32:

mysql_query("INSERT INTO `post` ( `id` , `post` ) VALUES (', '$post')") or die(mysql_error());

Du angiver ikke hvilken værdi der skal tilskrives feltet `id` i tabellen. Prøv at ret det til:

mysql_query("INSERT INTO `post` ( `id` , `post` ) VALUES ('$id', '$post')") or die(mysql_error());



VALUES(',

skal være

VALUES('',

alle de steder du har skrevet det og det er mange.


I øvrigt er der noget der hedder indrykning hvor man rykker koden længere og længere ind for hver block, dette gør ens kode en del lettere at læse.



%



Indlæg senest redigeret d. 10.08.2011 17:08 af Bruger #13771
Hvis id er sat til auto_increment i SQL basen, er feltet unødvendigt at inkludere i insert statements.

Desuden er en ting som dette
Fold kodeboks ind/udKode 

meget uheldigt, idet alle og enhver således kan injecte SQL kode direkte ind i din database.

Desuden, hvis det stykke kode overhovedet virker, så bruger du register globals, hvilket er en meget, meget dårlig ide, idet du får mere besvær end gavn ved at bruge dem, og præcis som i dette tilfælde, er der mulighed for at injecte kode ind i et script.

Du skal aldrig stole på input fra en bruger.



Skal case i switches ikke også afsluttes med :?

Altså:
case "thread":
istedet for:
case "thread";

Eller er det mig der er galt på den?



t