[php] logowanie

[spik]

Jestem bardzo początkujący w php i mam następujący problem. Próbowałem napisać własny skrypt do logowania który przekierowywuje każdy login do osobnej strony i wyszło mi coś takiego: [php]<?php

file_exists('user1.dat') or die('Brak pliku danych uzytkownikow');
$userData = file('user1.dat');

$auth=1; break;

if ($ud[0] == strtolower($_SERVER['PHP_AUTH_USER']) && rtrim($ud[1]) == strtolower($_SERVER['PHP_AUTH_PW']))
if($_SERVER['PHP_AUTH_USER']=='jakislogin')
include('jakisplik.php');
elseif($_SERVER['PHP_AUTH_USER']=='innylogin')
include('innastrona.htm');
elseif($_SERVER['PHP_AUTH_USER']=='jeszczeinnylogin')
include('costam.txt');
else
include('defaultowa_strona.php'); // gdy login nie jest taki jak zaden z powyzszych
include; break;

}

if (!$auth) {
header('WWW-Authenticate: Basic realm="Oceny"');
header('HTTP/1.0 401 Unauthorized');
echo 'To nie jest miejsce dla Ciebie! ';
die;
}

?>[/php] I ciągle mi wyskakują błedy. mugłby mi to ktoś poprawić? Byłbym bardzo wdzięczny.

[Rozek]

a możesz powiedzieć jakie błędy ?

[wixor]

właściwie, niektóre widać na pierwszyu rzut oka:
linia 6. :-> niepotrzebny break;
linia 17. -> cała linia niepotrzebna
Poza tym, piszesz nieuważnie: odczytujesz plik do zmiennej $userData (linia 4.) a w if-ie porównujesz ze zmienną $ud. poza tym, kod z linii 21-26 nigdy się nie wykona bo $auth zawsze jest równe "1" (ustawiasz to w linii 6.)
Prawdopodobnie , linia 6. jest w ogóle niepotrzebna,a "if(!$auth)" z linii 21 powinienes zamienić na "else".
Na koniec - popracuj nad wyglądem kodu - jest koszmarnie nieczytelny

[AnoniM]

Ja proponuje zainteresować się kodem do logowania z manuala. Napisany jest tak jak Twój, tylko... bez błędow .