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

[thek]

Dziwisz się? IF to operacja logiczna i wystarczy jeden false a cały warunek idzie w piach, bo używasz && czyli I logicznego. A zobacz kiedy to jest prawdziwe... Tylko gdy wszystkie warunki są prawdziwe. Jedna pierdółka na false i wszystko leży.

Dorzućmy odpowiedzi Ci... Czy jest sens stosować oddzielnie funkcje? Tak! Ponieważ wtedy masz je bardziej uniwersalne. Zamiast potem pisać pierdylion funkcji lub wywoływać jedną z różnymi, nie zawsze sensownymi parametrami, posługujesz się kilkoma w konkretnych sytuacjach. I tak musisz przejść przez nią całą i sprawdzić X warunków IF i jako parametr w wywołaniu dla tych miejsc które opuszczasz ustawić "tego nie rób". Innymi słowy: mniejsza funkcja -> mniej roboty. A czy jest sens robić jeden kombajn i pamiętać co jaki parametr znaczy czy prościej zrobić mniejsze i wyspecjalizowane funkcje, które nie tylko w tym jednym miejscu wykorzystasz, ale w dowolnym fragmencie?

ctype_digit akurat działa dobrze, poza tym nic o niej nie pisałem (IMG:style_emoticons/default/smile.gif)

Jeśli printujesz coś PRZED html, to chyba zupełnie nie rozumiesz, że tak nie powinno się robić. dokument html powinien się znacznikiem html zaczynać czy też specyfikacją DTD i nic nie ma prawa przed nim być wypisane. To zwyczajny błąd. To co chcesz osiągnąć, czyli wyświetlenie komunikatu błędu wraz z przekierowaniem na określoną stronę, rozwiązuje się zupełnie inaczej... Samą walidację także podejrzyj w nieco innych stronach, bardziej profesjonalnych - jakieś darmowe cms choćby.