Funkcja do sprawdzania treści z formularza Opcje

[andrzejt17]

Proszę was o ocene skryptu pod kątem przydatności i funkcjonalności oraz proszę o wskazówki :) Czyli co zmienić, czego nie używać itp :)

[PHP] pobierz, plaintext 
<?php
// (kod)
 
function checkString($name, $value, $length, $type='all') {
	function showAlert($msg) {
		print ' <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
				<script type="text/javascript">
					alert("'.$msg.'");
				</script>';
	}
 
	$value = strtolower(makeSafe($value));
	$charsToReplace = array('ą', 'ż', 'ź', 'ś', 'ę', 'Ć', 'ń', 'ł', 'ó', 'Ą', 'Ż', 'Ź', 'Ś', 'Ę', 'Ć', 'Ń', 'Ł', 'Ó');
	$valueToCount = str_replace($charsToReplace, 'x', $value);
	$length = explode('-', $length);
	if(strlen($length[0]) > 0 && strlen($length[1]) > 0) {
		$min = $length[0];
		$max = $length[1];
	} else {
		echo 'checkString length arg failed';
		exit;
	}
	if(strlen($valueToCount) == 0) {
		showAlert('Pole \''.$name.'\' jest obowiązkowe, nie może być puste.');
		return false;
	} elseif(strlen($valueToCount) <= $min) {
		showAlert('Liczba znaków w polu \''.$name.'\' jest za mała. Wymagana minimalna liczba znaków: '.$min.'.');
		return false;
	} elseif(strlen($valueToCount) >= $max) {
		showAlert('Liczba znaków w polu \''.$name.'\' jest za duża. Wymagana maksymalna liczba znaków: '.$max.'.');
		return false;
	}
 
	if($type == 'n') {
		if(!preg_match ('/^[0-9]+$/', $value)) {
			showAlert('Zawartość pola \''.$name.'\' musi być liczbą.');
			return false;
		}
	} elseif($type == 'tl') {
		if(!preg_match ('/^[0-9A-Za-z._-]+$/', $value)) {
			showAlert('Zawartość pola \''.$name.'\' może zawierać cyfry, małe i duże litery bez polskich znaków diakrytycznych oraz znaki specjalne: (._-)');
			return false;
		} 
	} elseif($type == 'm') {
		if(!preg_match ('/^([0-9a-z]([-.\w]*[0-9a-z])*@(([0-9a-z])+([-\w]*[0-9a-z])*\.)+[a-z]{2,6})$/i', $value)) {
			showAlert('Podany adres email jest nieprawidlowy.');
			return false;
		}
	}
 
	return true;	
}
 
// (kod)
?>
[PHP] pobierz, plaintext 

Do tego funcja makeSafe();

[PHP] pobierz, plaintext 
<?php
// (kod)
 
function makeSafe($string) {
	if(get_magic_quotes_gpc()) {
		$string = stripslashes($string);
	}
	$string = mysql_real_escape_string(trim($string));
	$string = htmlspecialchars($string);
	$string = str_replace("\r\n", "", $string);
 
	return $string;
}
 
 
// (kod)
?>
[PHP] pobierz, plaintext 

Wywyłanie:

[PHP] pobierz, plaintext 
<?php
// (kod)
// sprawdzenie testowo jednego pola z formularza
 
$imieUsera = makeSafe($_POST['imie']);
$nazwiskoUsera = makeSafe($_POST['nazwisko']);
 
if(checkString('Imie', $imieUsera, '3-10', $type='tl') && checkString('Nazwisko', $nazwiskoUsera, '3-10', $type='tl')) {
	echo 'Wszystko ok';
} else {
	echo 'Coś lipa';
}
 
// (kod)
?>
[PHP] pobierz, plaintext 

Ten post edytował andrzejt17 13.05.2011, 17:11:18

[Zyx]

Funkcje sprawdzające poprawność danych to dopiero początek, a nie powinny one niczego wyświetlać z prostego powodu: zaczniesz nowy projekt, skopiujesz kod i klops. Musisz przepisywać swoje funkcje, by je dopasować do nowej szaty graficznej, nowych wymagań i na pewno przy tym przepisywaniu zrobisz gdzieś błąd. Kod po to dzieli się na mniejsze, wyspecjalizowane w jednym zadaniu kawałki, by można ich było używać wielokrotnie w różnych sytuacjach.