Wątek: System logowania i rejestracji - ocena

Napisałem prosty system logowania i rejestracji i chciałby się was spytać co w nim jest ewentualnie źle i co można poprawić:

rejestracja:

Kod:

<?
function reg_form&#40;&#41; 
&#123;?> 
	<div id='reg_form'> 
			<form name='register' method='POST'> 
				Nick &#58; <input name='nick' type='text' maxlength='20' style='margin-left&#58;60px;' onfocus="Help&#40;'nick'&#41;" onblur="Out&#40;&#41;"><br> 
				Hasło &#58; <input name='pass1' type='password' maxlength='20' style='margin-left&#58;55px; margin-top&#58;5px;' onfocus="Help&#40;'pass1'&#41;" onblur="Out&#40;&#41;"><br> 
				Powtórz Hasło &#58; <input name='pass2' type='password' maxlength='20' style='margin-top&#58;5px;' onfocus="Help&#40;'pass2'&#41;" onblur="Out&#40;&#41;"><br> 
				<input name='reg_send' type='submit' value='Rejestracja' style='margin-left&#58;80px; margin-top&#58;10px;'> 
			</form> 
		</div> 
		<div id='help'><script>document.getElementById&#40;'help'&#41;.innerHTML = 'Nie podawaj nikomu swojego hasła!';</script> 
		</div> 
<?&#125; 
?> 

<?
function register&#40;&#41; 
&#123; 
	if&#40;!isset&#40;$_POST&#91;'reg_send'&#93;&#41;&#41; 
	&#123; 
		reg_form&#40;&#41;; 
	&#125;else if&#40;isset&#40;$_POST&#91;'reg_send'&#93;&#41;&#41; 
	&#123; 
		if&#40;&#40;empty&#40;$_POST&#91;'nick'&#93;&#41;&#41; || &#40;empty&#40;$_POST&#91;'pass1'&#93;&#41;&#41; || &#40;empty&#40;$_POST&#91;'pass2'&#93;&#41;&#41;&#41; 
		&#123; 
			echo 'Nie wypełniłeż&#40;aż&#41; wszystkich pól<br>'; 
			reg_form&#40;&#41;; 
		&#125;else
		&#123; 
			$pass = $_POST&#91;'pass1'&#93;; 
			$pass2 = $_POST&#91;'pass2'&#93;; 
			if&#40;$pass != $pass2&#41; 
			&#123; 
				echo 'Hasła się nie zgadzają!<br>'; 
				reg_form&#40;&#41;; 
			&#125;else
			&#123; 
				$nick = $_POST&#91;'nick'&#93;; 
				if&#40;&#40;ereg&#40;"&#91;&#91;&#58;punct&#58;&#93;&#93;", $nick&#41;&#41; || &#40;ereg&#40;"&#91;&#91;&#58;space&#58;&#93;&#93;", $nick&#41;&#41; || &#40;ereg&#40;"&#91;&#91;&#58;punct&#58;&#93;&#93;", $pass&#41;&#41; || &#40;ereg&#40;"&#91;&#91;&#58;space&#58;&#93;&#93;", $pass&#41;&#41;&#41; 
				&#123; 
					echo 'Nick lub Hasło zawiera niepoprawne znaki!<br>'; 
					reg_form&#40;&#41;; 
				&#125;else
				&#123; 
					$file = fopen&#40;"users.txt", "a"&#41;; 
					$dane = "$nick|$pass"; 
					fputs&#40;$file, "$dane\n"&#41;; 
					fclose&#40;$file&#41;; 
					echo '<h4>Rejestracja przebiegła pomyżlnie. Możesz przejżć teraz do logowania.</h4>'; 
				&#125; 
			&#125; 
		&#125; 
	&#125; 
&#125; 
?>

Logowanie:

Kod:

<?
function log_form&#40;&#41; 
&#123;?> 
	<div id='log_form'> 
			<form name='login' method='POST'> 
				Nick &#58; <input name='nick' type='text' maxlength='20' style='margin-left&#58;60px;' onfocus="Help&#40;'nick'&#41;" onblur="Out&#40;&#41;"><br> 
				Hasło &#58; <input name='pass' type='password' maxlength='20' style='margin-left&#58;55px; margin-top&#58;5px;' onfocus="Help&#40;'pass1'&#41;" onblur="Out&#40;&#41;"><br> 
				<input name='log_send' type='submit' value='Logowanie' style='margin-left&#58;80px; margin-top&#58;10px;'> 
			</form> 
		</div> 
		<div id='help'><script>document.getElementById&#40;'help'&#41;.innerHTML = 'Nie podawaj nikomu swojego hasła!';</script> 
		</div> 
<?&#125; 
?> 

<?
function logowanie&#40;&#41; 
&#123; 
	if&#40;!isset&#40;$_POST&#91;'log_send'&#93;&#41;&#41; 
	&#123; 
		log_form&#40;&#41;; 
	&#125;else
	&#123; 
		if&#40;&#40;empty&#40;$_POST&#91;'nick'&#93;&#41;&#41; || &#40;empty&#40;$_POST&#91;'pass'&#93;&#41;&#41;&#41; 
		&#123; 
			echo 'Nie uzupełniłeż&#40;aż&#41; wszystkich pól!'; 
			log_form&#40;&#41;; 
		&#125;else
		&#123; 
			$plik = "users.txt"; 
			$file = fopen&#40;$plik,"r"&#41;; 
			$zawartosc = fread&#40;$file, filesize&#40;$plik&#41;&#41;; 
			$i = 0; 
			while&#40;!feof&#40;$file&#41;&#41; 
			&#123; 
				$dane = fgets&#40;$file, 4096&#41;; 
				$dane = explode&#40;"|", $dane&#41;; 
				if&#40;&#40;$_POST&#91;'nick'&#93; == $dane&#91;0&#93;&#41; AND &#40;$_POST&#91;'pass'&#93; == $dane&#91;1&#93;&#41;&#41; 
				&#123; 
					$_SESSION&#91;'nick'&#93; = $_POST&#91;'nick'&#93;; 
					$_SESSION&#91;'log'&#93; = true; 
					echo 'Zostałeż&#40;aż&#41; pomyżlnie zalogowany&#40;a&#41;.'; 
				&#125; else &#123; 
					$_SESSION&#91;'nick'&#93; = ''; 
					$_SESSION&#91;'log'&#93; = false; 
					echo 'Nie zostałeż&#40;aż&#41; zalogowany&#40;a&#41;.'; 
					&#125; 
				$i++; 
			&#125; 
			fclose&#40;$file&#41;; 
		&#125; 
	&#125; 
&#125; 
?>

Wiem, że przydało by się jakież hashowanie hasła ale tym chcę się zająć na końcu.
Czekam na wasze podpowiedzi.

Userów rejestrujesz do pliku txt?oO
I nie sprawdzasz czy dany user już istnieje.

Tu jest jeszcze wersja na plikach, którą chcę przerobić na bazę danych.
Najbardziej mi chodzi o zabezpieczenia przed niepowołanymi działaniami.

morawcik89, ja dam tylko jedna prosta rade: Zrob to porzadnie bo jest bardzo niewydajne, nieprzejrzyste, denerwujace do analizowania

czy jest zabezpieczony skrypt? jak na moje oko to niebardzo

Nie lepiej odrazu na bazie danych robić? konwersja z txt na db taka łatwa nie jest...

Najważniejsze jest to, być filtrował dane wysyłane w polach inputowych - to b. duży błąd.