Poprawna metoda pisania Opcje

[slawo]

Witam, mam takie pytanie. Która wersja jest poprawna. (obie działają)

[PHP] pobierz, plaintext 
<?php
 
$ostatnio_pobrane = mysql_query("SELECT * FROM albumy_pobrania ORDER BY id DESC LIMIT 5")
or die(''); 
 
    while($op = mysql_fetch_assoc($ostatnio_pobrane)) {
 
		$album_op = mysql_query("SELECT * FROM albumy WHERE id='".$op['album']."'")
or die(''); 
 
    while($aop = mysql_fetch_assoc($album_op)) {
?>
<div id="tresc">
 
<a href="album/<?php echo "".$aop['url']."_".$aop['rok']."_".$aop['wydanie']."_".$aop['id']."";?>.html">
<img width="35" height="35" src="okladki/<?php echo "".$aop['okladka']."";?>" align="left" vspace="2" border="0"></a>
<b><?php echo "".$aop['grupa']."";?></b> - <a href="album/<?php echo "".$aop['url']."_".$aop['rok']."_".$aop['wydanie']."_".$aop['id']."";?>.html"><?php echo "".$aop['tytul']."";?></a>
<span class="details2"><br> pobrał: <a href="<?php echo "pokaz_profil.php?user=".urlencode($op['login'])."";?>"><b><?php echo "".$op['login']."";?></b></a> data: <b><?php echo "".$op['data']."";?></b></a></span>
</div>
<?php
	}
	}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$ostatnio_pobrane = mysql_query("SELECT * FROM albumy_pobrania ORDER BY id DESC LIMIT 5")
or die(''); 
 
    while($op = mysql_fetch_assoc($ostatnio_pobrane)) {
 
		$album_op = mysql_query("SELECT * FROM albumy WHERE id='".$op['album']."'")
or die(''); 
 
    while($aop = mysql_fetch_assoc($album_op)) {
 
 
 
				echo "<table border=2 width=100% style='border-collapse: collapse' bgcolor=#FAF8F8 bordercolor=#FFFFFF>
	<tr>
		<td>";
 
		echo "<a href=album/".$aop['url']."_".$aop['rok']."_".$aop['wydanie']."_".$aop['id'].".html><img width=35 height=35 src="."okladki/".$aop['okladka']." align=left vspace=2 border=0></a>";
        echo "<b>&nbsp;<font size=2 face=arial>".$aop['grupa']."</font></b> - ";
        echo "&nbsp;<a href=album/".$aop['url']."_".$aop['rok']."_".$aop['wydanie']."_".$aop['id'].".html style='text-decoration: none'><font size=2 face=arial color=black>".$aop['tytul']."</font></a>";
        echo "&nbsp;";
		echo "<br>&nbsp;<font size=1 face=verdana color=grey>pobrał:</font> <a href=pokaz_profil.php?user=".urlencode($op['login'])." style='text-decoration: none'><font size=1 face=verdana color=grey><b>".$op['login']."</b></font></a> <font size=1 face=verdana color=grey>data:</font> <font size=1 face=verdana color=grey><b>".$op['data']."</b></font></a>";
 
 
			echo "</td>
	</tr>
</table>";
 
	}
	}
[PHP] pobierz, plaintext 

[Hpsi]

W każdej cześci po zapytaniu or die(''); jest złym nawykiem bo nie widzisz błedu mysql
Powiem Ci , że wg mnie wygoniejsza jest druga wersja - ja bym podobnie napisał, tylko troszkę bezensu w każdej lini robić echo , skoro można to zrobić wszystko w jednym. Pozatym zamiast cudzysłowi powinno się używać apostrofów

Ten post edytował Hpsi 29.06.2011, 10:11:10

[tehaha]

obie wersje są tragiczne:
1. nigdy nie rób zapytania w pętli, to jest jeden z najgorszych możliwych błędów -> numery albumów z pierwszego zapytania zapisz do tablicy a w drugim zapytaniu pobierz od razu wszystkie używając konstrukcji WHERE id IN(1,2,3)
2. jeżeli Twoje pytanie dotyczyło tego czy parametry lepiej dawać w echo czy <?php echo ?> to wybieraj to co jest w danym przypadku bardziej czytelne i wygodne, akurat tutaj będzie to druga metoda
3. jak robisz echo z cudzysłowem to nie musisz z niego wychodzić, żeby wstawić zmienną możesz użyć nawiasu klamrowego:

[PHP] pobierz, plaintext 
echo "<a href='{$aop['url']}' title='{$aop['url']}'"
[PHP] pobierz, plaintext 

i możesz też to wszystko wrzucić w jedno echo, po co robić 6 jedno pod drugim

Ten post edytował tehaha 29.06.2011, 10:20:29