[php] skrypt rejestracji i logowania Opcje

[scorpio]

Witam

Na poczatku powiem ze nie jestem totalnie zielony w php :/
Napisałem skrypt rejstracji i logowań. składajaca sie z :
-regist.php ( rejestracja uzytkowników)
-login.php ( logowanie)
-site.html ( strona głowna)

zamieszcze te skrypty

regist.php

[PHP] pobierz, plaintext 
<?php
 
	switch($_GET['akcja'])
	{	default:
 
 
echo("
<br /><br /><b>REJESTRACJA</b><hr width=\"100%\"><br />
<form action=\"regist.php?akcja=zarejestruj\" method=\"post\">
nick: <input type=\"text\" name=\"nick\"><br />
haslo: <input type=\"text\" name=\"haslo\"><br />
email: <input type=\"text\" name=\"email\"><br />
<input type=\"submit\" value=\"zarejestruj\">
</form>
 
");
 
	break;
 
 
 
 
 
 
 
 
 
 
	case "zarejestruj":
 
setcookie("nick", time() + 1000000 * 6000000000000);
$nick = $_POST['nick'];
$haslo = $_POST['haslo'];
$email = $_POST['email'];
$plik = $nick.".txt";
	$fp = fopen($plik, "w");
	flock($fp, LOCK_EX);
	fwrite($fp, $nick."##".$haslo."##".$email."\n");
	flock($fp, LOCK_UN);
	fclose($fp);
 
echo("
konto o nazwie [".$_POST['nick']."] zostalo pomyslnie utworzone !<br /><r /><a href=\"login.php\">ZALOGUJ SIĘ</a>
");
 
	break;
 
 
	}
?>
[PHP] pobierz, plaintext 

login.php

[PHP] pobierz, plaintext 
<?php
 
	switch($_GET['akcja'])
	{	default:
 
 
echo("
 
 
<b>LOGOWANIE</b><hr width=\"100%\"><br />
<form action=\"login.php?akcja=zaloguj\" method=\"post\">
nick: <input type=\"text\" name=\"nick\"><br />
haslo: <input type=\"text\" name=\"haslo\"><br />
<input type=\"submit\" value=\"zaloguj\">
</form>
");
 
 
 
	break;
 
 
 
 
 
 
 
 
 
 
 
	case "zaloguj":
 
$nick = $_POST['nick'];
$haslo = $_POST['haslo'];
$plik = $nick.".txt";
 
	$fp = fopen($plik, "r");
	flock($fp, LOCK_SH);
	$zawartosc = fgets($fp);
	flock($fp, LOCK_UN);
	fclose($fp);
 
		$linia = explode("##", $zawartosc);
		$linia[1] = trim($linia[1]);
 
 
		if($nick==$linia[0] && $haslo==$linia[1])
		{
			echo("zalogowales sie pomyslnie!<br /><a href=\"www.google.pl\">przejdz do strony głownej !!!</a>");
		}
		else
		{
			echo("podales bledne dane! <br /><a href=\"login.php\">powrot</a><br /><br />");
		}
 
	break;
 
	}
?>
[PHP] pobierz, plaintext 

w sumie wszystko działą rejestruje uzytkowników i loguje. Ale nie tak jakby tego chciał :/

1. podczas rejestracji nie wymusza na uzytkowniku wypełnienia wszystkich pól :/
2. Nie sprawadza czy konto o podanej nazwie juz isnieje :/ i tworzy nowe o istniejacej juz nazwie :/

mam jeszcze 3 problem jak wymusić na uzytkowniku aby strone site.html mozna było ogladać dopiero po rejestracji??

Moze to dla was proste rzeczy ale ja mam z tym mały problem :/

POzdrawiam

[Pilsener]

1. echo używaj tak:

[PHP] pobierz, plaintext 
<?php
echo '<p class="paragraf">tekst/p>'
?>
[PHP] pobierz, plaintext 

- tak jest prościej, szybciej i czytelniej
2. Nie sprawdzasz danych odbieranych od użytkownika - poczytaj o walidacji danych
3. Nie szyfrujesz haseł
4. Wygodnie jest zapisywać każdego użytkownika w oddzielnym pliku?
5. Zapoznaj się z mechanizmami sesji
6. Nie zaszkodzi oddzielić kod html od php
7. Napisz to od nowa, opierając się na podobnych rozwiązaniach, choćby takim:
php.webhelp.pl/index.php?id=105&roz=php&ac=read