Obsługa wielostronicowego formularza, smarty + session + post Opcje

[onlyX]

Stworzyłem przed chwilą szkielet skryptu obsługi kilkustronicowego formularza.
Prosiłbym, żebyście przyjżeli się kodowi i sprawdzili czy jest on logicznie poprawny i czy nie zawiera błędów:

[PHP] pobierz, plaintext 
<?
//
// Rejestracja nowego uzytkownika
//
require 'includes/class_quick.php';
$quick = new quick;
 
// jeśli zalogowany
if($_SESSION['log']) {
	header(&#092;"Location: index.php\");
	exit;
}
 
if(!$_POST['reg_page']) {
	//wyświetl regulamin
	$quick->smarty->assign(&#092;"reg_page\",\"1\");
	$quick->smarty->display('../templates/register_reg.tpl');
}
else if($_POST['reg_page']==1) {
	//wyświetl formularz rejestracyjny
	$quick->smarty->assign(&#092;"reg_page\",\"2\");
	$quick->smarty->display('../templates/register_form.tpl');
}
else if($_POST['reg_page']==2) {
	//procedury sprawdzenia danych
	//jęsli dane są poprawne
	//wyśqietl potwierdzenie wprowadzonych danych
	$quick->smarty->assign(&#092;"reg_page\",\"3\");
	$quick->smarty->display('../templates/register_confirm.tpl');
	// jeśli dane nie są poprawne
	//wróć do formularza
	$quick->smarty->assign(&#092;"reg_page\",\"2\");
	$quick->smarty->display('../templates/register_form.tpl');
}
else {
	//jeśli dane wymagane (login, hasło, confirm, email) zostały podane
	//utworzenie nowego użytkownika w bazie danych
	//INSERT INTO ...
	//wyświetl podziękowania
	$quick->smarty->assign(&#092;"info\",\"Dziękujemy za rejestrację, za chwile zostaniesz przeniesiony do strony logo
w
nia.\");
	$quick->smarty->display('../templates/register_confirm.tpl');
	unset($_POST['reg_page'])
	//jeśli nie wróć do formularza
	$quick->smarty->assign(\"reg_page\",\"2\");
	$quick->smarty->display('../templates/register_form.tpl');
}
 
?>
[PHP] pobierz, plaintext 

Mam też kilka pytań:
- W jaki sposób zabrać się do sprawdzania poprawności danych?
Myślałem o funkcachi sprawdzających każde pole z osobna (czy jest liczbą, czy jest e-mailem, itp.) i zwracających prawdę lub fałsz. Potem bym sprawdzał tylko czy wszędzie jest true. Jeśli byłoby false wtedy wracałoby do formularza i wyświetlała nad nim informacje o niepoprawnych danych.

[sf]

Ja znalazłem trochę więcej różnic ...

1) użycie switch" title="Zobacz w manualu PHP" target="_manual, ponieważ bez sensu pisać if else if else if else
2) Twoja próba sprawdzania we funkcji... czyli do każdego pola będzie osobna funckja? po co? odrazu sprawdzam w każdym case czy zmienna jest błędna, nie potrzebuje odwoływać się do funkcji ... chyba, że napiszesz coś bardziej skomplikowanego do sprawdzenia poprawności email to wtedy tak, ale czy jest liczba... do tego nie trzeba pisać funkcji
3) w moim rozwiązaniu wyświetalnie formularza odbywa się po sprawdzeniu danych zależnie od zmiennej krok, a nie w każdym case sprawdzam if, który formularz mam wyświetlić... oszczędność pisania bo tylko jeden raz występuje echo $err;, a nie w każdym if

no i znalazłem jednego buga... (EDIT: zaleznie od tego jak wyglada kod strony bo tej czesci kodu nie widac (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) jesli to jest reg_page=0 to zadziala, jesli zmienna nie jest ustawiona to wywali notice jak napisalem (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) )

[PHP] pobierz, plaintext 
<?php
if(!$_POST['reg_page'])
?>
[PHP] pobierz, plaintext 

to wyrzuca notice ... (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) jak już to użyj isset

albo inne rozwiązanie jest takie :

(int)@$_POST['reg_place'] wtedy zwraca 0 jesli nic nie zdefiniowalismy, albo jest to litera czy inny znak od liczby :]

Ten post edytował sf 11.09.2004, 12:23:38