[php/mysql] query met variabelen samenstellen

[Verwijderd]

Ik gebruik dit stukje code om een sql-query samen te stellen:

PHP:

$sql = 'insert into ' . $table . ' (' . implode(', ', $TABLES[$table]) . ') 
values ("$' . implode('", "$', $TABLES[$table]) . '")';

Het resultaat van deze actie is:

code:

insert into pages (page_id, menu_id, volgnummer, titel, tekst) 
values ("$page_id", "$menu_id", "$volgnummer", "$titel", "$tekst")

(linebreak toegevoegd voor betere leesbaarheid)

De query wordt goed uitgevoerd, alleen worden de goede waarden uit de variabelen $titel, $tekst, etc. niet ingevoerd, maar de letterlijke tekst '$tekst'.

Hoe krijg ik MySQL nu aan het verstand dat-ie de variabelen moet gebruiken i.p.v. een dollarteken en het woordje 'tekst'?

[D2k]

'$var' != "$var"

[Cartman!]

velen zullen t ranzig vinden denk ik, maar :

PHP:

$query = "INSERT INTO tabel (waarde1, waarde2, waarde3) VALUES ('$waarde1', '$waarde2', '$waarde3')";

werkt perfect....

[Verwijderd]

Ja, bij mij normaal ook, maar niet in deze functie. (En ook met enkele quotes gaat het niet goed, FYI).

[Cartman!]

waarom voer je dan niet eerst die implodes uit, dus voor dat je de query maakt? stuk overzichtelijker vind ik.

[Edsger]

volgens mij probeert Arch code te generen VOOR php... dus in php, php code genereren, nee dat zal niet werken...

edit:
Om precieser te zijn, hij wil de namen van variabelen genereren met dat php commando, en dat de semantiek daarvan weer geinterpreteerd wordt door php. Ofwel dat de waardes aan de variabelen namen worden toegekend op het moment dat hij daarvan een string genereert. Dat gaat (uiteraard) niet.

[ Voor 58% gewijzigd door Edsger op 26-02-2004 15:28 ]

[Verwijderd]

@ g00fy:
Dit werkt ook niet:

PHP:

$rows = implode(', ', $TABLES[$table]);
$values = implode('\', \'$', $TABLES[$table]);
$sql = 'insert into ' . $table . ' (' . $rows . ') values (\'$' . $values . '\')';

@ Edsger: Eigenlijk is het code voor MySQL, lijkt mij. Maar hoe zou ik het dan wel kunnen doen?

[MatHack]

Om dit te laten werken moet je eval() gebruiken. eval() wordt overigens wel gezien als een ontzettend ranzige oplossing.

[Cartman!]

ArachCarrier, maak er anders dit van :

PHP:

$sql = "INSERT INTO $table ($rows) values ('$values')";

lijkt me sterk dat dat niet werkt


edit : " vergeten

[ Voor 14% gewijzigd door Cartman! op 26-02-2004 15:41 ]

[T-MOB]

Nee, Edsger heeft gelijk...

Iets als hieronder zou wel moeten werken:

PHP:

$columns = implode(',', $TABLES[$table]);

reset ($TABLE[$table]);
while (list($key, $value) = each($TABLE[$table])) 
{
$values[] = ${$value};
}

$values = "'" .implode("','" ,$values) ."'";

mysql_query("INSERT INTO $table ($columns) ($values)");

typevouten voorbehouden

[Verwijderd]

@ MatHack: Waarom is eval() ranzig? Het lijkt erop dat dit precies is wat ik nodig heb...

@ T-MOB: Ik ken deze constructie niet: $($value). Geef je daarmee aan dat je een variabele maakt van de string die is opgeslagen in $value? In dat geval zou dit ook erg handig kunnen zijn.

Ik ga er meteen mee bezig! (Eerste poging leek veelbelovend, alleen bevat $values slechts lege strings).

[T-MOB]

let op: niet met haakjes maar "{ }" (accolades heten ze geloof ik)!!

En ja, op die manier kun je variabele variabelen gebruiken. Is soms wel eens makkelijk, je code wordt er alleen niet altijd even overzichtelijk van.

[ Voor 9% gewijzigd door T-MOB op 26-02-2004 15:58 . Reden: duidelijkheid ]

[Bosmonster]

T-MOB schreef op 26 februari 2004 @ 15:54:
let op: niet met haakjes maar "{ }" (accolades heten ze geloof ik)!!

En ja, op die manier kun je variabele variabelen gebruiken. Is soms wel eens makkelijk, je code wordt er alleen niet altijd even overzichtelijk van.

Soms wel eens 'makkelijk', maar NOOIT nodig.

[ Voor 9% gewijzigd door Bosmonster op 26-02-2004 16:01 ]

[Edsger]

Verwijderd schreef op 26 februari 2004 @ 15:29:
@ Edsger: Eigenlijk is het code voor MySQL, lijkt mij. Maar hoe zou ik het dan wel kunnen doen?

Met een loopje dat array doorlopen en de string daarmee opbouwen.
Tip: gebruik dan wel de syntax:
$sql = "INSERT INTO $table SET col_name1 = '$value1', col_name2 = '$value2'";

die bouw je makkelijker op.

[MatHack]

Verwijderd schreef op 26 februari 2004 @ 15:50:
@ MatHack: Waarom is eval() ranzig? Het lijkt erop dat dit precies is wat ik nodig heb...

Zoek eens op eval & ranzig hier op GoT, kun je zien waarom sommigen vinden dat eval() ranzig is.

Bosmonster schreef op 26 februari 2004 @ 16:01:
[...]


Soms wel eens 'makkelijk', maar NOOIT nodig.

Soms zijn { nodig in PHP (met variabelen) hoor, zie http://www.php.net/manual/en/language.variables.variable.php laatste alinea

[ Voor 35% gewijzigd door MatHack op 26-02-2004 16:15 ]

[T-MOB]

Soms wel eens 'makkelijk', maar NOOIT nodig.

Ben ik met je eens, de TS heeft in dit geval echter zijn column_names opgeslagen in een array $TABLE[table][] en de waarden die ingevoerd moeten worden in de tabel opgeslagen in variabelen $"column_name".

Dan is de variabele variabele een snelle oplossing... Anders moet ie zijn hele script aanpassen. Ik neem aan dat hij niet aan het scripten is voor een schoonheidsprijs, soms moeten dingen gewoon "werken".

[oscarvdb]

g00fy schreef op 26 februari 2004 @ 15:21:
velen zullen t ranzig vinden denk ik, maar :

PHP:

$query = "INSERT INTO tabel (waarde1, waarde2, waarde3) VALUES ('$waarde1', '$waarde2', '$waarde3')";

werkt perfect....

waarom niet dit:

PHP:

$query = "INSERT INTO tabel (waarde1, waarde2, waarde3) VALUES ('".$waarde1."', '".$waarde2."', '".$waarde3."')";

[Verwijderd]

Ik neem aan dat hij niet aan het scripten is voor een schoonheidsprijs, soms moeten dingen gewoon "werken".

Mooi gezegd! Ik ben al een halve dag aan het klooien met deze functies, en ik wil over anderhalf uur eten

[T-MOB]

'k zij nog zo

typevouten voorbehouden

in mijn stukkie code zat nl. typefoutje

PHP:

<?
reset ($TABLE[$table]);
while (list($key, $value) = each($TABLE[$table]))
?> 

Moet naturlijk zijn:

PHP:

reset ($TABLES[$table]);        // let op die 'S'
while (list($key, $value) = each($TABLES[$table]))       // let op die 'S'

Hoop dat je toch nog op tijd kunt eten