Forum PHP.pl

Witam

Dopiero co raczkuje w PHP, na początek chciałem stworzyć skrypt walidujący prosty formularz.
Dane przekazywane są z formularza w postaci dwóch tablic: required i optional.

Mój problem polega na tym że po zatwierdzeniu formularza, wypełnione pola stają się na nowo puste,
a staram się wymusić żeby tak nie było.

Wszystko znajduje się w jednym pliku:

skrypt:

[PHP] pobierz, plaintext 
<?php
 
   error_reporting(E_ALL);
   $a_req = (array) $_POST['required'];
   //$a_opt = (array) $_POST['optional'];
	array_validate($a_req);
 
	/*
	Przeprowadza walidację danych z tablicy.
	Wykonując kolejne funkcje pomocnicze, gdy poprzednia zakończy swoje
	działanie z powodzeniem.
 
	@param $array - tablica z wartościami do walidacji
	@return true - jeśli całe sprawdzanie zakończyło się powodzeniem.
	*/
	function array_validate($array) {
 
		if (array_empty_val($array)) {
			echo 'Nie wszystkie pola są wypełnione!<br />';
			return false;
		}	
		array_walk($array, 'disable_danger_chars');
		array_walk($array, 'show_array_value');
		return true;
	}
 
	/*Usuwa z przekazanej zmiennej znaki mogące zniszczyć
	strukturę strony itd. Dodatkowo dodając znak ucieczki w przypadku
	nie dokonania tego automatycznie.
 
	@param $value - wartość do sprawdzenia
	*/
 
	function disable_danger_chars(&$value) {
		$value = htmlspecialchars($value);
		if (!get_magic_quotes_runtime())
			 $value = addslashes($value);
	}
 
	/*Sprawdza obecność pustych pól w tabeli.
	W zależności od stanu pola ustala wartość w tabeli zmiennych sesyjnych
	przy pomocy indeksu tego pola.
 
	@return TRUE jeśli występują puste pola w tabeli
	*/
 
	function array_empty_val(&$array) {
		$has_empty = FALSE;
		foreach ($array as $key => &$value) {
			$value = trim($value);
			if (empty($value)) {
				$_SESSION[$key] = FALSE;
				$has_empty = TRUE;
				$a_empty_vals[] = $key;
			} else $_SESSION[$key] = TRUE;
		}
		return $has_empty;
	}
 
	/*
    Funkcja operująca na tablicy, przygotowana do przekazania funkcji array_walk().
	*/
	function show_array_value($value, $key) {
		echo $key.'=>\''.$value.'\'<br />';
	}
 
	function html_warn_msg($value) {
		static $msg = "<tr><td colspan=2 align=center><b>Powyższe pole jest wymagane</b></td></tr>";
		if (isset($_SESSION[$value])) echo $msg;
	}
 
	function show_val_by_key($key, $flag=TRUE) {
		if ($flag) {
		if (isset($a_req[$key])) echo $a_req[$key];
		} 
		// else to samo w przypadku tabeli optional.
	}
 
?>
 
[PHP] pobierz, plaintext 

formularz html:

