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

[rudald]

poświęciłem temu kilka dni by przez wszystko przebrnąć więc:

[PHP] pobierz, plaintext 
  <? 
  $mail = $_POST['mail'];
 
  if (preg_match('[[\w\S\.\-]+\@[\w\S\.\-]+[\.\w{2,4}]]', $_POST['$mail']))
  {
  echo "Konto zostalo wlasciwie utworzone";
  }
  else
  {
  echo "BŁĄD";
  echo "<br>";
  echo '<a href="index.php">jeszcze raz</a>';
  }
  ?>
 
[PHP] pobierz, plaintext 

w - cyfry litery podkreślenie
S - wszystko bez spacji
. - może być użyta kropka
- może być użyty minus
+ - oznacza, że mail ma mieć co najmniej 1 znak
później powinna być @
analogicznie co do portalu z mailem i później w{2,4} czyli litery lub cyfry posiadające co najmniej 2 znaki a co najwyżej 4
napisałem to, by prosić o potwierdzenie mojego toku rozumowania

jednak gdzieś popełniłem błąd, bo wpisując cokolwiek ciągle wyskakuje 'błąd' - Proszę o pomoc : )