Rejestracja i Logowanie, pomysły i rozwiązania Opcje

[rudald]

Rejestracja

[PHP] pobierz, plaintext 
<?
$name_user = $_POST['name_user']; 
$haslo = $_POST['haslo']; 
$haslo2 = $_POST['haslo2']; 
$mail = $_POST['mail']; 
$mail2 = $_POST['mail2']; 
 
if (($name_user<>'') and ($haslo<>'') and ($haslo2<>'') and ($mail<>'') 
and ($mail2<>''))
{
if (file_exists("./users/$name_user"))
{
echo "Ta nazwa użytkownika już istnieje!";
}
else
{
if (($haslo<>$haslo2) and ($haslo='') and ($name_user==$haslo))
{
echo '<br>';
echo 'Błędnie podany adres email lub hasło';
echo "Hasło nie może być takie same jak login";
}
else
{
if (($name_user<>$haslo) and ($haslo==$haslo2))
{
echo "Konto zostalo wlasciwie utworzone";
echo '<a href="logowanie.php">Loguj</a>';
$plik=fopen("./users/$name_user","w+");
      fclose($plik);
$plik2=fopen("./users/$name_user","a+");
       fwrite($plik2,$haslo);
fclose($plik2);
$plik3=fopen("./users/$name_user a","w+");
       fwrite($plik3," Email: ");
       fwrite($plik3,$mail2);
       fwrite($plik3,' ');
       fclose($plik3);
}
else
{
echo "<br>Hasło nie moze być takie samo jak login,!!!<br>";
echo "Hasła muszą być takie same.";
}
}
}
}
else
{
echo "Wypelnij wszystkie pola";
}
?>
[PHP] pobierz, plaintext 

Logowanie

[PHP] pobierz, plaintext 
<?
if (file_exists("./users/$name_user"))
{ $plik=fopen("./users/$name_user","r");
$dane = fread($plik,"100");
fclose($plik);
 
if ($haslo==$dane)
{
echo "jesteś zalogowany";
}
 
else
{
echo "Błędne hasło";
}
}
 
else
{
echo "Podany login nie istnieje";
}
 
?>
[PHP] pobierz, plaintext 

Chciałbym tylko, żeby podczas podawania maila program sprawdzał czy ciąg znaków zawiera '@'.
Prócz tego wszystko działa jak należy - ma ktoś jakieś propozycje, co zrobić, by zwiększyć bezpieczeństwo, a może po prostu zmodernizować? Ten zestaw rejestracji udało mi się stworzyć, na bazie podstawowych funkcji, ale jest wiele innych rozwiązań, tylko że jestem wstępniakiem, także jeśli ktoś odpisze to proszę o tłumaczenie szeroko pojęte ; ).

Ten post edytował rudald 9.08.2009, 11:02:40

[Pawel_W]

poczytaj podstawy wyrażeń regularnych na wikibooks, bo robisz całą masę problemów

1. Wszystko powinno być w nawiasie () a nie [], jeżeli chcesz to wyciągnąć
2. Długość ciągu określa się za [] a nie w nim np. [a-z]{1,2} a nie [a-z{1-2}]
3. Jeżeli chcesz aby ci to coś zwróciło to jako 3 parametr podajesz zmienną (IMG:style_emoticons/default/winksmiley.jpg)
4. $_POST['$mail]'] - poczytaj o różnicach między ' a ", taki kod:

[PHP] pobierz, plaintext 
$mail = '123';
echo '$mail';
echo '<br>';
echo "$mail";
[PHP] pobierz, plaintext 

wyświetli ci
$mail
123;


Poprawnie:

[PHP] pobierz, plaintext 
preg_match('/^([\w\S]+\@[\w\S\.]+[\.\w]{2,4})$/', $_POST[$mail], $zmienna)
print_r($zmienna);
[PHP] pobierz, plaintext 

Pozdrawiam, Pawel_W (IMG:style_emoticons/default/winksmiley.jpg)

EDIT:
sprawdziłem to wyrażenie i nie za bardzo weryfikuje email (IMG:style_emoticons/default/winksmiley.jpg) możesz wpisać email@.pl i wyświetli Ci że poprawny, dlatego lepiej zamiast wymyślać weż jakiś przykład z neta, chyba że to w celach treningowych ;P

Ten post edytował Pawel_W 16.08.2009, 08:04:14