Forum PHP.pl > [PHP] Pętla, Panel

Pomoc - Szukaj - Użytkownicy - Kalendarz

Wieczny

9.06.2008, 22:35:11

da się to szybciej napisać ?

[PHP] pobierz, plaintext 
<?php
echo "<link rel='Stylesheet' type='text/css' href='style.css' />";						
 echo "
 <div id='leftsite'>";
 
  $result = mysql_query("SELECT id, value, name, align, valign, user
						FROM wpp_panels WHERE align='leftsite' ORDER by id ASC");
 
	while ($row = mysql_fetch_array($result))
	 {
 
		  echo"<table border='1' class='panel'>
		  <tr>
		  <td background='bgpanel.jpg'><b>".$row['name']."</b></td>
		  </tr>
		  <tr>
 
			 <td>";
	 
	  include($row['value']);
 
			 echo"</td>
			  <tr>
 
		  </table>";
 
	 }
 
 
echo "</div>";
 
?>
[PHP] pobierz, plaintext

Pilsener

10.06.2008, 08:06:56

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

10.06.2008, 08:10:43

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

Wieczny

10.06.2008, 08:29:32

Jedno echo na całą stronę .... Ciekawe jak ;D

sniezny_wilk

10.06.2008, 08:40:04

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 
<?php
echo "<link rel='Stylesheet' type='text/css' href='style.css' />";						
 echo "
 <div id='leftsite'>";
?>
[PHP] pobierz, plaintext

a lepszym i szybszym rozwiązaniem będzie:

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

A to trochę mało bezpieczne:

[PHP] pobierz, plaintext 
<?php
include($row['value']);
?>
[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.

JoShiMa

10.06.2008, 08:43:51

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

10.06.2008, 09:06:45

Cytat(sniezny_wilk @ 10.06.2008, 09:40:04 )

A to trochę mało bezpieczne:

[PHP] pobierz, plaintext 
<?php
include($row['value']);
?>
[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

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

10.06.2008, 09:12:27

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 
<?php
$echo = '';
$echo .= '<html><head></head><body>';
$echo .= '<p>lalalalal</p>';
$echo .= '</body></html>';
 
echo $echo;
?>
[PHP] pobierz, plaintext

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.