Forum PHP.pl

Witam.

Napisałem w proste logowanie, które nie działa (IMG:style_emoticons/default/biggrin.gif) Logować loguje się, lecz po przeładowaniu strony... tzn kliknę przycisk zaloguj ale mnie nie zaloguje, dopiero muszę dać np: zakładkę home i wtedy pojawi się, że jestem zalogowany. A wyloguj już całkowicie nie działa...
To jest plik logowania:

[PHP] pobierz, plaintext 
<?php
 
include("conn.php");	
 
if ($_SESSION['zalogowany'] != 1) {
 
	$login = $_POST['login'];
		$password = $_POST['password'];				
 
if ( isset($login) AND isset($password) )
			{
 
				$sql = mysql_query("SELECT * FROM users");
					$row = mysql_fetch_object($sql);		
 
						if($_SESSION['login']=='') $_SESSION['login']="nieznajomy";
 
							if($row->login == $login)
 
								if($row->password == $password){
 
		  							$_SESSION['login'] = $login;
 
										$_SESSION['zalogowany']=1;
			}}
 
	echo '
 
	<form action="index.php" method="post">	
 
		<span class="nazwa_lp">Login:</span> 
			<input type="text" id="login" name="login">
 
		<span class="nazwa_lp">Password:</span> 
			<input type="password" id="pass" name="password">			
 
		<input type="submit" value="zaloguj" class="zaloguj">		
 
	</form>
 
	'; }
 
	else {
 
	echo('<span class="nazwa_zalogowany">Witaj: &nbsp;&nbsp;&nbsp;' .$_SESSION['login'].'</span>     
 
     <a href="logout.php"><input type="button" name="wyloguj" value="Wyloguj" class="wyloguj"></a>'); 
 
 
	  };
 
?>
[PHP] pobierz, plaintext 

To jest plik logout.php

[PHP] pobierz, plaintext 
<?
		$_SESSION['zalogowany']=0;
 
			include("index.php");
?>
[PHP] pobierz, plaintext 

A to jest kod który mam w index.php

[PHP] pobierz, plaintext 
<div id="div_logowanie">	
 
			  <?
			  	if($_SESSION['zalogowany']!=1) include('login.php');
 
				else {      
 
     echo('<span class="nazwa_zalogowany">Witaj: &nbsp;&nbsp;&nbsp;' .$_SESSION['login'].'</span>     
 
     <a href="logout.php"><input type="button" name="wyloguj" value="Wyloguj" class="wyloguj"></a>');    
  					}
			  ?>		
 
			</div>
[PHP] pobierz, plaintext 

Z góry dziękuję za pomoc w naprawieniu tych skryptów. Pozdrawiam

Na początku każdego skryptu, w którym korzystasz z sesji musi

[PHP] pobierz, plaintext 
session_start();
[PHP] pobierz, plaintext 

Kod jest nieczytelny!

[PHP] pobierz, plaintext 
if ( isset($login) AND isset($password) )
[PHP] pobierz, plaintext 

1. zawartość tego ifa jest napisana chaotycznie, nic z tego nie zrozumiałem.
było sprawdzić to tak:

[PHP] pobierz, plaintext 
<?php
if $SERVER['REQUEST_METHOD'] == "POST" {
// definiowanie dobrehaslo i dobrylogin na podstawie bazy danych
 if ($password == $dobrehaslo && $login == $dobrylogin)
 {
 $_SESSION['login'] = $login;
 $_SESSION['zalogowany']=1;
 echo 'zalogowano';
 } else {
 echo 'zle dane';
 }
} else {
//logowajdło
?>
[PHP] pobierz, plaintext 

2. NIGDZIE nie inicjujesz sesji.
3. Plik do wylogowywania wersja poprawiona:

[PHP] pobierz, plaintext 
<?php
ob_start();
session_start();
session_destroy();
header("index.php");
ob_end_flush();
[PHP] pobierz, plaintext 

4. System sesji jest do niczego. Świetnie napisany i rozpisany system jest na Wikibooks. http://pl.wikibooks.org/wiki/PHP/Sesje
Dobrą praktyką jest pisanie komentarzy, żeby móc potem zrozumieć kod.

do wylogowania lepiej uzywac session_destroy();

Zapomniałem dodać, że oczywiście w pliku index.php na samym początku jest

[PHP] pobierz, plaintext 
<?php
	session_start();
 
	session_register($zalogowany, $login);
 
?>
[PHP] pobierz, plaintext 

Jeżeli w pliku logout.php daje session_destroy(); wyskakuje mi taki błąd:

Warning: session_start() [function.session-start]: Cannot send session cache limiter - headers already sent (output started at C:\Program Files\WebServ\httpd\cms\logout.php:7) in C:\Program Files\WebServ\httpd\cms\index.php on line 2

Logowanie cały czas mi działa... lecz nie znacie przyczyny, czemu muszę przeładować stronę aby pojawił się komunikat Witaj admin zamiast miejsc do wpisywania loginu i hasła?

Przed session_start() nic nie powinno się znajdować !

Oczywiście, że nic nie ma...

[PHP] pobierz, plaintext 
<?php
	session_start();
 
	session_register($zalogowany, $login);
 
?>
 
<html> 
<head>
[PHP] pobierz, plaintext 

tak wyglądają pierwsze linijki index.php

Nie nie , to albo podaj cały kod, który oddajesz do analizy, albo nikt nie będzie wróżył, co Ty tam jeszcze masz wcześniej w pliku index.php i dalej w innych plikach. Najlepiej gdybyś podał pełny listing wszystkich plików biorących udział w logowaniu + podstrony, o których wspominasz.

plik index.php

[PHP] pobierz, plaintext 
<?php 
	session_start();
 
	session_register($zalogowany, $login);
 
?>
 
<html> 
<head>
...
 
			<div id="div_logowanie">	
 
			  <?
			  	if($_SESSION['zalogowany']!=1) include('login.php');
 
				else {      
 
     echo('<span class="nazwa_zalogowany">Witaj: &nbsp;&nbsp;&nbsp;' .$_SESSION['login'].'</span>     
 
     <a href="logout.php"><input type="button" name="wyloguj" value="Wyloguj" class="wyloguj"></a>');    
  					}
			  ?>		
 
			</div>
 
		</div>
 
			<div id="logo"></div>		
...
 
[PHP] pobierz, plaintext 

plik login.php

[PHP] pobierz, plaintext 
<?php
 
include("conn.php");	
 
if ($_SESSION['zalogowany'] != 1) {
 
	$login = $_POST['login'];
		$password = $_POST['password'];				
 
if ( isset($login) AND isset($password) )
			{
 
				$sql = mysql_query("SELECT * FROM users");
					$row = mysql_fetch_object($sql);		
 
						if($_SESSION['login']=='') $_SESSION['login']="nieznajomy";
 
							if($row->login == $login)
 
								if($row->password == $password){
 
		  							$_SESSION['login'] = $login;
 
										$_SESSION['zalogowany']=1;
 
			}}
 
	echo '
 
	<form action="index.php" method="post">	
 
		<span class="nazwa_lp">Login:</span> 
			<input type="text" id="login" name="login">
 
		<span class="nazwa_lp">Password:</span> 
			<input type="password" id="pass" name="password">			
 
		<input type="submit" value="zaloguj" class="zaloguj">		
 
	</form>
 
	'; }
 
	else {
 
	echo('<span class="nazwa_zalogowany">Witaj: &nbsp;&nbsp;&nbsp;' .$_SESSION['login'].'</span>     
 
     <a href="logout.php"><input type="button" name="wyloguj" value="Wyloguj" class="wyloguj"></a>'); 
 
 
	  };
 
?>
[PHP] pobierz, plaintext 

plik logout.php

[PHP] pobierz, plaintext 
<?php
 
	$_SESSION['zalogowany']=0;
		include("index.php");
 
?>
[PHP] pobierz, plaintext 

Czytałem trochę i dowiedziałem się, że musze zrobić przeładowanie strony po zalogowaniu... myślałem, że robi się to automatycznie po przyciśnięciu przycisku submit (zaloguj) z formularza logowania... Niestety nie wiem dalej jak to zrobić.

Próbowałem wylogowywać się również za pomocą tego pliku o nazwie logout.php:

[PHP] pobierz, plaintext 
<?php		
	ob_start();
	session_start();
	session_destroy();
	ob_end_flush();
		include("index.php");	
?>
[PHP] pobierz, plaintext 

i wyskakuje mi błąd:

Warning: session_start() [function.session-start]: Cannot send session cache limiter - headers already sent (output started at C:\Program Files\WebServ\httpd\cms\logout.php:5) in C:\Program Files\WebServ\httpd\cms\index.php on line 2

Z góry dziękuje za pomoc.

w pliku login.php dodaj jedno else:

[PHP] pobierz, plaintext 
if($row->login == $login)
if($row->password == $password)
{
    $_SESSION['login'] = $login;
    1$_SESSION['zalogowany']=1;
}
else echo '<form action="index.php" method="post">
   <span class="nazwa_lp">Login:</span>';...
[PHP] pobierz, plaintext 

i bym zapomniał w pliku index.php powinno być cosik takiego:

[PHP] pobierz, plaintext 
 
if($_SESSION['zalogowany']!=1) include('login.php');
if($_SESSION['zalogowany']==1)
    echo '<span class="nazwa_zalogowany">Witaj:';
    ...
[PHP] pobierz, plaintext 

a session_destroy(); wyrzuca ci błąd bo powinieneś wykonać to przed wysłaniem jakichkolwiek danych do klienta. Najlepiej zaraz po session_start();
a najlepiej zarządzanie sesją wyżucić przed cały kod zarządzający treścią to unikniesz takich błędów (IMG:style_emoticons/default/winksmiley.jpg)

Ten post edytował Thorang Hoog 17.12.2009, 13:20:28