Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [PHP] Pętla, Panel
Wieczny
post 9.06.2008, 22:35:11
Post #1





Grupa: Zarejestrowani
Postów: 191
Pomógł: 1
Dołączył: 21.03.2007

Ostrzeżenie: (60%)
XXX--

da się to szybciej napisać ?
[PHP] pobierz, plaintext 
  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. ?>
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
Pilsener
post 10.06.2008, 08:06:56
Post #2





Grupa: Zarejestrowani
Postów: 1 590
Pomógł: 185
Dołączył: 19.04.2006
Skąd: Gdańsk

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

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

Ten post edytował Pilsener 10.06.2008, 08:09:48
Go to the top of the page
+Quote Post
pyro
post 10.06.2008, 08:10:43
Post #3





Grupa: Zarejestrowani
Postów: 2 148
Pomógł: 230
Dołączył: 26.03.2008

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

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


--------------------
ET LINGUA EIUS LOQUETUR IUDICIUM
Go to the top of the page
+Quote Post
Wieczny
post 10.06.2008, 08:29:32
Post #4





Grupa: Zarejestrowani
Postów: 191
Pomógł: 1
Dołączył: 21.03.2007

Ostrzeżenie: (60%)
XXX--

Jedno echo na całą stronę .... Ciekawe jak ;D
Go to the top of the page
+Quote Post
sniezny_wilk
post 10.06.2008, 08:40:04
Post #5





Grupa: Zarejestrowani
Postów: 732
Pomógł: 80
Dołączył: 25.05.2005
Skąd: Szczecin

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

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:

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


a lepszym i szybszym rozwiązaniem będzie:

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


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


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.

Ten post edytował sniezny_wilk 10.06.2008, 08:43:16


--------------------
DevBlog - http://www.bartekrogus.pl
Go to the top of the page
+Quote Post
JoShiMa
post 10.06.2008, 08:43:51
Post #6





Grupa: Zarejestrowani
Postów: 1 374
Pomógł: 149
Dołączył: 1.03.2006

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

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.


--------------------
Zajrzyj, poczytaj, naucz się
Go to the top of the page
+Quote Post
Wieczny
post 10.06.2008, 09:06:45
Post #7





Grupa: Zarejestrowani
Postów: 191
Pomógł: 1
Dołączył: 21.03.2007

Ostrzeżenie: (60%)
XXX--

Cytat(sniezny_wilk @ 10.06.2008, 09:40:04 ) *
A to trochę mało bezpieczne:
[PHP] pobierz, plaintext 
  1. <?php
  2. include($row['value']);
  3. ?>
[PHP] pobierz, plaintext


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ć ?
Go to the top of the page
+Quote Post
phpion
post 10.06.2008, 09:12:27
Post #8





Grupa: Moderatorzy
Postów: 6 072
Pomógł: 861
Dołączył: 10.12.2003
Skąd: Dąbrowa Górnicza
Cytat(Wieczny @ 10.06.2008, 10:06:45 ) *
Możesz mi ktoś zaprezentować prosty przykładowy kod jak by to miało wyglądać ?

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

aczkolwiek ja bym się zastanowił nad szablonami... niekoniecznie SMARTY, zwykłe "szablony" w PHP są wg mnie nawet lepsze.
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

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

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Wersja Lo-Fi Aktualny czas: 31.07.2025 - 06:16
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn