[HTML][PHP]Formularz - wykrywanie pustych inputów, określenie znaków Opcje

[StadiPL]

Witam
Mam pewien problem i nie potrafię znaleźć rozwiązania na internecie. Chciałem zacząć powoli i małymi kroczkami robić sobie system kont i logowania. Zrobiłem sobie prosty formularz w HTLM i zacząłem pisać do niego odpowiedni kod PHP. Mój kod potrafi już sprawdzić czy hasło oraz jego powtórzenie się zgadza (to samo z e-mailem), jednak zauważyłem przy testach że mogę równie dobrze zostawić pola puste i też będzie się to zgadzać (logiczne). Moje pytania są następujące:
Jak sprawdzać czy pole nie jest puste?
Jak ograniczyć użytkownika aby mógł wpisać tylko wybrane przezemnie znaki?
Jak sprawdzić czy email jest poprawny? tzn czy posiada @ i domene
Dołączam kody:

[HTML] pobierz, plaintext 
Rejestruj się:
<form action="register_function.php" method="post">
nick: <input type="text" name="nickname" /><br />
haslo: <input type="password" name="password" /><br />
powtorz haslo: <input type="password" name="password_r" /><br />
email: <input type="text" name="email" /><br />
powtorz email: <input type="text" name="email_r" /><br />
<input type="submit" value="Rejestruj" />
</form>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
  if ($_POST['password'] === $_POST['password_r']) {
    if ($_POST['email'] === $_POST['email_r']) {
      echo "GIT";
    } else {echo "Podane adresy email nie sa identyczne"; }
  } else {echo "Podane hasla nie sa identyczne"; }
?><br />
[PHP] pobierz, plaintext 

[Szymciosek]

1) W html5 (może i wcześniej) wprowadzono required
<input type="text" name="username" required />

2) Regexp
3) Regexp (email)

[session]

Jak sprawdzać czy pole nie jest puste?

required to bajer najlepiej jeszcze:

[PHP] pobierz, plaintext 
if(empty($_POST['nickname'])){echo'Wypełnij wszystkie wymagane pola';} // I tak dla wszystkich wymaganych 
[PHP] pobierz, plaintext 

Jak ograniczyć użytkownika aby mógł wpisać tylko wybrane przezemnie znaki?

Jak kolega wyżej napisał: wyrażenia regularne i funkcja preg_match

Jak sprawdzić czy email jest poprawny?

Najlepiej mało znanym:

[PHP] pobierz, plaintext 
if(!filter_var($_POST['email'],FILTER_VALIDATE_EMAIL)){echo'Nieprawidłowy adres email';}
[PHP] pobierz, plaintext 

PS. Nie używaj operatora === (zarezerwowoanego dla typów bool) do porównań tylko: ==

Ten post edytował session 11.05.2013, 23:59:05

[StadiPL]

Spoko, tylko jak dokładnie to wstawić, bo jak dam tak to wyświetli że mam uzupełnić, ale mimo to wykona dalej :/

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['nickname'])){echo'Wypełnij wszystkie wymagane pola';}
  if ($_POST['password'] === $_POST['password_r']) {
    if ($_POST['email'] === $_POST['email_r']) {
      echo "GIT";
    } else {echo "Podane adresy email nie sa identyczne"; }
  } else {echo "Podane hasla nie sa identyczne"; }
?>
[PHP] pobierz, plaintext 

reszte waszych odpowiedzi sprawdze zaraz/jak uporam sie z tym ;d

[nikestylex7]

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['nickname'])){
echo'Wypełnij wszystkie wymagane pola';
}else if(empty($_POST['password']) || empty($_POST['password_r'])){
 echo'Wypełnij wszystkie wymagane pola';
}else if($_POST['password'] != $_POST['password_r']){
    echo'Hasła różnią się!';
}
// i tak dalej to tylko przykład żeby Cię nakierować :)
?>
[PHP] pobierz, plaintext 

[StadiPL]

Ok, to zorumiem. jezeli nick jest pusty wyswietla echo, jak nie to jedzie dalej itd... tylko gdzie jak sprawdzi calosc i okaze sie ze wszystko jest ok to ma wykonac reszte kodu (w moim przypadku jak na razie to echo "git";). Gdzie to wstawić? ;] robie tak:

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['nickname'])){echo'Podanie nazwy uzytkownika jest wymagane';}
else if(empty($_POST['password']) || empty($_POST['password_r'])){echo'Podanie hasla i jego powtorzenie jest wymagane';}
else if($_POST['password'] != $_POST['password_r']){echo'Podane hasla roznia sie';}
else if(empty($_POST['email']) || empty($_POST['email_r'])){echo'Podanie e-maila i jego powtorzenie jest wymagane';}
else if($_POST['email'] != $_POST['email_r']){echo'Podane e-maile roznia sie';}
echo "GIT";
?>
[PHP] pobierz, plaintext 

[nikestylex7]

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['nickname'])){
echo'Podanie nazwy uzytkownika jest wymagane';
}
else if(empty($_POST['password']) || empty($_POST['password_r'])){
echo'Podanie hasla i jego powtorzenie jest wymagane';
}
else if($_POST['password'] != $_POST['password_r']){
echo'Podane hasla roznia sie';
}
else if(empty($_POST['email']) || empty($_POST['email_r'])){
echo'Podanie e-maila i jego powtorzenie jest wymagane';
}
else if($_POST['email'] != $_POST['email_r']){
echo'Podane e-maile roznia sie';
}else{
echo "GIT";
}
?>
[PHP] pobierz, plaintext 

Ten post edytował nikestylex7 12.05.2013, 00:55:30

[StadiPL]

Gdzieś wyczytałem że ! ma zrobić "zaprzeczenie". Czyli empty normalnie zwraca gdy pusty, a !empty wyswietla gdy jest rozny od pustego. Zgodnie z tą myślą zrobiłem coś takiego:

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['nickname'])){echo'Podanie nazwy uzytkownika jest wymagane';}
else if(empty($_POST['password']) || empty($_POST['password_r'])){echo'Podanie hasla i jego powtorzenie jest wymagane';}
else if($_POST['password'] != $_POST['password_r']){echo'Podane hasla roznia sie';}
else if(empty($_POST['email']) || empty($_POST['email_r'])){echo'Podanie e-maila i jego powtorzenie jest wymagane';}
else if($_POST['email'] != $_POST['email_r']){echo'Podane e-maile roznia sie';}
else if(!preg_match('/^[a-zA-Z0-9]$/D', $_POST['nickname'])) {echo "nie ok";}
else { echo "GIT"; }
[PHP] pobierz, plaintext 

Oczywiście niestety nie działa. Czegos nie rozumiem, czy to z wykrzyknikiem to nieprawda?

[nikestylex7]

wykrzyknik z if empty czyli jeśli jest puste zrobi jeśli nie jest puste (if !empty) czyli jak chcesz to zastosować to tylko do czegoś co nie ma być puste czyli możesz użyć to do naszego przykładu np:

[PHP] pobierz, plaintext 
if(!empty($_POST['nickname']){
echo"Jest Nick";
}
[PHP] pobierz, plaintext 

[StadiPL]

Zauważyłem że to jednak nie ja mam złe myślenie tylko coś jest nie tak z moją funkcją preg_match. Wygląda ona następująco:

[PHP] pobierz, plaintext 
<?php
 if(!preg_match('/^[a-zA-Z0-9]$/D', $_POST['nickname'])) {echo "nie ok";}
else { echo "GIT"; }
?>
[PHP] pobierz, plaintext 

Gdy nickname to "a" wyświetla "GIT", gdy nickname to np. "stadi" wyświetla "nie ok". Nie rozumiem dlaczego.. część /^[a-zA-Z0-9]$/D wykonałem dokładnie tak jak na internecie. Powinna ona sprawić że znaki użyte w nickname muszą być z przedziału od a do z (duże lub małe) lub/oraz od 0 do 9. Dlaczego zatem coś się psuje i działa dobrze gdy nickname jest jednoznakowy (pasujący do klucza), a nie działa gdy nickname składa się z wielu znaków (także pasujących do podanego klucza)?
EDIT: to samo tyczy sie polskich znakow

Ten post edytował StadiPL 12.05.2013, 01:37:56

[nikestylex7]

[PHP] pobierz, plaintext 
if(strlen($_POST['nickname']) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $_POST['nickname'])) {
echp"klapa";
}else{
echo"git";
}
[PHP] pobierz, plaintext 

sprawdz to od razu sprawdza czy nick ma wiecej niż 3 znaki

Ten post edytował nikestylex7 12.05.2013, 01:37:08

[StadiPL]

Jako że w tym temacie wszystko już jest ogarnięte, to wrzucam efekt końcowy ;] Może komuś się przyda do przeanalizowania
register.php

[PHP] pobierz, plaintext 
Rejestruj się:
<form action="register_function.php" method="post">
nick: <input type="text" name="nickname" />(Powyżej 3 zaków. Dozwolone: a-z A-Z 0-9. Bez polskich liter)<br />
haslo: <input type="password" name="password" />(Powyżej 3 zaków. Dozwolone: a-z A-Z 0-9. Bez polskich liter)<br />
powtorz haslo: <input type="password" name="password_r" /><br />
email: <input type="text" name="email" /><br />
powtorz email: <input type="text" name="email_r" /><br />
<input type="submit" value="Rejestruj" />
</form>
[PHP] pobierz, plaintext 

register_function.php

[PHP] pobierz, plaintext 
<?php
$no_nickname="Podanie nazwy uzytkownika jest wymagane";
$no_password="Podanie hasla i jego powtorzenie jest wymagane";
$different_password="Podane hasla roznia sie'";
$no_email="Podanie e-maila i jego powtorzenie jest wymagane";
$different_email="Podane e-maile roznia sie";
$bad_nickname="Niedozwolona nazwa uzytkownika";
$bad_password="Niedozwolone haslo";
$bad_email="Nieprawidłowy adres email";
 
if(empty($_POST['nickname'])){$no_nickname;}
else if(empty($_POST['password']) || empty($_POST['password_r'])){echo $no_password;}
else if($_POST['password'] != $_POST['password_r']){echo $different_password;}
else if(empty($_POST['email']) || empty($_POST['email_r'])){echo $no_email;}
else if($_POST['email'] != $_POST['email_r']){echo $different_email;}
else if(strlen($_POST['nickname']) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $_POST['nickname'])) {echo $bad_nickname;}
else if(strlen($_POST['password']) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $_POST['password'])) {echo $bad_password;}
else if(!filter_var($_POST['email'],FILTER_VALIDATE_EMAIL)){echo $bad_email;}
else{echo"git";}
?>
[PHP] pobierz, plaintext 

[Szymciosek]

Nie robisz żadnych wcięć tylko tak wszystko w jednej linii zawsze?

[PHP] pobierz, plaintext 
<?php
$no_nickname="Podanie nazwy uzytkownika jest wymagane";
$no_password="Podanie hasla i jego powtorzenie jest wymagane";
$different_password="Podane hasla roznia sie'";
$no_email="Podanie e-maila i jego powtorzenie jest wymagane";
$different_email="Podane e-maile roznia sie";
$bad_nickname="Niedozwolona nazwa uzytkownika";
$bad_password="Niedozwolone haslo";
$bad_email="Nieprawidłowy adres email";
 
if(empty($_POST['nickname']))
{
    $no_nickname;
}
else if(empty($_POST['password']) || empty($_POST['password_r']))
{
    echo $no_password;
}
else if($_POST['password'] != $_POST['password_r'])
{
    echo $different_password;
}
else if(empty($_POST['email']) || empty($_POST['email_r']))
{
    echo $no_email;
}
else if($_POST['email'] != $_POST['email_r'])
{
    echo $different_email;
}
else if(strlen($_POST['nickname']) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $_POST['nickname']))
{
    echo $bad_nickname;
}
else if(strlen($_POST['password']) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $_POST['password']))
{
    echo $bad_password;
}
else if(!filter_var($_POST['email'],FILTER_VALIDATE_EMAIL))
{
    echo $bad_email;
}
else
{
    echo"git";
}
?>
[PHP] pobierz, plaintext 

Co do zmiennych zawierających tekst błędu, to trzymałbym to osobiście w jakiejś osobnej klasie i umieściłbym je tam jako stałe.

Dodatkowo z tego co mi wiadomo nie używa się już <br /> na rzecz css.

Teraz powyższy kod może Ci się wydawać trochę chaotyczny, ale w przyszłości na pewno nie poprzestaniesz tylko na echo.

Ten post edytował Szymciosek 12.05.2013, 08:56:31

[session]

Widze, że wszystko już zostało wyjaśnione, ale odnosząc się jeszcze do pierwszego postu:

[PHP] pobierz, plaintext 
 if ($_POST['password'] == $_POST['password_r']) {
if ($_POST['email'] == $_POST['email_r']) {
if(!empty($_POST['nickname'])){
echo "GIT";
} else {echo "Wypełnij wszystkie wymagane pola"; }
} else {echo "Podane adresy email nie sa identyczne"; }
} else {echo "Podane hasla nie sa identyczne"; }
?>
[PHP] pobierz, plaintext 

W celu zachowania spójności można by to też w taki sposób zapisać, chociaż poprawność tego kodu jest dyskusyjna (to tylko dla ścisłości). Stosowanie jak w powyższym przykładzie elseif nie jest też do końca dobre dla użytkownika, ponieważ jeśli nie wypełni pierwszego pola, źle przepisze hasła, użyje niedozwolonych znaków w haśle i w nicku, zapomni o domenie w adresie email, będzie musiał wypełniać formularz 6 razy za każdym razem przepisując dwa razy hasła, jak nie cały formularz to krew go zaleje i to przy założeniu, że nie pomyli się gdzieś w trakcie, więc prędzej się nie zarejestruje. Błędy należy wyświetlać wszystkie jednocześnie, więc konstrukcja: if {} if{} do tego należy dodać $error=false; i zmieniać jej wartość na true jeśli wystąpi bład, a na końcu:

[PHP] pobierz, plaintext 
if(!$error){echo 'Jest git';}
[PHP] pobierz, plaintext 

Ten post edytował session 12.05.2013, 09:56:14