Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [MySQL][PHP]Nie wyświetlanie pustego pola
kosior11
post 7.03.2012, 12:49:29
Post #1





Grupa: Zarejestrowani
Postów: 101
Pomógł: 0
Dołączył: 20.03.2011

Ostrzeżenie: (10%)
X----


Witam!

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

  1. <?php
  2. include("connect.php");
  3. $start = $_GET['start'];
  4. $na_stronie = 10;
  5. if (empty($_GET['start']))
  6. {
  7. $start=0;
  8. }
  9.  
  10. $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."";
  11. $result = mysql_query ($query);
  12. while ($row = mysql_fetch_array($result))
  13. {
  14. $body .= '
  15. <li>
  16. <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>
  17. </li>';
  18. } echo '<ul>'.$body.'</ul>';
  19.  
  20. .
  21. .
  22. .
  23.  
  24. ?>


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
  1. <div class="main_image"><img class="images" src="photos/'.$row[4].'" ></div>


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

  1. if ($row[4] == '')
  2. {$body .= '
  3. <li>
  4. <a href="look.php?id='.$row[0].'"><h1>'.$row[0].' </h1></a></br>'.$row[2].' , '.$row[1].'</br><p>'.$row[3].'</p>
  5. </li>';
  6. }
  7. else
  8. {$body .= '
  9. <li>
  10. <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>
  11. </li>';} echo '<ul>'.$body.'</ul>';
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi (1 - 1)
mortus
post 7.03.2012, 12:58:01
Post #2





Grupa: Zarejestrowani
Postów: 2 178
Pomógł: 596
Dołączył: 25.09.2009
Skąd: Piwniczna-Zdrój

Ostrzeżenie: (0%)
-----


Warunek jest dobry, choć można jeszcze bardziej przystosować kod do reguły DRY:
  1. $imgHTML = empty($row[4]) ? '' : '<div class="main_image"><img class="images" src="photos/'.$row[4].'" ></div>';
  2. $body .= '
  3. <li>
  4. <a href="look.php?id='.$row[0].'"><h1>'.$row[0].' </h1></a>'.$imgHTML.'</br>'.$row[2].' , '.$row[1].'</br><p>'.$row[3].'</p>
  5. </li>';

Reszty kodu nie oceniam, ale dodam, że w HTML nie ma znacznika </br>, a jest <br />.
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Wersja Lo-Fi Aktualny czas: 14.08.2025 - 06:20