wat is hier fout aan?

$sql = "INSERT INTO topics SET id = '', titel = '" . $_POST['tit_el'] . "', categorie = '" . $_GET['categorie'] . "'', bericht = '" . $_POST['beri_cht'] . "', datum = '" . $datum . "', naam = '" . $_POST['na_am'] . "'";
categorie = '" . $_GET['categorie'] . "''

kijk naar die aanhalingstekens op het einde.... klopt niet ;-)

1 ' te veel
hij doet het maar hij wil geen zet dan bij categorie 0?
manaus schreef op 08.08.2006 21:32
wat is hier fout aan?

$sql = "INSERT INTO topics SET id = '', titel = '" . $_POST['tit_el'] . "', categorie = '" . $_GET['categorie'] . "'', bericht = '" . $_POST['beri_cht'] . "', datum = '" . $datum . "', naam = '" . $_POST['na_am'] . "'";
Wat dacht je er van om de boel eens wat overzichtelijker te gaan noteren? Nu zie je door de bomen het bos niet meer.
<?
$sql = "
INSERT INTO
topics
SET
id = '',
titel = '" . $_POST['tit_el'] . "',
categorie = '" . $_GET['categorie'] . "'',
bericht = '" . $_POST['beri_cht'] . "',
datum = '" . $datum . "',
naam = '" . $_POST['na_am'] . "'
";
?>
Verder is $_POST['tit_el'] een redelijk beroerde naam of een tikfout.

En let er op dat je een vreemde variant van SQL gebruikt! Bij een INSERT hoort, bij mijn weten, geen SET. MySQL slikt het wel, maar andere databases volgens mij niet.

Dan nog een vraagje: Heeft $datum wel het formaat yyyy-mm-dd? En heb je de overige variabelen wel gecontroleerd op inhoud, slashes toegevoegd wanneer dat nodig is, etc. etc. ?


En let er op dat je een vreemde variant van SQL gebruikt! Bij een INSERT hoort, bij mijn weten, geen SET. MySQL slikt het wel, maar andere databases volgens mij niet.


dat SET dat wordt gebruikt in een van de tutorials hier

maar het hoort er inderdaad niet bij

ik gebruik ook altijd VALUES
Ik vraag me trouwens af waarom wij niet worden verblijd met de foutmelding die de functie mysql_error() geeft. 99 van de 100x geeft die vrij precies aan waar de fout zit.

Deze keer zal dat zijn op dit stukje:
categorie = '" . $_GET['categorie'] . "'',
Wat Boris dus ook al zei.

Reageren