Witam,

Jestem potocznie nazywanym w Internecie "noobem" , jednak przyszło mi pisać drobny skrypcik na potrzeby własne, prosty CMS jednak jest problem po krótce kod:

Login.class.php

[PHP] pobierz, plaintext 
<?php
 
class Login
{
	private static $instance;
	public $userStatus;
 
	private function __construct() { $this -> setAnonymous(); }
 
	public function setAnonymous ()
	{
		$this -> userStatus = 'anonymous';
	}
 
	public function setUser ()
	{
		$this -> userStatus = 'user';
	}
 
	public function setAdmin ()
	{
		$this -> userStatus = 'admin';
	}
 
	public function checkStatus ()
	{
		return $this -> userStatus;
	}
 
	// Singleton
 
	public static function singleton()
	{
		if (!isset(self::$instance))
		{
			$c = __CLASS__;
            self::$instance = new $c;
		}
		return self::$instance;
	}
 
	public function __clone()
	{
		trigger_error('no clone', E_USER_ERROR);
	}
}
 
 
?>
[PHP] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
<?php
 
ob_start();
 
require_once ( 'classes/MySql.class.php' );
require_once ( 'classes/Login.class.php' );
require_once ( 'classes/Templates.class.php' );
 
require_once ( 'functions/Configuration.php' );
require_once ( 'functions/Helpers.php' );
 
// Singletons
$MySQL = Mysql::singleton();
$Login = Login::singleton();
// ----- end
 
// set Configuration
$Data = niceURL();
// ----- end
 
// View Core
$coreTpl = new Template();
 
$coreArray = array (
 
	"menu" => Menu ($Login -> checkStatus(), $MySQL),
	"content" => Content ($Data, $MySQL, $Login),
 
	"url_template" => __url( $MySQL ).'templates',
	"footer" => 'copyright'
);
 
echo $coreTpl -> change ( $coreArray, 'Login.tpl', 'mtLogin' );
// ----- end
 
ob_end_flush();
 
?>
[PHP] pobierz, plaintext 

funkcja Contents()

[PHP] pobierz, plaintext 
function Content ( $Data, $MySQL, $Login )
{
	$userStatus = $Login -> checkStatus();
 
	if ( !empty ( $Data[0] ) )
	{
		$Answer = $MySQL -> qr ("SELECT * FROM epmenu WHERE name='$Data[0]'");
 
		if (($Answer['access'] == 'admin' || $Answer['access'] == 'user') and $userStatus == 'anonymous') 
		{
			$tmp = 'Access denied'; 
		}	
		elseif ( empty ( $Answer['id'] ) )
		{
			$tmp = '404';
		}
		else
		{
			switch ( $Answer['name'] )
			{
 
					// ANONYMOUS
				case 'Login':
					$tmp = cnLogin ( $MySQL, $Login );
					//if ( $tmp == 'Success' ) header ('Location: '.__url( $MySQL ).'index.php/Successfull/1');
					//if ( $tmp == 'Error' ) header ('Location: '.__url( $MySQL ).'index.php/Error/1');
					break;
 
				case 'Forgot':
					$tmp = '1';
					break;
 
 
					//  USER
				case 'Undone':
					$tmp = '1';
					break;
 
				case 'Progress':
					$tmp = '';
					break;
 
				case 'Done':
					$tmp = '1';
					break;
 
				case 'Docs':
					$tmp = '1';
					break;
 
				case 'Upload':
					$tmp = '';
					break;
 
				case 'Profile':
					$tmp = '1';
					break;
 
 
					// ADMINISTRATION
				case 'Administration':
					$tmp = '1';
					break;
 
 
					// SYTUATIONS
				case 'Successfull':
					$Success = new Template();
 
					switch ( $Data[1] )
					{
						case '1':
							$message = 'Congratulations! You have successful log in!';
							$back = __url( $MySQL ).'index.php/';
							$url = __url( $MySQL );
							break;
					}
 
					$tmp = $Success -> change ( array ( 'message' => $message, 'back' => $back, 'url' => $url ), 'Success.tpl', 'mtScreen' );
					break;
 
				case 'Error':
					$Error = new Template();
 
					switch ( $Data[1] )
					{
						case '1':
							$message = 'Login error: username or password is wrong!';
							$back = __url( $MySQL ).'index.php/Login';
							$url = __url( $MySQL );
							break;
					}
 
					$tmp = $Error -> change ( array ( 'message' => $message, 'back' => $back, 'url' => $url ), 'Error.tpl', 'mtScreen' );
					break;
			}
		}
 
		return $tmp;
	}
}
// ----- end
[PHP] pobierz, plaintext 

