Wskazówki do klasy Opcje

[GameMaker]

Wczoraj przysiadłem do OOP i napisałem klasę , która sprawdza czy user jest zalogowany i nadaje mu poziomy dostępu.

Proszę o wskazówki :

[PHP] pobierz, plaintext 
 
 
 
 
class User {
global $db;
public $logged_in = null;
public $user_level = null;
 
 
function __construct()
      {
 
		  $this->startSession();
      }
 
 
	  private function loginCheck()
	  {
          if (isset($_SESSION['id']) && $_SESSION['id'] > 0) {
 
 
              return true;
          } else {
              return false;
          }  
	  }
 
	  private function levelCheck()
	  {
	  $id = intval($_SESSION['id']);
	  $q = $db->query("Select * from mpa_users where id='$id'");
 
	  if($db->num_rows($q) == 1)
	  {
	  $row = $db->fetch_array($q);
 
	  if($row['admin'] == 1)
	  {
	  return 7;
	  }elseif($row['admin'] == 0 && $row['premium'] == 1 && $row['premium_time'] > time())
	  {
 
	  return 2;
 
	  }elseif($row['admin'] == 0 && $row['premium'] == 1 && $row['premium_time'] < time())
	  {
	  $db->query("Update mpa_users set premium='0' where id='$id'");
 
	  return 1;
 
	  }elseif($row['admin'] == 0 && $row['premium'] == 0)
	  {
 
	  return 1;
	  }
 
	  }
	  else
	  {
	  session_destroy();
 
	  }
 
	  }
 
 
private function startSession()
      {
		session_start();
		$this->logged_in = $this->loginCheck();
		if($this->logged_in)
		{
		$this->user_level = $this->levelCheck();
		}
		else
		{
		$this->user_level = 0;
		}
 
      }
 
 
 
 
 
 
 
 
}
 
[PHP] pobierz, plaintext 

Ten post edytował GameMaker 9.10.2011, 07:56:42

[IceManSpy]

1. Nie ma już zmiennych global.
2. Wszystkie prywatne funkcje, jak chcesz to w kodzie wywołać?

[GameMaker]

deklaruje klase , a potem zmienna logged_in i user_level sprawdzam dostep do aplikacji //

z tego co wyczytałem global daje dostep innej klasie zadeklarowanej w zmiennej (wczesniej) do danej klasy

Ten post edytował GameMaker 9.10.2011, 09:47:24

[Spawnm]

Wszystko źle (IMG:style_emoticons/default/smile.gif)
$db przekazujesz przez dependency injection a nie global (dziwne że ci błędu nie wywaliło).
Czemu klasa autoryzacji zajmuje się zadaniami klasy Session?
Daj

[PHP] pobierz, plaintext 
__construct(Db $db, Session $session)
{
  $this -> _db = $db;
  $this -> _session = $session;
} 
[PHP] pobierz, plaintext 

[GameMaker]

Na razie sesje zostawie w tej klasie , po poprawkach :

[PHP] pobierz, plaintext 
 
class User {
 
public $logged_in = null;
public $user_level = null;
 
 
function __construct($db)
      {
		  $this->_db = $db;
		  $this->startSession();
      }
 
 
	  private function loginCheck()
	  {
          if (isset($_SESSION['id']) && $_SESSION['id'] > 0) {
 
 
              return true;
          } else {
              return false;
          }  
	  }
 
	  private function levelCheck()
	  {
	  $id = intval($_SESSION['id']);
	  $q = $this->_db->query("Select * from mpa_users where id='$id'");
 
	  if($this->_db->num_rows($q) == 1)
	  {
	  $row = $this->_db->fetch_array($q);
 
	  if($row['admin'] == 1)
	  {
	  return 7;
	  }elseif($row['admin'] == 0 && $row['premium'] == 1 && $row['premium_time'] > time())
	  {
 
	  return 2;
 
	  }elseif($row['admin'] == 0 && $row['premium'] == 1 && $row['premium_time'] < time())
	  {
	 $this->_db->query("Update mpa_users set premium='0' where id='$id'");
 
	  return 1;
 
	  }elseif($row['admin'] == 0 && $row['premium'] == 0)
	  {
 
	  return 1;
	  }
 
	  }
	  else
	  {
	  session_destroy();
 
	  }
 
	  }
 
 
private function startSession()
      {
		session_start();
		$this->logged_in = $this->loginCheck();
		if($this->logged_in)
		{
		$this->user_level = $this->levelCheck();
		}
		else
		{
		$this->user_level = 0;
		}
 
      }
 
 
 
 
 
 
 
 
}
[PHP] pobierz, plaintext