[HTML] pobierz, plaintext 
<form id="register_frm" action="" method="post">
	<table id="frm_tbl">
		<caption>Formularz Rejestracyjny</caption>
		<tr>
			<th>Identyfikator</th>
		</tr>
		<tr>
			<td><label for="login">Login</label></td>
			<td><input type="text" name="required[login]" id="login" maxlength="50" value="<?php show_val_by_key('login') ?>"/></td>
		</tr>
		<?php html_warn_msg('login') ?>
		<tr>
			<td><label for="pass">Hasło</label></td>
			<td><input type="password" name="required[pass]" id="pass" maxlength="50" /></td>
		</tr>
		<tr>
			<td><label for="pass2">Powtórz hasło</label></td>
			<td><input type="password" name="required[pass2]" id="pass2" maxlength="50"/></td>
		</tr>
		<tr>
			<td><label for="email">E-mail</label></td>
			<td><input type="text" name="required[email]" id="email" maxlength="50"/></td>
		</tr>
		<tr>
			<td><label for="email2">Powtórz E-mail:</label></td>
			<td><input type="text" name="required[email2]" id="email2" maxlength="50"/></td>
		</tr>
		<tr>
			<th>Dane Personalne</th>
		</tr>
		<tr>
			<td><label for="name">Imię</label></td>
			<td><input type="text" name="required[name]" id="name" maxlength="50" value="<?php show_val_by_key('name') ?>"/></td>
		</tr>
		<?php html_warn_msg('name') ?>
 
		<tr>
			<td><label for="surname">Nazwisko</label></td>
			<td><input type="text" name="required[surname]" id="surname" maxlength="50" value="<?php show_val_by_key('surname') ?>"/></td>
		</tr>
		<?php  html_warn_msg('surname') ?>
                <!-- inne pola -->
		<tr>
			<td colspan="2" align="left"><input type="reset" name="reset_btn" value="Wyczyść"/></td>
		</tr>
		<tr>
			<td colspan="2" align="right"><input type="submit" id="submit_btn" value="Rejestruj"/></td>
		</tr>
	</table>	
	</form>
[HTML] pobierz, plaintext 

Proszę o porady.

szczerze nie wczytuje się w te funkcje, ale jeśli chcesz by po wypełnieniu formularza pola nie były puste musisz dodać w polach formularza np. tu:

opcję value,
w ten sposób:

[PHP] pobierz, plaintext 
value="<?php if(isset($_POST['pass'])) echo $_POST['pass']; ?>"
[PHP] pobierz, plaintext 

oczywiśćie przykład z hasłem nie najlepszy, ale to tylko przykład:)

Ten post edytował lamcpp 29.10.2009, 00:29:51

Wszystko zależy od Twojego podejścia do formularza. Brak pliku w action sugeruje, że całość walidacji i przetwarzania jest wykonywana w tym samym pliku. Jeśli tak to znaczy, że po otworzeniu strony po raz pierwszy nie masz nic, ale jeśli dałeś Submit i jest to przeładowanie, to masz dostęp do zmiennej $_POST lub jej odpowiednika jeśli używasz programowania obiektowego i masz to w jakiś sposób w nim zaszyte. Jeśli jest to proste to rozwiązaniem jest pomysł kolegi, czyli wpisujesz do value zmienne POST. Jeśli robisz to obiektowo, to przekazujesz do formularza obiekt z wartościami domyślnymi value tych pól. Tyle, że po wciśnięciu Submit w owym obiekcie wartości domyślne zostają zastąpione wartościami z POST, przy czym w takiej sytuacji mogą one już być w postaci poprawionej przez system walidacji. Przykładowo:

[PHP] pobierz, plaintext 
<?php
$form = array ('example1' => 'domyślna', 'example2' => 'domyślna', 'example3' => 'domyślna');
$error =array();
if( array_key_exists('send', $_POST) ) {
    $form['example1'] = validation( $_POST['example1'] );
    $form['example2'] = validation( $_POST['example2'] );
    $form['example3'] = validation( $_POST['example3'] );
    //a tutaj zrobisz sobie co tam jeszcze chcesz z tymi danymi
}
?>
<form action="" method="post" name="simple_form">
<?php show_error( $error['example1'] ) ?>
<input type="text" name="example1" value="<?php echo $form['example1']; ?>" />
<?php show_error( $error['example2'] ) ?>
<input type="text" name="example2" value="<?php echo $form['example2']; ?>" />
<?php show_error( $error['example3'] ) ?>
<input type="text" name="example3" value="<?php echo $form['example3']; ?>" />
<input type="submit" name="send" value="Wyślij" />
</form>
[PHP] pobierz, plaintext 

Dałem Ci taki zarys jak by to mniej więcej mogło być proceduralnie rozwiązane. Zauważ, że przygotowuje sobie już tam grunt pod komunikaty błędów itp. A w zależności od tego czy ktoś posłał formularz, zmieniają się wartości pól. Oczywiście to tylko zgrubne, byś widział JAK by to mogło wyglądać, bo osobiście nieco bym tam pozmieniał pod kątem innego rozwiązania walidacji, systemu komunikatów błędów i kilku rzeczy ale poza tym sam szkielet jest już prawidłowy i róznica będzie tylko drobna w implementacji

