Forum PHP.pl

Witam, zaczne od pokoazania jak wyglada uklad mojej strony:

index.php

[PHP] pobierz, plaintext 
<?php
ob_start();
require...
require...
require...
 
 
if(isset($_GET['act']))
{
  switch($_GET['act'])
  {
	case 'news' : $site = 'news';
	  break;
		.......
		.......
	default : $site = 'news';
	  break;
  }
}
else
{
  $site = 'news';
}
 
$smarty =  new Smarty;	   
$smarty->assign('site', $site);
$smarty->display('index.tpl');
 
ob_end_flush();
?>
[PHP] pobierz, plaintext 

index.tpl

[PHP] pobierz, plaintext 
<?php
<td //ustawienia">{include_php file="menu.php"}</td>
	<td //ustawienia">
	{if $site eq "news"}
 
	{include_php file="news.php"}
 
	{elseif ......
	..........
	..........
	..........
 
	{/if}	</td>
	<td //ustawienia>{include_php file="login.php"}</td>
?>
[PHP] pobierz, plaintext 

Funkcja require wczytuje mniedzy innymi takie pliki jak ten glowny plik od smartow, polaczenie z baza czy funkcje... W napisanych skryptach nie uzywam require, poniewaz zawsze beda wczytywanie poprzez index, a tam sa juz owe requiry. Wszystko pieknie dzialalo do czasu, az zaczalem tworzyc skrypt logowania. Sam skrypt logowania potrzebowal pliku laczenia z baza, z funkcjami i smartami, ale mam jeszcze jeden plik rozpoczynajacy sesje i nadajacy kilka zmiennych globalnych, ale jak ten plik umieszcze w index require to nie dziala jak nalezy..., czyli tak jakby byl wczytany bezposrednio do skryptu logowania. Tego pliku bedzie potrzebowalo jeszcze kilka innych skryptow...
Co robic? Uzywac require z tym plikiem w kazdym skrypcie wymagajacym go?

A dlaczego chcesz inkludować php do szablonu? Idea szablonu powstała po to, żebyś to co masz załatwić w php załatwił to najpierw a potem dopiero przekazał dane do szablonu i je wyświetlił. A Ty na dodatek mieszając z powrotem to co miało być rozdzielone to jeszcze, z tego co rozumiem wczytujesz gdzieś w środku skrypt, który Ci otwiera sesje. Kompletny bałagan robisz.

To jak proponujesz zrobic uklad strony w smarty tak, aby po bokach bylo menu, u gory logo, a na dole stopka, a w zeby w srodku byly wyswietlane wszystkie podstrony?

Normalnie. O tym jaka ma być podstrona decyduję w obrębie php i informację podaję do skryptu, który ewentualnie includuje odpowiedni plik tpl w miejsce podstrony jeśli podstrony mają różne budowy.

Popatrz sobie na bicykle.pl/index09.php Tak właśnie to realizuję.

index tpl zawiera szkielet strony. Dostaje info o tym jaki dział się ma wyświetlić. Wczytuje tpl odpowiedniego dla tego działu menu lewego i odpowiednie tpl dla zawartości w zależności jakie informacje dostanie.

W szablonach smarty mam tylko pętle warunki i includowanie innych tpl. Żadnego kodu php. Grunt to dobrze przemyśleć tworzenie struktury danych wsadowych dla szablonu i ich obsługę w samym szablonie.

Ten post edytował JoShiMa 22.07.2008, 11:13:03

Zastosowalem twoj sposob, ale mam jeszcze 2 pytania.
1: Np. w pliku news.php mialem w petli while

[PHP] pobierz, plaintext 
<?php
$smarty =  new Smarty;
$smarty->assign...
zmienne
.....
$smarty->display('news.tpl');
?>
[PHP] pobierz, plaintext 

i te zmienne przekazywaly do news.tpl wszystkie zmienne do wyswietlenia (tytul, tresc, itp...), a ze skoro to bylo w petli to sie wyswietlaly wszystkie newsy. Teraz w news.php ucialem new smarty i display a zostawilem same assign, bo index.php wyglada tak:

[PHP] pobierz, plaintext 
<?php
$smarty =  new Smarty;
if(isset($_GET['act']))
{
  switch($_GET['act'])
  {
	case 'news': 
	include ('news.php');
	$smarty->assign('site', 'news.tpl');
	break;
 
   cos tam....
 
  }
}
else
{
  cos tam....
}
 
$smarty->display('index.tpl');
?>
[PHP] pobierz, plaintext 

a index.tpl krótko, zawiera na srodku {include file="$site"}

..i wyswietla sie tylko jeden news. Troche pokopane, no ale moze sa inne sposoby ;/

2: Jak umieszcze w index.tpl, aby wyswietlalo np. menu.tpl - to wyswietla sam szablon, bo niby jak ma sie zaladowac menu.php?

No pewnie, że wyświetli się jeden, bo przekazujesz tylko jeden. Jak przekażesz tablicę newsów to w pętli wczytasz tyle razy news.tpl z odpowiednimi parametrami ile razy Ci to będzie potrzebne to znaczy tyle ile newsów.


No nie ma się załadować. menu.php uruchamiasz robisz w obrębie działania twoich skryptów php, wstrzykujesz do szablony dane dla menu a potem w szablonie wczytujesz tpl menu, który te dane zinterpretuje i pokaże to co należy.

Znowu mam problem
Nie moge przekazac tablicy newsow w taki sposob (akurat pokazuje na przykladzie innego skryptu)...

[PHP] pobierz, plaintext 
<?php
$sources=array();
  $sql="SELECT * FROM news_sources ORDER BY sourceID DESC";
  $result=mysql_query($sql) or die(mysql_error());
  while($source=mysql_fetch_array($result))
  {
	$sources[]=$source;
  }  
  $smarty->assign('sources', $sources);
  $smarty->display('news_sources.tpl');
?>
[PHP] pobierz, plaintext 

i
tpl

[HTML] pobierz, plaintext 
<ol>
  {section name=idx loop=$sources} 
  <li>{$sources[idx].link} <a href="news.php?act=delsource&id={$sources[idx].sourceID}">usuń</a></li>
  {/section}
</ol>
[HTML] pobierz, plaintext 

... pomimo ze to dziala dobrze, bo wczesniej musze zastosowac kilka funkcji na zmiennych z bazym, a potem dopiero moge przekazc szablonowi.
Co robic? Mecze sie z tym juz pare godz...

EDIT:
zrobilem cos takiego i dziala:

[PHP] pobierz, plaintext 
<?php
$sor=array();
  $sources=array();
  $sql="SELECT * FROM news_sources ORDER BY sourceID DESC";
  $result=mysql_query($sql) or die(mysql_error());
  while($source=mysql_fetch_array($result))
  {
	$sor['link']=$source['link'];
	$sor['sourceID']=$source['sourceID'];
	$sources[]=$sor;	
  }  
  $smarty->assign('sources', $sources);
  $smarty->display('news_sources.tpl');
?>
[PHP] pobierz, plaintext 

...ale wedlug mnie kod wyglada nieestetycznie Moze jakos inaczej to zapisac?

Ten post edytował iMax 22.07.2008, 21:10:00

zamień while na foreach ... do smarty moim zdaniem jest lepsza ... to samo w *.tpl zamiast section używaj foreach ...

przynajmniej ja tak robie, 0 problemów ...

Dwie bzdury. foreach jest dobra w jednych przypadkach a while (w php) i section (w tpl) w innych Foreach jest typową pętlą do tablic asocjacyjnych i nie ma sensu stosowac jej na siłę do zwykłych tablic.

iMax Nie wydaje Ci się, że ten kod jest sensowniejszy niż to co miałeś do tej pory, czyli poplątanie z pomieszaniem? Clou tkwi w dobrej konstrukcji danych wsadowych tak, alby łatwo je było wyświetlić w formularzu bez zbędnych przekształceń, które można wykonać wcześniej w php.

A jak chcesz, żeby było jeszcze ładniej i bardziej elastycznie daj:

[PHP] pobierz, plaintext 
<?php
$sor['tpl']='news_sources.tpl'
?>
[PHP] pobierz, plaintext 

Wtedy do tablicy $sources[] będziesz mógł dodawać różne sekcje o różnych podszablonach i wszystkie obsługiwać jedną pętlą section.