Prosze o ocene.., .. i poprawienie bledów Opcje

[Diablos]

[PHP] pobierz, plaintext 
<?
					   
include('config.php');
										   
session_cache_limiter(&#092;"must-revalidate\");
session_start();
 
 
 
/* LOGOWANIE */
 
  if (!isset($_SESSION['login']))
  {
   if (!isset($_POST['login']))
   {
   echo &#092;"<form action=\"index.php\" method=\"post\"></center>&nbsp;&nbsp;&nbsp;&nbsp;Login: <center><input name=\"login\" type=\"text\" size=\"15\"><br> </center>&nbsp;&nbsp;&nbsp;&nbsp;Hasło:<center> <input name=\"haslo\" type=\"password\" size=\"15\"> <br><input type=\"submit\" value=\"Zaloguj\"><br> Nie masz konta ? [<a href=\"index.php?module=register\">Zarejestruj sie</a>]
		</form>&#092;";
   }
   else {
	$polaczenie = @mysql_pconnect($host, $user, $pass);
 
  if (!$polaczenie) {
	echo &#092;"Nie udało się nawiązać połączenia z serwerem.\";
	exit;
   }
 
  mysql_select_db($name);
$haslo=$_POST['haslo'];
$zapytanie = 'SELECT * FROM ea_users WHERE xywka=''.addslashes($_POST['login']).'' AND haslo=''.$haslo.'' AND aktywny='1' ';
$wynik = mysql_query($zapytanie);
if(!$wynik) {
	echo mysql_error();
	exit;
}
if(mysql_num_rows($wynik)==0) {
	echo &#092;"<br>Podano błędne dane. <br>[<a href=\"index.php\">Spróbuj jeszcze raz</a>]\";
}
else {
	$login = mysql_fetch_array($wynik);
	$_SESSION['login'] = $login['xywka'];
}
  }
}
if(isset($_SESSION['login']))
  {
  switch($mod) {
  default:
 
   $form = mysql_query('SELECT * FROM ea_users WHERE xywka=\"' . $_SESSION[login] . '\"');
  while ($user = mysql_fetch_array($form))  {
  if ( $user[funkcja] == '1' OR $user[funkcja] == '2') {
 
   $q = mysql_query('SELECT id FROM ea_users WHERE xywka=\"' . $_SESSION[login] . '\"');
$row = mysql_fetch_array($q);
   // Gdy zalogowano
   echo &#092;"Jestes zalogowany jako <b><a href=\"index.php?module=view&ID=$row[id]\">\".$_SESSION['login'].\"</a></b>!\";
   // Wylogowywanie
   echo &#092;" <br><br>[<a href=\"index.php?module=admin\">Admin Page</a>]<br>
   [<a href=&#092;"index.php?module=useredit\">Edytuj Profil</a>]<br>
   [<a href=&#092;"index.php?mod=logout\">Wyloguj sie</a>]\";
   }
 
   else   {
   
	 echo &#092;"Jestes zalogowany jako <b><a href=\"index.php?module=view&ID=$row[id]\">\".$_SESSION['login'].\"</a></b>!\";
   // Wylogowywanie
   echo &#092;" [<a href=\"index.php?module=useredit\">Edytuj Profil</a>]<br>[<a href=\"index.php?mod=logout\">Wyloguj sie</a>]\";
   }	 }
   
   break;
 
/* WYLOGOWYWANIE */
  case logout:
 
  session_unset();
  session_destroy();
 
  echo &#092;"<b>Wylogowano pomyslnie !</b><br><a href=\"index.php\">Zaloguj się ponownie</a>\";
  break;
 
 }
}
 
echo &#092;"</center>\";
 
?>
[PHP] pobierz, plaintext 

Zalezy mi na poprawieniu błędów, chciałbym uzyc tego logowania w swoim Portalu. Chcę zeby było idealne, wiec zalezy mi na kazdej uwadze :roll2:

[crash]

Skąd się wzięło $mod? Zapomniałeś o $_GET? Nie podoba mi się przedstawianie ciągów jako stałych (w kluczach tablic i case'ach).

[Diablos]

Jakas podpowiedz ?

[crash]

Niewyraźnie napisałęm?

[PHP] pobierz, plaintext 
<?php
$mod = $_GET['mod'];
 
$_SESSION[login]; // źle
$_SESSION['login']; // dobrze
?>
[PHP] pobierz, plaintext 

[Diablos]

Nie o to chodzi

To juz poprawiłem

Chodziło mi o podpowiezd odnosnie case'ów.

[crash]

No to samo: masz case logout: zamiast case 'logout':

[Diablos]

Hm, tego to nawet nie wiedzialem ze sie bierze po case w ' .

[dr_bonzo]

Nie po 'case' tylko stringi (lancuchy znakow) otacza sie "" lub ''.
Bo w $_GET[ costam ] , costam jest stala (stringim bedzie 'costam') i glupie php wyrzuca NOtice'a (albo inny blad -- nie pamietam) ze stala costam nie jest zdefiniowana i przypisuje jej domyslna wartosc rowna 'costam'. Gdyby tego nie robilo i wyrzucalo powazniejszy blad ludzie by to zauwazali i poprawiali.

[Diablos]

No i spróbowałem sie zalogowac przy register_globals off, wywaliło komunikat:

Warning: Unknown(): Your script possibly relies on a session side-effect which existed until php 4.2.3. Please be advised that the session extension does not consider global variables as a source of data, unless register_globals is enabled. You can disable this functionality and this warning by setting session.bug_compat_42 or session.bug_compat_warn to off, respectively. in Unknown on line 0

Co mam z tym zrobic ? Poprawic cos w skrypcie ?

[crash]

You can disable this functionality and this warning by setting session.bug_compat_42 or session.bug_compat_warn to off, respectively

Czytaj ze zrozumieniem...

[Diablos]

No rozumiem, że moge zablokowac to ostrzezenie poprzez kolejne ustawienie na off.
Ale nie mowi mi to za wiele