Forum PHP.pl

Postanowiłem na swojej stronie skorzystać z systemu szablonów, aby oddzielić treść od logiki i ułatwić późniejsze modyfikcje. W zaiązku z tym mam takie pytanie, który sposób na przekazanie zmiennych do OPT i zwrócenie komunikatu jest lepszy.

Pierwsza propozycja:
Jakas funkcja wewnątrz klasy:

[PHP] pobierz, plaintext 
<?php
function checkUser($id)
{
	if(!is_numeric($id)
	{
		return -2;
	}
 
	$user = $db -> Execute('SELECT * FROM USERS WHERE `id`='.$id);
	if(!$user -> fields['id'])
	{
		return -1; // Tutaj rownie dobrze moze byc FALSE, albo 'N'
	}
		else
	{
		return $user -> fields;
	}
}
?>
[PHP] pobierz, plaintext 

Plik php:

[PHP] pobierz, plaintext 
<?php
$result = $klasa -> checkUser($_GET['id']);
$tpl -> assign('result', $result);
?>
[PHP] pobierz, plaintext 

No i wreszcie pliki opt:

[PHP] pobierz, plaintext 
<?php
{if $result > 0)
Witaj {$result['login']}
{elseif $result == -1}
Nie ma takiego usera
{else}
Podałeś nieprawidłowe id.
{/if}
?>
[PHP] pobierz, plaintext 

Druga propozycja:
Jakas funkcja klasy:

[PHP] pobierz, plaintext 
<?php
function checkUser($id)
{
	if(!is_numeric($id))
	{
		 return 'Podales nieprawidzlowe ID';
	}
	$user = $db -> Execute('SELECT * FROM `users` WHERE `id`='.$id);
	if(!$user -> fields['id'])
	{
		return 'Nie ma takiego usera';
	}
		else
	{
		return 'Witaj '.$user -> fields;
	}
}
?>
[PHP] pobierz, plaintext 

Plik php:

[PHP] pobierz, plaintext 
<?php
$result = checkUser($_GET['id']);
 
$tpl -> assign('result', $result -> fields['login']);
?>
[PHP] pobierz, plaintext 

No i .tpl

[PHP] pobierz, plaintext 
<?php
{$result}.
?>
[PHP] pobierz, plaintext 

Chodzi mi konkretnie o to, czy jest sens robienia warunkow (podczas pisania czegos bardziej skomplikowanego bardzo pomieszanych) w plikasz szablonow, skoro moge tam przekazac gotowy wynik do wyswietlenia... Co prada w tym przykladzie nie do konca jest jasne o co chodzi, ale np. przy rejestracji uzytkownikowi moze wyswietlic sie 5 roznych bledow (puste pole, powtorzony login, powtorzony mail, zle haslo itd.). Gdy przekazuje wyniki jako liczby (albo wartosci TRUE, FALSE, 'Y', 'N' etc.), w szablonach tworza sie bardzo skomplikowane warunki. A w koncu nie o to chodzi. Czy moge przekazywac gotowy tekst do szablonu i czy ma to sens? Caly skrypt pisalem od podstaw i ma sluzyc tylko za przyklad, wiec gdzies moze byc blad. Nie zwracajcie na to uwagi (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) .

Ten post edytował Albitos 16.08.2006, 04:26:34

NA pewno nie rob tego tak:

[PHP] pobierz, plaintext 
<?php
{if $result > 0)
Witaj {$result['login']}
{elseif $result == -1}
Nie ma takiego usera
{else}
Podałeś nieprawidłowe id.
{/if}
?>
[PHP] pobierz, plaintext 

Stworz tablice z komunikatami bledow, indeksowana kodami bledow, przekaz ja do OPT i pobierz z niej komun. na podst. kodu bledu.

Moglbys mniej wiecej pokazac jak to mialoby wygladac? Musialbym wtedy do OPT dodac funkcje pobierajaca klucz z tablicy, czy tak?