fejl i php.

Problemet er navnet by

Den fortolker det som sql i stedet for feltnavn.

Du skal sætte det ind sådanne: `by`='$_GET.......

Læg mærke til ` og ikke '

Og det var et svar

Altså den fortolker det ligesom dette: order BY
i stedet for feltnavn
En kommando

by er et såkaldt reserveret navn i SQL brug evt. et prefix/postfix til at ordne det:

eks. DB_by

///McCookie

Det er sandt hvad amews_aj siger at BY er et reseveret navn og må derfor ikke bruges som de andre. Det skal støttes af nogle ` rundt om sig. Dog er det også vigtigt lige at nævne at den query d har opbygget dér, er utrolig usikker og kan nemt misbruges. Du bør aldrig benytte en GET direkte i en query uden at den er gennemtjekket. Brug mysql_real_escape_string til at fixe det problem! :-)

jeg må skuffe, og sige at det ikke hjalp ?

simplus: kan du lave en 'sikker' til mig så? :)

$hent = mysql_query("SELECT * FROM vaelger `by`='".mysql_real_escape_string($_GET[by])."' ORDER BY firma DESC") or die("Fejl: " . mysql_error());

Fejl: 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 '='roenne' ORDER BY firma DESC' at line 1

stadig den samme fejl ? :s

Ved ikke hvad der er galt i den, men uanset hvad, er det ikke samme fejl:

Før
near 'by='roenne'

Nu
near '='roenne'

nej, det kan jeg godt se. åh!

Nej det er klart du mangler en WHERE

<?php
$city = mysql_real_escape_string($_GET[by]);

$hent = mysql_query("SELECT * FROM vaelger WHERE `by` = '$city' ORDER BY firma DESC") or die("Fejl: " . mysql_error());
?>

selvfølgelig, den overså jeg lige - heh

man kan nemlig ikke bare skrive

`by` = 'værdi'

man skal istedet gøre brug af WHERE

WHERE `by` = 'værdi'

sådan! - så skete der noget.

hvis jeg lige må spørge om en sidste ting, simplus? - nu hedder min side visalle.php?by=roenne, hvis jeg nu vil kalde den visalle.php så får jeg den fejl igen, burde den ikke vise alle?

simplus et svar ;)

nej, men du kan vidst nok snyde PHP og skrive visalle.php?by=' også vil den vise alle, men det er slet ikke nødvendigt at gøre sådan. Du kan da bare lave en værdi der hedder visalle.php?by=alle og skrive:

if ($_GET[by] == "alle") {

$hent = mysql_query("SELECT * FROM vaelger ORDER BY firma DESC") or die("Fejl: " . mysql_error());

}

nå ja, og bare behold pointene :-)

okay, det vil jeg prøve.

mange tak! :)

jeg lukker!

plus til simplus!

Det var så lidt.