query error

Overzicht Reageren

Sponsored by: Vacatures door Monsterboard

Viktor

Viktor

26/12/2005 19:14:00
Quote Anchor link
Ik heb dus een klein simpel aanmeld scriptje gemaakt maar ik krijg een query error :S
dit is mijn code van sql en de query. Ik denk tenminste dat hier een fout in moet zitten.



Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
<?
$sql
= "INSERT INTO members (gebruikersnaam, wachtwoord, email, voornaam, achternaam, datum, ip) VALUES ('".$_POST["gebruikersnaam"].", ".$_POST["wachtwoord"].", ".$_POST["email"].", ".$_POST["voornaam"].", ".$_POST["achternaam"].", ".$datum.", ".$_SERVER["REMOTE_ADDR"]."')";
mysql_query($sql) or die ("query error");
?>
 
PHP hulp

PHP hulp

15/02/2025 12:56:17
 
Niels Beckers

Niels Beckers

26/12/2005 19:19:00
Quote Anchor link
als datum NU is kan je beter NOW() gebruiken.
Gewijzigd op 26/12/2005 19:19:00 door Niels Beckers
 
Viktor

Viktor

26/12/2005 19:21:00
Quote Anchor link
De datum is wanneer iemand zich heeft geregistreerd.
 

26/12/2005 19:25:00
Quote Anchor link
mysql_error()
 
Burdy

Burdy

26/12/2005 19:42:00
Quote Anchor link
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
<?php
$sql
= "INSERT INTO members (gebruikersnaam, wachtwoord, email, voornaam, achternaam, datum, ip) VALUES ('".$_POST["gebruikersnaam"]."', '".$_POST["wachtwoord"]."', '".$_POST["email"]."', '".$_POST["voornaam"]."', '".$_POST["achternaam"]."', '$datum', '".$_SERVER["REMOTE_ADDR"]."')";
mysql_query($sql) or die ("query error");
?>
Gewijzigd op 26/12/2005 19:47:00 door Burdy
 
Willem Jan Z

Willem Jan Z

26/12/2005 19:49:00
Quote Anchor link
Denk ook dat dat de fout is, want hij probeert nu alles als int te inserten... En er zitten.... 0 ints bij..
 
Viktor

Viktor

26/12/2005 19:51:00
Quote Anchor link
You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near "".
Nu heb ik deze fout...
 
Burdy

Burdy

26/12/2005 19:58:00
Quote Anchor link
Willem-Jan:
Denk ook dat dat de fout is, want hij probeert nu alles als int te inserten... En er zitten.... 0 ints bij..


Al die quotes, daar is iets mis mee.
Zelf zet ik al die $_POST om in gewone variabelen.
Dan wordt de INSERT-query een stuk eenvoudiger.
 
Willem Jan Z

Willem Jan Z

26/12/2005 19:59:00
Quote Anchor link
Ik doelde ook op je gebruik van quotes :D
 

26/12/2005 20:01:00
Quote Anchor link
Ik gebruik een functie ervoor. Na de functie gebruikt de hebben hoeft ik alleen nog maar INSERT INTO tabel ( {$FIELD['names']} ) VALUES ( {$FIELD['values']} )
 
Willem Jan Z

Willem Jan Z

26/12/2005 20:02:00
Quote Anchor link
Ik heb zoiets ook in een andere script gezien, maar het enige wat je dan doet is je query opdelen... (En je hoeft de query zelf niet meer te schrijven alleen de waardes...)
 
Jan Koehoorn

Jan Koehoorn

26/12/2005 20:16:00
Quote Anchor link
Sowieso _nooit_ meteen invoer vanuit POST variabelen inserten in je db. Altijd zoiets doen:

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
$gebruikersnaam
= htmlspecialchars ($_POST['gebruikersnaam']);
$wachtwoord = htmlspecialchars ($_POST['wachtwoord']);
$email = htmlspecialchars ($_POST['email']);
$voornaam = htmlspecialchars ($_POST['voornaam']);
$achternaam = htmlspecialchars ($_POST['achternaam']);
$ip = $_SERVER['REMOTE_ADDR'];

$sql = "
    INSERT INTO members
    (gebruikersnaam, wachtwoord, email, voornaam, achternaam, datum, ip)
    VALUES
    ('$gebruikersnaam', '$wachtwoord', '$email', '$voornaam', '$achternaam', NOW(), '$ip')"
;
?>
 
Burdy

Burdy

26/12/2005 20:21:00
Quote Anchor link
Goeie tip, kan ik me helemaal in vinden!
 

26/12/2005 20:27:00
Quote Anchor link
En dan de keverneukerij nog even: variabelen buit de quotes houden, of in accolades. :)
 
Viktor

Viktor

26/12/2005 23:39:00
Quote Anchor link
Unknown column 'achternaam' in 'field list'metResource id #1 nou deze... ik heb er naar gekeken maar alles staat er toch echt..
edit: toch een spelfout :S.
Gewijzigd op 26/12/2005 23:41:00 door Viktor
 
Arjan Kapteijn

Arjan Kapteijn

26/12/2005 23:41:00
Quote Anchor link
Denk ook aan hoofdletter gevoeligheid, en 'field list' is wel een hele rare tabelnaam...
 



Overzicht Reageren

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.