Hoi,

Ik heb de onderstaande pagina gemaakt, ik ben nog niet zolang bezig met php en wil daarom vragen of iemand er even overheen wilt kijken en kijken of ik het zo een beetje goed heb gedaan.

Met vriendelijke groet,
Robin Peters

------------
<?php
include('siteconfig/config.php');

//Echo main table
echo'<table width="100%">';
echo'<td width="30%"><div align="left">Name</div></td>';
echo'<td width="30%"><div align="left">Game</div></td>';
echo'<td width="20%"><div align="left">Size</div></td>';
echo'<td width="20%"><div align="left">Add Date</div></td>';
echo'</table><br>';

//Get download from database
$sql = "SELECT * FROM downloads ORDER BY id DESC";
$res = mysql_query($sql) or die (mysql_error());
while($list = mysql_fetch_object($res)) {

//Echo information that granded from database
echo'<table width="100%">';
echo'<td width="30%"><div align="left">'.$list->name.'</div></td>';
echo'<td width="30%"><div align="left">'.$list->game.'</div></td>';
echo'<td width="20%"><div align="left">'.$list->size.'</div></td>';
echo'<td width="20%"><div align="left">'.$list->adddate.'</div></td>';
echo'</table>';}

?>
de echo() tussen dubble quotes zetten

echo "<td width="30%"><div align='left'>'.$list->name.'</div></td>";

tussen de dubbele qoutes van echo alleen maar enkele gebriuken.

En het kan overzichterlijker....
Kalle schreef op 30.12.2005 02:00
de echo() tussen dubble quotes zetten


Pardon? Waarom dubbele quotes?
Hij sluit ze weer af door de ' van de '.list->name.' . En dan echo-ed ie niet de .list->date.

Of zijn die punten daarvoor dan...
$list->date kom ik in dat hele script niet tegen
@jan

dit is toch een goed script of niet??


ps: Probeer ook wat te lere:P:P
@ Thijs: dankjewel voor je advies!
Ik zal wel even verduidelijken wat ik bedoelde (ook al is misschien fout)

Dit stond er:
<?
echo'<td width="30%"><div align="left">'.$list->name.'</div></td>';
?>
(Dit is btw gewoon 1 deel ervan, hiermee bedoel ik ook de rest tussen die tussen die while staat)

Maar door het gebruik van enkele na "left"> :

<?
echo'<td width="30%"><div align="left">'
?>
wordt de rest niet getoond.

Daarom dacht ik:
<?
echo "<td width='30%'><div align='left'>'.$list->name.'</div></td>";
?>
maar het had net zogoed:

<?
echo'<td width="30%"><div align="left">".$list->name."</div></td>';
?>

kunnen zijn.

Dit is teminste wat ik dacht dat er misschien fout was... ik hoop dat het nu helder is wat ik bedoelde (ook al was het dus fout)
Het was inderdaad dus fout. De aanpak die Robin had gekozen is momenteel al de meest nette aanpak. Voordeel is namelijk dat je nog gewoon " in je tags kan gebruiken zonder ze te hoeven escapen en je je variabelen gedwongen buiten quotes moet halen. En zoals je zelf al ziet in de highlighted code, zie je ze dan veel sneller terug.
En aangezien dubbele quotes zeer aangeraden zijn :D

Reageren