Forum PHP.pl

Witam

Próbuję zrobić bardziej złożoną funkcję walidującą i zastanawiam się czy nie przesadziłem z funkcją check_user:

[PHP] pobierz, plaintext 
<?php
Function check_user($var) {
    $var = trim(filter_var($var,FILTER_SANITIZE_STRING,FILTER_FLAG_ENCODE_HIGH|FILTER_FLAG_ENCODE_LOW,'ucwords')),
    return (strlen($var) < 51 && strlen($var) > 2) ? $var : FALSE;
} 
 
$defs = array(
    'name'      => array( 'filter'  =>  FILTER_CALLBACK,
                          'options' => 'check_user') );
 
$input = filter_input_array(INPUT_POST, $defs);
 
// testujemy 
if ($input['name'] === FALSE) {
    echo 'Za dużo znaków!';
 } else { echo 'Ok'; }
?>
[PHP] pobierz, plaintext 

Proszę o opinię!
Z góry dziękuję
Pozdr.
KaO

Ten post edytował Kao 27.05.2009, 23:23:14

Przesadziłeś z kilkukrotnym wywołaniem strlen" title="Zobacz w manualu PHP" target="_manual.

Poza tym, dlaczego do sprawdzania długości robisz takiego tasiemca...? Wystarczyłby mb_strlen" title="Zobacz w manualu PHP" target="_manual.

PS. Tag tematu.

A strlen" title="Zobacz w manualu PHP" target="_manual raptem 2 razy bo zakres od 3 do 50 znaków chyba wymaga użycia dwukrotnego?

A czy mb_strlen" title="Zobacz w manualu PHP" target="_manual działa szybciej, poza tym, że ma dodatkowo $encoding?

Ps.
W sumie chodzi mi o walidację danych wejściowych POST (pól ze stringami typu: imię, nazwisko, opis, temat itp.) gdzie liczy się aby nie przedostały się "niechciane znaki" i aby długość tych chcianych mieściła się w określonym zakresie.
"Grzebałem" z CALLBACK aby do standardowych filtrów - czysto filtrujących dodać jakieś uniwersalne sprawdzanie długości, ale nie potrafiłem przekazać 2 parametrów min i max długości ciągu znaków.

Proszę jeśli można o korektę mojego kawałka kodu.

Z góry dziękuję.
Pozdr.
KaO

strlen" title="Zobacz w manualu PHP" target="_manual, to jedna z najwolniejszych funkcji. Jeśli chcesz sprawdzać długość, to najefektywniej robisz tak:

[PHP] pobierz, plaintext 
<?php
if(isset($string{51})){
  // $string dłuższe niż 50 znaków
}
 
if(!isset($string{3})){
  // $string krótsze niż 3 znaki
}
?>
[PHP] pobierz, plaintext 

Nie. Jeśli korzystasz z wielobajtowego kodowania znaków, np. utf-8/unicode, to ani strlen" title="Zobacz w manualu PHP" target="_manual, ani powyższy sposób nie zwrócą Ci dokładnego wyniku, jeśli pojawiłyby się w ciągu znaki narodowe.

A czemu po prostu nie dasz wyrażenia regularnego?

[PHP] pobierz, plaintext 
<?php
if(!preg_match('/^[A-ZŁ&brvbar;Ż]{1}[a-zćęłńóż&plusmn;&para;Ľ]{2,14}$/',$imie)){$error = 'niepoprawne imię';}
?>
[PHP] pobierz, plaintext 

Imię od 2 do 14 znaków i pisownia zgodna z regułami ortografii (pierwsza litera duża, reszta małe), z imionami dwuczłonowymi radzę sobie tak, że rozbijam je explodem i waliduję każdy człon oddzielnie (pewnie można od razu pregiem, ale nie jestem w tym za dobry). Oczywiście ideałem jest wykorzystać bazę imion, ale chodzi tylko przykład - podobnie można sprawdzać login i moim zdaniem jest to prostsze niż kombinowanie ze strlen.

Wszystko ładnie, pięknie a co z u i o "umlaut", imiona nie tylko polskie, głownie chodzi mi o ludzi z PL, DE, ale jak przyjdzie ktoś inny to też bym chciał aby był wpis (bo to teraz chodzi o walidację imion w formularzu do wysyłania maila ze strony, to samo będzie się tyczyło TEAMAT'u oraz samego WPISU)

Pozdr.
Kao

Mała rzecz, a cieszy (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)
Dzięki,
pozdr.
KaO