[php/mysql]waarom is dit fout/goed?

[Wbdsgnr]

Ik ben een scripts aan het maken om downloads te kunnen adden, ben nogal noob met mysql dus eerst had ik was mijn query zo:

PHP:

$query = "INSERT INTO temp (email, naamw, urlw, naamd, urld, type) VALUES ($_POST['email'], $_POST['naamw'], $_POST['urlw'], $_POST['naamd'], $_POST['urld'], $_POST['type'])";

ik gebruik $_POST['...'] omdat ik gelezen had dat je dat nodig had in php 4.2 nou dat werkte dus niet, dus toen heb ik een stukkie gejat van een ander script :

PHP:

$query = "INSERT INTO temp (email, naamw, urlw, naamd, urld, type) VALUES ('". addslashes($email) . "','" . addslashes($naamw) . "','" . addslashes($urlw) . "'," . addslashes($naamd) . "," . addslashes($urld) . ",'" . addslashes($type) ."')";

en dit werkte wel, tenminste ik kreeg geen foutmelding meer.

[whoami]

1. Welke foutmelding kreeg je?
2. Draai je wel PHP 4.2 ?

[ACM]

Wbdsgnr schreef op 05 oktober 2002 @ 16:30:
Ik ben een scripts aan het maken om downloads te kunnen adden, ben nogal noob met mysql dus eerst had ik was mijn query zo:

PHP:

$query = "INSERT INTO temp (email, naamw, urlw, naamd, urld, type) 
VALUES ($_POST['email'], $_POST['naamw'], $_POST['urlw'], $_POST['naamd'], $_POST['urld'], $_POST['type'])";

Dit gaat mis om deze redenen:
http://www.php.net/manual...uage.types.string.parsing

Let er dus op dat je liefst alle vars er als: {$_POST['email']} inzet.
Eventueel kan $_POST[email] ook, maar de {} methode is flexibeler.

[Wbdsgnr]

{} werkt, bedankt.

[SWINX]

ACM schreef op 05 oktober 2002 @ 16:40:

Eventueel kan $_POST[email] ook, maar de {} methode is flexibeler.

foei!

named array's moet je quoten

[Verwijderd]

Gebruik je magic Quotes?

[ACM]

SWINX schreef op 05 oktober 2002 @ 17:01:
foei!

named array's moet je quoten

Hehehe, guess again!

"blaat $array[veld] bladiebla"
is dat het zelfde als:
"blaat $array['veld'] bladiebla"

?

Magoed, vandaar ook mijn voorkeur voor {}

[drm]

Als we nou gewoon allemaal { } gebruiken, kan je je named arrays quoten wat je wilt

[Wbdsgnr]

hehe Ok nu snap ik er nog minder van ik heb nu dit:

PHP:

mysql_select_db($dbnaam,$con);
    $query = "INSERT INTO temp (email, naamw, urlw, naamd, urld, type) VALUES ({$_POST['email']}, {$_POST['naamw']}, {$_POST['urlw']}, {$_POST['naamd']}, {$_POST['urld']}, {$_POST['type']})";
    mysql_query($query, $con);
    mysql_close($con);  
    print "Succeeded, Your download will be added after checking it";

ik heb alles paar keer gecheckt en geen typefouten gemaakt, maar er verschijnt gewoon helemaal niks in die database, al print ik $_POST[...] dan verschijnt wel de inhoud dus de form is ook goed

[DeTeraarist]

Mag dit nog, of is dat al weer ouderwets?

PHP:

$sql = 'blaat' . $blaat['blaat'] . 'blaat nog wat';

[ACM]

Wbdsgnr schreef op 05 oktober 2002 @ 18:03:
ik heb alles paar keer gecheckt en geen typefouten gemaakt, maar er verschijnt gewoon helemaal niks in die database, al print ik $_POST[...] dan verschijnt wel de inhoud dus de form is ook goed

Je moet ook nog de query goed maken
Zie vooral ook 'echo $query;' enzo

Die moet eruit zien als:
insert into ... values ('string', 'string')

etc getallen hoeven niet perse ' omheen.

DeTeraarist schreef op 05 oktober 2002 @ 18:07:
Mag dit nog, of is dat al weer ouderwets?

Tuurlijk mag dat
Maar daar ging het niet om

[Wbdsgnr]

Edit:

ah stomme fout was de ' ' vergeten thx voor je hulp.

[drm]

offtopic:
wt* staat jouw nickname voor Wat dat betreft snugger kon ik er van maken, maar dan kloppen er 2 letters niet...

[Ericston]

offtopic:
Ik denk eerder dat het een woord is waar de klinkers tussenuit zijn gehaald. In tegenstelling tot jouw nickname die gewoon een acroniem voor "dit ruikt muf" is.

[Verwijderd]

offtopic:
ik gok op webdesigner

[Verwijderd]

offtopic:
[quote]Verwijderd schreef op 06 oktober 2002 @ 18:06:
[ot]ik gok op webdesigner[/ot][/quote]
U gaat door voor de wasmachine

[Verwijderd]

Ik bewerk (addslashes en die zut) mijn $_POST-vars eerst voordat ze de dbase ingaan, waardoor je die probleem eigenlijk niet hebt.

Je kunt ook eerst een $sql-string maken, en dan die $sql uitvoeren

[drm]

offtopic:

• ______________________________________________________ [b][message=15461015,noline]CowMike[/message]:[/b][list]______________________________________________________ [b][message=15458620,noline]RobinVR[/message]:[/b] ik gok op webdesigner ______________________________________________________[/list] U gaat door voor de wasmachine :7 _______________________________________________________

agreed