[PHP]zmienna w klasie się gubi Opcje

[BugsBunny]

oto kawałek klasy:

[PHP] pobierz, plaintext 
<?php
class login extends database
{
	private $timeout;
 
	private $starttime;
 
	private $rank;
 
	private $user;
 
	private $login = false;
 
	function __construct()
	{
		require('./config.php');
		$this->timeout = $cfg['page']['SessionDuration'];
		$this->rank = 0;
 
	}
 
	function logIn($user1,$md5pass)
	{
		$this->connect();
		$Validate = $this->select('SELECT id,rank FROM user WHERE login=''.$user1.'' and pass=''.$md5pass.'' LIMIT 1;' );
 
		  if($Validate)
		{
		   $_SESSION['id'] = md5(microtime);
			$tmptime = time();
			  $this->insert('INSERT INTO session(id,start,user,rank) VALUES (''.$_SESSION['id'].'',''.$tmptime.'','.$Validate[0]['id'].','.$Validate[0]['rank'].' )');
			$this->starttime = $tmptime;			
			$this->rank =  $Validate[0]['rank'];
			$this->login = true;
			$this->user = $user1;
 
			return true;
		}
		else
			return false;
 
	}
 
	function logOut()
	{
	   $this -> remove('DELETE FROM session WHERE id =''.$_SESSION['id'].'' LIMIT 1;');
		session_destroy();
	   $this->login = false;
	}
 
	function isTimeOut()
	{		  
		if($this->starttime+$this->timeout < time())
		{	
 
			$this->logOut();
			return true;
		}
		else
		{
			$this -> update('UPDATE  session SET start = ''.time().'' WHERE id =''.$_SESSION['id'].'' LIMIT 1;');
			$this->starttime = time() ;
 
			return false;
		}
 
	}
?>
[PHP] pobierz, plaintext 

chodzi o to, że w funkcji logIn wartość $this->starttime jest prawidłowa tzn przyjmuje time();

natomiast już w funkcji isTimeOut() nic się nie wyświetla tak jakby zmienna nie istniała.

Ten post edytował BugsBunny 26.08.2008, 20:17:41

[webdice]

Proszę zmienić tytuł tematu, na taki który opisuje problem.

[nospor]

natomiast już w funkcji isTimeOut() nic się nie wyświetla

Nie wiem czy zauwazyles, ale w isTimeOut nie masz zadnego kodu odpowiedzialnego za wyswietlanie to i nic sie nie wyswietla.

pozatyym:
$this->starttime+$this->timeout < time()
skoto $starttime to time() to kazda warttosc dodana do time() bedzie wieksza od time(), wiec warunek nigdy sie nie spelni - pomijam fakt liczb ujemnych.

Moze napisz po ludzku co to ma robic, jak to uzywasz a nie strzelasz tekstami ktore tylko ty rozumiesz

lektura obowiązkowa:
http://rtfm.killfile.pl/

[BugsBunny]

Nie wiem czy zauwazyles, ale w isTimeOut nie masz zadnego kodu odpowiedzialnego za wyswietlanie to i nic sie nie wyswietla.

Nigdzie nie ma kodu odpowiedzialnego za wyświetlanie. DODAJE SOBIE GO w celu sprawdzenia

[PHP] pobierz, plaintext 
<?php
echo $this->starttime;
die();
?>
[PHP] pobierz, plaintext 

filozofia jest taka, że po zalogowaniu mamy czas ostniego działania w zmiennej starttime i w bazie danych.
IsTimeOut sprawdza czy tamten czas plus zdefiniowany TimeOut nie przekroczył aktualnego czasu, jeżeli tak wylogowuje i wyświetlam stosowny komunikat, jeżeli nie ustawiam starttime na aktualny czas.

Ale chyba nie ma to nic wspólnego z tematem bo jeżeli w LogIn wklepie podany wyżej kod to mi wyśiwetli liczbę, w isTimeOut już nie(PUSTO)

Ten post edytował BugsBunny 26.08.2008, 20:21:59

[nospor]

rety... co rozumiesz pod pojęciem

jak to uzywasz a nie strzelasz tekstami ktore tylko ty rozumiesz

Bo ja rozumiem, ze nalezy pokazac kod i powiedziec jak uzywasz tego obiektu. Bo dalej tego nie zrobiles.

Zgaduje wiec ze:
w jednym request masz LOGIN a nastepnym request masz sprawdzanie isTimeOut. No ale moj drogi, przeciez jesli tego gdziesz nie przechowasz, to w nastepnym request obiekt bedzie mial wyzerowane swoje wartosci. Musisz to gdziesz zapamietac, np. sesja - robisz tak?

[BugsBunny]

na poczatku to jest w index.php tworze obiekt

[PHP] pobierz, plaintext 
<?php
$ologin = new login();
?>
[PHP] pobierz, plaintext 

tutaj go obsługuje:

[PHP] pobierz, plaintext 
<?php 
 
 
if(!empty($_SESSION['id']))
{ 
	if($_GET['logout'] == 'true')
	{
 
	   $ologin->logOut();
	   $tpl->assign('login',$ologin->isLogIn());	
	}
	else
	{ 
		if($ologin->isTimeOut())
		{
 
			$tpl->assign('error','Automatycznie wylogowano - brak reakcji użytkownika');
			$tpl->assign('login',$ologin->isLogIn());
		}	
		else
		{
 
			$tpl->assign('login',$ologin->isLogIn());
		}	
	}
}
else
{
	$tpl->assign('login',false);
}
 
if(!empty($_POST['login']) && !empty($_POST['pass']))
{
	  $login = $_POST['login'];
	$pass  = md5($_POST['pass']);
 
	if($ologin->logIn($login,$pass))
	{
		$tpl->assign('rank', $ologin->getRank());
		$tpl->assign('login', $ologin->isLogIn());
	}
}
?>
[PHP] pobierz, plaintext 

[nospor]

no to robisz tak jak ci napisalem (czyli źle): w oddzielnych requestach sprawdzasz rzeczy z innych requestow. Tlumacze ci przeciez ze to sie samo z siebie nie zapamietuje. Musisz uzyc np. sesji do zapamietania starttime. No juz prosciej nie da sie tego napisac

[BugsBunny]

w sensie, że przy przeładowaniu strony obiekt jest tworzony jeszcze raz?

Mam w sesji trzymać obiekt ?

Może bardziej optymalne będzie pobieranie w konstrukutorze z bazy tych danych ( bo tam też je mam)

Ten post edytował BugsBunny 26.08.2008, 20:38:13

[nospor]

w sensie, że przy przeładowaniu strony obiekt jest tworzony jeszcze raz

bingo. Przy przeladowaniu strony wykonuje sie jeszcze raz kod, ktory napiszesz, z uwzględnieniem rzecz jasnych nowych warunkow jesli wystapią. Jesli wiec za kazdym razie w kodzie tworzysz nowy obiekt, to on za kazdym razem sie tworzy.

Mam w sesji trzymać obiekt

mozesz obiekt, a mozesz tylko wybrane dane. zalezy od ciebie

[BugsBunny]

Wielkie dzięki za pomoc. Jako, że w bazie trzymam wszystko związane z sesją to w konstruktorze pobieram dane i przywracam stan obiektu.