Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP] Pętla, Panel
Forum PHP.pl > Forum > Przedszkole
Wieczny
da się to szybciej napisać ?
  1. <?php
  2. echo "<link rel='Stylesheet' type='text/css' href='style.css' />";
  3.  echo "
  4.  <div id='leftsite'>";
  5.  
  6. $result = mysql_query("SELECT id, value, name, align, valign, user
  7. FROM wpp_panels WHERE align='leftsite' ORDER by id ASC");
  8.  
  9. while ($row = mysql_fetch_array($result))
  10.  {
  11.  
  12. echo"<table border='1' class='panel'>
  13. <tr>
  14. <td background='bgpanel.jpg'><b>".$row['name']."</b></td>
  15. </tr>
  16. <tr>
  17.  
  18.  <td>";
  19.  
  20. include($row['value']);
  21.  
  22.  echo"</td>
  23. <tr>
  24.  
  25. </table>";
  26.  
  27.  }
  28.  
  29.  
  30. echo "</div>";
  31.  
  32. ?>
Pilsener
Oczywiście:
- atrybuty w kodzie HTML piszemy w cudzysłowach, a nie apostrofach
- wywal te echa, jedno echo na stronę chyba styknie, nie?
- pobieraj z bazy tylko te pola, których potrzebujesz
- wywal zbędne atrybuty i tagi html

- zapomniałem jeszcze dopisać, że stringi piszemy w apostrofach, a nie cudzysłowiach - tak jest szybciej
pyro
Cytat(Pilsener @ 10.06.2008, 09:06:56 ) *
Oczywiście:
- atrybuty w kodzie HTML piszemy w cudzysłowach, a nie apostrofach


Co to za różnica?
Cytat
- pobieraj z bazy tylko te pola, których potrzebujesz


Cytat
$result = mysql_query("SELECT id, value, name, align, valign, user
FROM wpp_panels WHERE align='leftsite' ORDER by id ASC");


Myślę, że Wieczny właśnie tak robi smile.gif
Wieczny
Jedno echo na całą stronę .... Ciekawe jak ;D
sniezny_wilk
Cytat(Wieczny @ 10.06.2008, 07:29:32 ) *
Jedno echo na całą stronę .... Ciekawe jak ;D


@Pilsner troszkę przesadził, generalnie chodziło mu pewnie o to:

  1. <?php
  2. echo "<link rel='Stylesheet' type='text/css' href='style.css' />";
  3.  echo "
  4.  <div id='leftsite'>";
  5. ?>


a lepszym i szybszym rozwiązaniem będzie:

  1. <?php
  2. echo '<link rel="Stylesheet" type="text/css" href="style.css" /><div id="leftsite">';
  3. ?>


A to trochę mało bezpieczne:
  1. <?php
  2. include($row['value']);
  3. ?>


ewentualnie można przymknąć na to oko, jeśli tej wartości user jej nie wypełnia. Ja również użyłbym require_once, zamiast include.
JoShiMa
Cytat(Wieczny @ 10.06.2008, 07:29:32 ) *
Jedno echo na całą stronę .... Ciekawe jak ;D

Normalnie. Sklejasz wszystko do stringu a na koniec robisz echo. Z powodzeniem piszę w ten sposób nawet całkiem rozbudowane stronki.
Wieczny
Cytat(sniezny_wilk @ 10.06.2008, 09:40:04 ) *
A to trochę mało bezpieczne:
  1. <?php
  2. include($row['value']);
  3. ?>


ewentualnie można przymknąć na to oko, jeśli tej wartości user jej nie wypełnia. Ja również użyłbym require_once, zamiast include.


Tak wypełnia to pole administrator z panelu smile.gif


Cytat(JoShiMa @ 10.06.2008, 09:43:51 ) *
Normalnie. Sklejasz wszystko do stringu a na koniec robisz echo. Z powodzeniem piszę w ten sposób nawet całkiem rozbudowane stronki.


Możesz mi ktoś zaprezentować prosty przykładowy kod jak by to miało wyglądać ?
phpion
Cytat(Wieczny @ 10.06.2008, 10:06:45 ) *
Możesz mi ktoś zaprezentować prosty przykładowy kod jak by to miało wyglądać ?

  1. <?php
  2. $echo = '';
  3. $echo .= '<html><head></head><body>';
  4. $echo .= '<p>lalalalal</p>';
  5. $echo .= '</body></html>';
  6.  
  7. echo $echo;
  8. ?>

aczkolwiek ja bym się zastanowił nad szablonami... niekoniecznie SMARTY, zwykłe "szablony" w PHP są wg mnie nawet lepsze.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.