[HTML][PHP]Formularz rejestracji i logowania bez bazy danych Opcje

[badowl]

Witam mam pewien problem. To treść zadania z książki którą czytam:

"Napisz program, który zapyta użytkownika nazwę użytkownika oraz hasło.
(...)Nazwa użytkownika oraz nowe hasło powinny być przechowywane w pliku o nazwie
db.password. Następnie Utwórz program PGP pytający o nazwę użytkownika oraz hasło
i wpuszczający użytkownika tylko wtedy, gdy poda on prawidłowe dane.
Jeśli dane uwierzytelniające będą prawidłowe należy wyświetlić zawartość pliku datebook."

Plik index.php:

[PHP] pobierz, plaintext 
<?php
if (isset($_POST['submit'])){
	$login = $_POST['login'];
	$pass = $_POST['pass'];
	$pass1 = $_POST['pass1'];
	if($pass === $pass1){
		$filename = "$_SERVER[DOCUMENT_ROOT]/file/bd.password";
	$file = fopen($filename, 'r+');
	$nowy = $login . " " . $pass  . "\r\n";
	$filehandle = fopen($filename, 'ab');
	fwrite($filehandle, $nowy, strlen($nowy));
		print " rejstracja powiodła się! za 2 sekund zostaniesz automatycznie przekierowany do strony logowania.";
		header("Refresh: 2; URL=index2.php");
	}
	else{
		echo "Podane hasła muszą być identyczne!";
		header("Refresh: 2; URL=index.php");
	}
}
else {
?>
Jeśli nie masz konta utwórz je!
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method='post'>
login
<input type="text" name="login" /> <br />
hasło
<input type="password" name="pass" /> <br />
powtórz hasło
<input type="password" name="pass1" /> <br />
<input type="submit" name="submit" value="rejestruj" />
</form>
Jeśli masz już konto kliknij <a href="index2.php">tutaj</a> aby sie zalogować.
<?php
}
?>
[PHP] pobierz, plaintext 

Plik index2.php

[PHP] pobierz, plaintext 
<?php
if (isset($_POST['submit'])){
	$login = trim($_POST['login']);
	$pass = trim($_POST['pass']);
	$password = "$_SERVER[DOCUMENT_ROOT]/file/bd.password";
	$list = file($password);
	$count = 0;
	foreach($list as $key => $val){
		$log = explode(' ', $val);
		if(strcasecmp($login, $log[0]) == 0 && strcasecmp($pass, $log[1]) == 0){ //// PRAWDOPODOBNIE COŚ TU JEST ŹLE.
		$filename = "$_SERVER[DOCUMENT_ROOT]/file/datebook";
		$text = file_get_contents($filename);
		echo $text;
		$count++;
		}
	}
	if($count == 0){
		print "Użytkownik $login nie istnieje w bazie! Zarejstruj się!";
	}
}
else {
?>
Wpisz nazwę użytkownika i hasło.
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method='post'>
login
<input type="text" name="login" /> <br />
hasło
<input type="password" name="pass" /> <br />
<input type="submit" name="submit" value="loguj" />
</form>
<?php
}
?>
[PHP] pobierz, plaintext 

Zawartość pliku db.password

Kod

uzytkownik1 haslo1
uzytkownik2 haslo2

Zawartość pliku datebook:

Kod

Imie Nazwisko dd\mm\rrrr Adres
Imie2 Nazwisko2 dd\mm\rrrr Adres2

Wie ktoś na czym polega błąd?
Pozdrawiam (IMG:style_emoticons/default/smile.gif)

[mortus]

W pliku index2.php zamień albo linijkę 6 na:

[PHP] pobierz, plaintext 
$list = file($password, FILE_IGNORE_NEW_LINES);
[PHP] pobierz, plaintext 

albo linijkę 10 na:

[PHP] pobierz, plaintext 
if(strcasecmp($login, $log[0]) == 0 && strcasecmp($pass, trim($log[1])) == 0){
[PHP] pobierz, plaintext 

Mam nadzieję, że zdajesz sobie sprawę, że funkcja strcasecmp jest cese-insensitive, co oznacza, że TomeK = toMEk, przy porównywaniu tą funkcją.
Po pomyślnym zalogowaniu należałoby przerwać pętlę foreach (słówko break) i dopiero wtedy wykonywać pozostałe operacje.
Poza tym przy rejestracji nie sprawdzasz, czy użytkownik o podanym loginie i haśle już istnieje, czy nie istnieje i można go utworzyć (zatem możesz mieć użytkowników TomeK, ToMeK, tOMEk, itp. z różnymi hasłami, co będzie powodować problemy).