Właściwa konstrukcja funkcji preg_match. Wymóg hasła

Question

Witam, tworzę wymogi hasła w formularzu rejestracji.

Chciałbym aby w haśle były następujące wymogi: 

* minimum 1 duża litera
* minimum 1 cyfra, tak aby nie musiały być one obok siebie(kolejność nie ma znaczenia)
* minimum jeden znak specjalny (./;',!@#$%^& itd.)
*długość hasła od 6 do  30 znaków

Zrobiłem coś takiego:

function check_pass($password1) {
    if(preg_match('/^(?=.*[A-Z])(?=.*\d)(?=[^a-zA-Z\d])\w{6,30}$/', $password1)){
        return true;
    }
    else{
        return false;
    }

if (!check_pass($password1)) 
{
	echo " Źle!"; 
    exit();
}

Niestety to nie działa, bo jak wpiszę np. Testowo1. to dalej mówi że jest źle. Próbowałem jeszcze coś takiego, ale to także nie działa:

function check_pass($password1) {
    if(preg_match('/^(?=.*[A-Z])(?=.*\d)(?=.*[$%^&])\w{6,30}$/', $password1)){
        return true;
    }
    else{
        return false;
    }

Co mam nie tak w funkcji wymogu? Bardzo mi na tym zależy więc proszę o pomoc. Pozdrawiam! Będę wdzięczny za każdą wskazówkę.

Comments

Co mówi że jest źle?

---

Wywołuje to: echo " Źle!";

Answer 1

W pierwszym regexp problemem jest ostatni warunek: \w{6,30}

w którym deklarujesz, że w ciągu mogą być jedynie znaki zawarte w \w, a jednocześnie wymagasz wcześniej znaku z poza tego zakresu (znaku specjalnego jak to nazywasz), czyli de facto robisz dwa kryteria, wzajemnie się wykluczające :)

W drugim przypadku niedziała z tego samego powodu :)

na końcu zamiast \w umieść znaki, które dopuszczasz jako prawidłowe dla całego ciągu i powinno być oki.

Comments

Czy napiszesz mi tego przykład bo nie rozumiem tego. Czy liczbę znaków mam zastąpić w ten sposób?

if ((strlen($password1) < 6) || (strlen($password1) > 30)) {
echo "Password must be between 6 to 30 characters!";
exit();                                  
}

 

---

nie mam teraz na mobilce za bardzo jak testować, ale spróbuj tak:

/^(?=.*[A-Z])(?=.*\d)(?=.*[$%^&])[\w$%^&]{6,30}$/

tylko pamiętaj, że \w obejmuje też znak "_", jeśli ma go nie być to w miejsce \w daj A-Za-z\d.

Przetestuj i daj znać czy działa, jak nie to odezwij się jutro to na PC coś pomyślę.

---

Nie działa ta konstrukcja co napisałeś, dalej pokazuje błąd

---

Rozwiązałem to tak: '/^(?=.*\d)(?=.*[A-Za-z])[0-9A-Za-z!@#$%]{6,30}.*$/'

Dziękuję za wszelką pomoc

---

Poprawka, tak: '/^(?=.*[A-Z])(?=.*\d)(?=.*[^A-Za-z\d]).*$/'