@down: Owszem... To tylko przykład. To jak postąpisz z polami w walidacji zależy od piszącego. Ja rozbiłem na osobne by zilustrować, że należy to zrobić i w mniej więcej jakim miejscu skryptu niż to, że w ten sposób. Wspominając o tym, że inaczej bym rozwiązał walidację miałem na myśli właśnie przekazanie albo całości, albo pozostawienie na poziomie rozbitym, ale potratowanie od razu pól uniwersalną funkcją (zamiast robić to dopiero na poziomie głównej funkcji walidującej), która je zwaliduje mając określone parametry zmieniające jej zachowanie w stylu $pole = validate( $pole, 'valid_email'), $pole = validate($pole, 'valid_text', 150) czy $pole = validate( $pole, 'required') by uzyskać łatwo rozszerzalny formularz. Dlatego też pisałem że szkielet powinien być mniej więcej podobny, choć mogą zaistnieć różne implementacje Przykładowo wyświetlanie błędów. Wielu robi je zbiorczo na górze formularza. Według mnie to mylące bo trzeba potem szukać gdzie dano pole jest. Lepiej oznaczyć pola z błędem w jakiś sposób. Czy to kolorem, czy komunikatem, ale nie u góry całości, ale bezpośrednio nad/pod tymże polem.Widze, że też podobnie podchodzisz do sprawy.

Co do walidacji znów, to sprawdzanie pól obowiązkowych już na starcie strony może Ci błędy powodować. O ile nie sprawdzasz czy zmienna POST istnieje, a z kodu php przytoczonego nie można tego wywnioskować(choć już sprawdzanie pól jest, ale w metodzie wyświetlającej). To samo tyczy się value. W całym kodzie html nie masz w części kodu nadawania im jakichś wartości, więc lamcpp nie powinien być za to przez Ciebie rugany. Nie masz tego w kodzie = nie stosujesz tego = można Ci to podsunąć jako rozwiązanie. Proste Z kodu jaki podałeś wynika, że:
1) konwertujesz do tablicy formularz (wspominasz, że takie 2 będą - required i optional) i walidujesz
2) masz kilka funkcji w tym jedną wyświetlającą i nawet stosujesz ją w kilku przypadkach, tyle że potrzebuje ona jakichś danych, których zwyczajnie nie masz przez część życia formularza i dodatkowo tylko w części z nich stosujesz wyświetlanie. To zrozumiałe, że olewasz value dla password, ale email już może ją mieć.
3) zauważ, że nie masz nigdzie czegoś takiego jak dane domyślne. Jeśli jest to rozwiązane tak, że sprawdzanie jest na początku to próbujesz metody wyświetlające wywołać dla tablicy $required, której struktury na moment wejścia na stronę formularz i skrypt nawet nie znają. Dlatego ja mam na samym starcie ją utworzoną i zainicjowaną jako $form i dopiero po walidacji i przetwarzaniu przypisuje do niej przetworzone $_POST. To czy przypiszę każde pole z osobna czy może po prostu $form = $_POST zależy od tego co zrobiłem podczas walidacji z nimi.
4) widzę, że masz isset dla wyświetlania więc teoretycznie punkt 3 można by nawet sobie odpuścić. To bardziej moje podejście do problemu, by unikać ciągłego sprawdzania zawartości. Chciałem jednak rozdzielić tablicę post i wartości formularza wyświetlane, te bowiem nie zawsze są takie same.
5) myślę, że pewnym ułatwieniem byłoby dla Ciebie takie zmienienie funkcji walidującej by to ona decydowała co jest, a co nie jest opcjonalne. W ten sposób upraszczasz sobie zapis bo wyskakujesz z tablic required i optional.

Ten post edytował thek 29.10.2009, 09:35:25