Funkcja cnLogin

[PHP] pobierz, plaintext 
function cnLogin ( $MySQL, $Login )
{
	$username = $_POST['username'];
	$password = $_POST['password'];
 
	if ($Login -> checkStatus != 'anonymous') header ('Location: '.__url( $MySQL ).'index.php/');
 
	if ( empty($username) and empty($password) )
	{
		$Formular = new Template();
		$tmp = $Formular -> change ( array('url' => __url( $MySQL )), 'Formular.tpl', 'mtLogin' );
	}
	else
	{
		$Number = $MySQL -> nr ("SELECT * FROM epusers WHERE username='$username'");
 
		if ( $Number > 0 )
		{
			$Answer = $MySQL -> qr ("SELECT * FROM epusers WHERE username='$username'");
 
			if ( $password == $Answer['password'] )
			{
				$tmp = 'Success';
				if ( $Answer['access'] == 'admin' ) $Login -> setAdmin();
				else $Login -> setUser();
			}
			else
			{
				$tmp = 'Error';
			}
		}
		else
		{
			$tmp = 'Error';
		}
		$tmp = $Login -> checkStatus();
	}
 
	return $tmp;
}
// ----- end
[PHP] pobierz, plaintext 

No właśnie i teraz jak wywołam

[PHP] pobierz, plaintext 
$Login -> checkStatus()
[PHP] pobierz, plaintext 

To w index.php wartość jest anonymous, jednak w funkcji cnLogin warość jest user (loguje się na użytkownika z takim statusem)

2 róże wartości singletona? Wiem że singleton to nie _global, no ale chyba nie powinno tak być?
Proszę oświećcie mnie

P.S. Prosiłbym o konkretne wypowiedzi, a nie w stylu "naucz się php", czy " po co używasz singletona on ssie"


Z góry dziękuje i pozdrawiam!

W cnLogin

na


Zapewne masz wyłączone pełne wyświetlanie błędów (a jest bardzo przydatne) to nie widziałeś tego błędu.

Dzięki za zainteresowanie, i wykrycie tego byka, ale niestety to nie zmienia faktu, że nadal są 2 różne wartości i w funkcji jestem zalogowany, ale już w index.php nie...

Sporo tego kodu. Mój pomysł to wklejenie do każdej twojej metody zmianiającej wartość userStatus tej funkcji ( http://pl.php.net/manual/pl/function.debug...t-backtrace.php )
Uczyń swój userStatus prywatny bo od zwracania jej wartości masz metode, a nóż zabezpieczysz żeby przypadkiem tej wartości nie zmienić.

Zrobiłem tak jak mi doradziłeś zmieniam wartość userStatus tylko w cnLogin, więc wstawiłem tam:

[PHP] pobierz, plaintext 
echo '<pre>';
debug_print_backtrace();
echo '</pre>';
[PHP] pobierz, plaintext 

efektem jest:

[PHP] pobierz, plaintext 
#0  cnLogin(Mysql Object ([] => localhost,[] => root,[] => root,[] => evoplus), Login Object ([] => anonymous)) called at [C:\Program Files (x86)\WebServ\httpd-users\Evo+\functions\Helpers.php:71]
#1  Content(Array ([0] => Login), Mysql Object ([] => localhost,[] => root,[] => root,[] => evoplus), Login Object ([] => anonymous)) called at [C:\Program Files (x86)\WebServ\httpd-users\Evo+\index.php:27]
[PHP] pobierz, plaintext