[php]logowanie Opcje

[-Gość_Leon_*-]

Mam dziwny problem.
Napisałem skrypt logowania.
Loginy i hasła mam w pliku passwords.txt, w stylu login:hasło każde w osobnej linijce.
Wszystko łądnie ale jak sie loguje to zawsze wyskakuje że złe login lub hasło.
Podaję kod.
Wpisywanie danych:

[HTML] pobierz, plaintext 
<form name = "logowanie"
	action = "administracja/login.php"
	methot = "post"
>
Użytkownik:<br><input type="text" name="user"><br>
Hasło:<br><input type="password" name="haslo">
<br><br>
<input type="submit" value="Wejdź">
</form>
[HTML] pobierz, plaintext 

Weryfikacja:

[PHP] pobierz, plaintext 
<?php
function checkPass($user, $pass)
{
  if(!$fd = @fopen("passwords.txt", "r")) return 1;
  $result = 2;
  while (!feof ($fd)){
	$line = trim(fgets($fd));
	$arr = explode(":", $line);
	if(count($arr) < 2) continue;
 
	if($arr[0] != $user) continue;
 
	if($arr[1] == $pass){
	  $result = 0;
	}
	break;
  }
  fclose($fd);
  return $result;
}
if(!isSet($_POST["haslo"]) || !isSet($_POST["user"])){
  include('bad_login.html');
  exit();
}
$val = checkPass($_POST["user"], $_POST["haslo"]);
switch($val){
  case 0 : include('c5jkvc.php');break;
  case 1 : include('error_server.html');break;
  case 2 : include('bad_login.html');break;
  default : include('error_server.html');
}
?>
[PHP] pobierz, plaintext 

Co robie źle?

[strife]

Witam,

1. Trzymanie takiej bazy w pliku tekstowym nie ma sensu na dłuższą metę ponieważ:
a) Jest to wolniejsze rozwiązanie od baz danych, i posiada mniejsze możliwości.
b) Twój sposób przechowywania login'u uniemożliwia stworzenie użytkownika o nazwie user:tralala:bum
c) Podsumowując nie polecam logowania na plikach tekstowych, no ale nie wiem do czego Ty to wykorzystujesz :)

2. Nawiązując do poprzedniego punktu, proponuje abyś ( jeżeli musisz już to robić na plikach ) przechowywał dane w zserializowanej tablicy ( serialize" title="Zobacz w manualu php" target="_manual ), wtedy będziesz miał łatwiejszy do niej dostęp. Może pokusisz się nawet o stworzenie klasy obsługującej taką małą bazę :>

3. Przechodząc do sedna i odpowiadając już bezpośrednio na Twoje pytanie to:
a) Na początku zwróce uwagę na zapis, jest trochę dziwny, można taki kod zapisać dużo krócej o przedewszystkim czytelniej!
b) Robisz błędy, powinno być method, a nie methot.
c) Poprawiłem ten Twój kod trochę, myślę, że teraz lepiej wygląda :)

[PHP] pobierz, plaintext 
<form name="logowanie" action="?" method="post">
	Użytkownik: <br />
	<input type="text" name="user" /> <br />
	Hasło: <br />
	<input type="password" name="haslo"><br /><br />
	<input type="submit" value="Wejdź" />
</form>
 
<?php
function checkPass( $user, $pass )
{  
  $file = file( 'passwords.txt' );
 
  foreach( $file as $k )
  {
	  $eUser = explode(':', $k );
 
	  if ( trim( $eUser[ 0 ] ) == $user && trim( $eUser[ 1 ] ) == $pass )	
	  {
		  return TRUE;		
	  }
  }	
 
  return NULL;
 
}
echo '<pre>';
var_export( checkPass( $_POST["user"], $_POST["haslo"]  ) );
 
 
 
?>
[PHP] pobierz, plaintext 

Mój plik z użytkownikami

Kod

strife:demo
demo:demo
admin:demo

Pozdrawiam!

ps. Zobacz może któryś z temat w mojej sygnaturce Ci pomorze ;)