Witam!
Chciałbym poznać wasze zdanie na temat mojego systemu rejestracji który jest fragmentem większej aplikacji. Interesuje mnie, co sądzicie o takim sposobie przkazywania danych i zarzadzania fragmentami kodu HTML (Smarty).

[PHP] pobierz, plaintext 
<?php
 
$registerForm = array('visible' => 'true');
$registerMessage = array('text' => '', 'visible' => 'true');
switch($_GET['option']) {
	case 'modify':
		$modifyUser = getUser($_SESSION['authorized']);
		$smarty->assign('login', $modifyUser[0]['login']);
		$smarty->assign('password', $modifyUser[0]['password']);
		$smarty->assign('email', $modifyUser[0]['email']);
		$smarty->assign('name', $modifyUser[0]['name']);
		$smarty->assign('surname', $modifyUser[0]['surname']);
		$smarty->assign('option', $option = 'update');
		break;
	case 'update':
		//if(!$_POST['login']) $errorMessage = 'login';
		if($_POST['login'] && $_POST['password'] && $_POST['name'] && $_POST['surname']) {
			if(updateUser($_SESSION['authorized'], $_POST['password'], $_POST['email'], $_POST['name'], $_POST['surname'])) {
				$registerForm['visible'] = 'false';
				$registerMessage['text'] = 'Dane zostały zmienione.';
			} else {
				$registerForm['visible'] = 'false';
				$registerMessage['text'] = 'Dane nie zostały zmienione.';
			};
		} else {
			$registerMessage['text'] = 'Wypełnij formularz.';
		};
		break;
	case 'add':
		if($_POST) {
			if(!$_POST['login']) $error .= ' login,';
			if(!$_POST['email']) $error .= ' email,';
			if(!$_POST['password']) $error .= ' hasło,';
			if(!$_POST['name']) $error .= ' imię,';
			if(!$_POST['surname']) $error .= ' nazwisko,';
			$errorMessage = 'Uzupełnij pola:'.$error;
			$smarty->assign('login', $_POST['login']);
			$smarty->assign('password', $_POST['password']);
			$smarty->assign('email', $_POST['email']);
			$smarty->assign('name', $_POST['name']);
			$smarty->assign('surname', $_POST['surname']);
		};
		if($_POST['login'] && $_POST['password'] && $_POST['name'] && $_POST['surname']) {
			if(addUser($_POST['login'], $_POST['password'], $_POST['email'], $_POST['name'], $_POST['surname'])) {
				$registerForm['visible'] = 'false';
				$registerMessage['text'] = 'Użytkownik został dodany.';
			} else {
				$registerForm['visible'] = 'false';
				$registerMessage = 'Użytkownik nie został dodany.';
			};
		} else {
			$registerMessage = 'Wypełnij formularz.';
		};
		$smarty->assign('option', $option = 'add');
		break;
}
$smarty->assign('option', $option);
$smarty->assign('registerForm', $registerForm);
$smarty->assign('registerMessage', $registerMessage);
$smarty->assign('errorMessage', $errorMessage);
 
?>
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
{include file="header.tpl"}
<h3>Rejestracja</h3>
{if $registerMessage.visible == "true"}
<h4>{$registerMessage.text}</h4>
{/if}
<h4>{$errorMessage}</h4>
{if $registerForm.visible == "true"}
<form action="index.php?action=register&user={$login}&option={$option}" method="post">
<table>
{if $registerForm.visible == "true"}
<tr><td>login:</td><td><input name="login" maxlength="30" value="{$login}"><td></tr>
{else}
<tr><td>login:</td><td>{$login}<input type="hidden" name="login" maxlength="30" value="{$login}"><td></tr>
{/if}
<tr><td>email:</td><td><input name="email" maxlength="30" value="{$email}"><td></tr>
<tr><td>hasło:</td><td><input type="password" name="password" maxlength="30" value="{$password}"></td><td>
<tr><td>imie:</td><td><input name="name" maxlength="30" value="{$name}"></td><td>
<tr><td>nazwisko:</td><td><input name="surname" maxlength="30" value="{$surname}"></td><td>
<tr><td colspan="2"><input type="submit" value="wyślij"></td><td>
</table>
</form>
{/if}
{include file="footer.tpl"}
[HTML] pobierz, plaintext 

Może wklejanie tutaj tego kodu nie jest do końca dobrym pomysłęm ale co tam
Mam nadzieję że jakoś się odnajdziecie w tych moich bazgrołąch.
Pozdrawiam!
I jeszcze jedno. System nie jest bezbłędny i jest trochę wybrakowany, Bardziej chodzi mi o to czy system o takim szkielecie jest realizowany prawidłowo.