[php]parse error

[Paultje3181]

Ik heb zelf een stukje code geproduceerd om een hyperlink uit een mysql database te produceren. Ik krijg echter een parse-error bij de regel waar de <a href geproduceerd wordt. Ik gok dat ergens een " verkeerd staat, maar ik zie het niet. Kan iemand mij wellicht opheldering geven?

PHP:

<?php
include("../../../config.php");
$var1=$_GET['var1'];
if($var1==""){$var1="type"; }
$var2=$_GET['var2'];
if($var2==""){$var2="hyperlink"; }
$var3=$_GET['var3'];
if($var3==""){$var3="naam"; }

$result = mysql_query("SELECT * FROM linkspaul ORDER BY ".$var1." ASC);
?>

<?
while($row = mysql_fetch_array($result))
{
echo "<a href="" . $row['hyperlink'] . ""> . $row['naam'] . "</a><br />"";
}
mysql_close($connect);
?>

De fout zit dus in de echo. De tabel die ik heb heeft 3 rijen: naam, hyperlink en type. Type wordt op gesorteerd, maar komt nu nog niet terug. Komt evt. later nog.
Volgens mij klopt het zo, maar blijkbaar toch niet

[Snake]

PHP:

echo "<a href=\"" . $row['hyperlink'] . "\">" . $row['naam'] . "</a><br />";

Maar als iemand een ' in $_GET['var1'] steekt, dan gaat je query al over kop.

[ Voor 32% gewijzigd door Snake op 04-09-2007 18:59 ]

[Palomar]

of echoën met ' ' ipv. " ".

[Duroth]

Om maar even door te gaan op Snake's (juiste) antwoord:

Je bent dus, in de code, vergeten de 'HTML-aanhalingstekens' te escapen. Omdat PHP zonder voorafgaande \ een dubbel aanhalingsteken als code zal interpreteren, en niet als output (HTML), klopt er opeens niks meer van de regel. Gewoon op de juiste plekken een backslash erin zetten (zie Snake's post) en alles werkt.

[ibmos2warp]

Laten we leuk wat appelstrofs platen, maakt toch niet uit dat de code in de war gaat?

Escape en / of Addslashes de zooi.

edit:
spuit elf.

[ Voor 26% gewijzigd door ibmos2warp op 04-09-2007 19:09 ]

[Johnny]

Zoals hierboven al gezegd wordt moet je enkele aanhalingstekens gebruiken of een backslash ervoor zetten, anders denkt de parser dat het einde bereikt is waarna een punt of een punt-komma geldige tekens zijn, nog een aanhalingsteken niet.

Let wel op wanneer je escape characters zoals \n \r \t \\ binnen enkele aanhalingstekens zet worden ze letterlijk genomen, '\\' geeft \\ terug en "\\" geeft \ terug.

[Psychokiller]

Behalve dat dit wel vrij basic is, heb je ook een " vergeten in je query zelf, na de ASC.
Alleen al door de parser kun je zien dat dingen niet goed gekleurd zijn en dat de HTML en PHP code door elkaar lopen.

[robbert]

En zullen we de sql-injections ook niet vergeten... Ongecontroleerd een variabelen in een query proppen is natuurlijk not done.

Daarnaast gaat de query ook al mis als je uberhaupt een bier bestaande kolom naam in die $var1 krijgt.

[ Voor 71% gewijzigd door robbert op 04-09-2007 19:34 ]

[RobIII]

Dit is toch echt een gevalletje code debuggen en gewoon even de quotes bij elkaar matchen. Je ziet hier de highlighter al de fout aangeven, maar een beetje IDE doet dat ook voor je. Zie ook Debuggen: Hoe doe ik dat?. Omdat dit vrij basic is en ik verder geen discussiewaarde hierin zie (en alleen nog maar meer van bovenstaande reacties verwacht die verder, na deze reacties weinig zullen toevoegen) gaat dit topic dan ook op slot.