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
$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);
?>
{include file="header.tpl"}
{if $registerMessage.visible == "true"}
<h4>{$registerMessage.text}
</h4> {/if}
{if $registerForm.visible == "true"}
<form action="index.php?action=register&user={$login}&option={$option}" method="post"> {if $registerForm.visible == "true"}
{else}
<tr><td>login:
</td><td>{$login}
<input type="hidden" name="login" maxlength="30" value="{$login}"><td></tr> {/if}
<tr><td>hasło:
</td><td><input type="password" name="password" maxlength="30" value="{$password}"></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> {/if}
{include file="footer.tpl"}
Może wklejanie tutaj tego kodu nie jest do końca dobrym pomysłęm ale co tam (IMG:
http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
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.
Ten post edytował lharmatowski 17.05.2006, 13:12:41
implementacja rejestracji
