Witam!

Mam taki skrypt wyświetlający wiadomości z bazy danych:

[PHP] pobierz, plaintext 
<?php
include("connect.php");
$start = $_GET['start'];
$na_stronie = 10;
if (empty($_GET['start']))
{
$start=0;
} 
 
$query = "(SELECT tytul,autor,data,tresc,zdjecie AS wiadomosci FROM wiadomosci) UNION (SELECT tytul,autor,data,tresc,zdjecie AS artykuly FROM artykuly) ORDER BY data DESC LIMIT ".($start).",".$na_stronie."";
$result = mysql_query ($query);
while ($row = mysql_fetch_array($result))
        {
$body .= '
<li>
<a href="look.php?id='.$row[0].'"><h1>'.$row[0].' </h1></a><div class="main_image"><img class="images" src="photos/'.$row[4].'" ></div></br>'.$row[2].' , '.$row[1].'</br><p>'.$row[3].'</p>
 </li>';
} echo '<ul>'.$body.'</ul>';
 
.
.
.
 
?>
[PHP] pobierz, plaintext 

Chciałbym go zmodyfikować w taki sposób, żeby w momencie gdy dodaje wiadomość przez formularz i pomijam opcje dodania zdjęcia, żeby wyświetlało się wszystko oprócz tego elementu

[PHP] pobierz, plaintext 
 <div class="main_image"><img class="images" src="photos/'.$row[4].'" ></div>
[PHP] pobierz, plaintext 

myślałem o czymś takim, ale nie wiem czy jest to dobry kod:

[PHP] pobierz, plaintext 
if ($row[4] == '')
{$body .= '
<li>
<a href="look.php?id='.$row[0].'"><h1>'.$row[0].' </h1></a></br>'.$row[2].' , '.$row[1].'</br><p>'.$row[3].'</p>
 </li>';
}
else 
{$body .= '
<li>
<a href="look.php?id='.$row[0].'"><h1>'.$row[0].' </h1></a><div class="main_image"><img class="images" src="photos/'.$row[4].'" ></div></br>'.$row[2].' , '.$row[1].'</br><p>'.$row[3].'</p>
 </li>';} echo '<ul>'.$body.'</ul>';
[PHP] pobierz, plaintext 

Warunek jest dobry, choć można jeszcze bardziej przystosować kod do reguły DRY:

[PHP] pobierz, plaintext 
$imgHTML = empty($row[4]) ? '' : '<div class="main_image"><img class="images" src="photos/'.$row[4].'" ></div>';
$body .= '
<li>
<a href="look.php?id='.$row[0].'"><h1>'.$row[0].' </h1></a>'.$imgHTML.'</br>'.$row[2].' , '.$row[1].'</br><p>'.$row[3].'</p>
</li>';
[PHP] pobierz, plaintext 

Reszty kodu nie oceniam, ale dodam, że w HTML nie ma znacznika </br>, a jest <br />.