Как достать переменную из класса в var_dump (код внутри)?

Question

[Артур]

Есть реализованная вот такая фабрика:

$role = $_SESSION['role'];
	class admin {
		protected $addItem = true;
		protected $editItem = true;
		protected $deleteItem = true;
	}
	class moderator {
		protected $addItem = true;
		protected $editItem = true;
		protected $deleteItem = false;
	}
	class publicator {
		protected $addItem = true;
		protected $editItem = false;
		protected $deleteItem = false;
	}

	class Factory{
		public static function createUsers($role){
			if(class_exists($role))
				return new $role;
			else
				echo ("Роль".$role."отсутствует");
		}
	}

	$login = Factory::createUsers($role);
	var_dump($login);

Как в нем из var_dump($login) достать значение переменных соответствующего класса?
Или возможно напрямую из $login это сделать?

Comments

[Артур]

Алексей, Спасибо, не помог.

[Артур]

Алексей, доступа снаружи и не должно быть т.к. мы определяем $role и по нему обращаемся к свойствам класса

[Дмитрий Гординский]

А в чём вообще смысл пустых классов с полями к которым ни у кого нет доступа?

[ThunderCat]

Артур,

доступа снаружи и не должно быть т.к. мы определяем $role и по нему обращаемся к свойствам класса

Интересно, и ОТКУДА будет обращение к свойствам? Не вижу методов внутри этих классов, соответственно обращаться вы к ним не можете...

[Артур]

DmitriyGordinskiy, делал по этому принципу.

[Артур]

ThunderCat, тогда как это реализовать с методом?

[ThunderCat]

Артур, что реализовать? Не очень понятно какой практический смысл в этом коде. У вас он выглядит как посылка, которую Печкин имеет, но не отдает, потому что у вас докУментов нету.

делал по этому принципу.

Там просто приводится пример, абстрактный, тоже не для чего не годный. Как вариант - слелайте свойства пабликами, либо пишите геттер. Ну и еще вариант - объяснить что вы хотите сделать, и получить нормальный совет, а не гадание.

[Артур]

ThunderCat, задача состоит в следующем: на основе паттерна фабричного метода нужно реализовать роль пользователей свойствами классов и, в зависимости от этих свойств давать им доступ к определенной части кода. Надеюсь не слишком запутанно изложил мысль.

[Артур]

ThunderCat, Допустим, использую геттер вида:

public function hasAddItemPermission()
    {
        return $this->addItem;
    }

Как мне через него проверять, есть доступ для этой роли (класса) к свойству или нет?

[ThunderCat]

Артур, видимо вы пытаетесь реализовать ACL или RBAC, в любом случае, кроме того что это немного иначе работающие механизмы, в вашем случае самое простое и верное решение - сделать свойства публичными. И еще понять что протектед - это не про защиту от взлома и доступа, а инструмент инкапсуляции и использовать там где надо, а не там где дядя написал нашару.

[ThunderCat]

Артур, скорее вам надо будет создать метод проверки, а не геттер, что-то типа

public function checkPermission($action)
 {
         if(isset ($this->$action))
         {
           return $this->$action;
           }
          return false;
 }

[Артур]

ThunderCat, а если все таки геттером, как его проверять в дальнейшем?

[ThunderCat]

Артур, как обычно - получаете значение, далее if(значение){код выполняется}

[Артур]

ThunderCat, echo hasAddItemPermission();
Выдает ошибку

[Артур]

ThunderCat, Сделал вот так. Насколько это правильно?

$role = $_SESSION['role'];
	class admin {
		public $addItem;
		public static function hasAddItemPermission()
			{
				return $addItem = true;
			}
		public $editItem;
		public function hasEditItemPermission()
			{
				return $editItem = true;
			}
		public $deleteItem;
		public function hasDeleteItemPermission()
			{
				return $deleteItem = true;
			}
	}
	class moderator {
		public $addItem;
		public static function hasAddItemPermission()
			{
				return $addItem = false;
			}
		public $editItem;
		public function hasEditItemPermission()
			{
				return $editItem = true;
			}
		public $deleteItem;
		public function hasDeleteItemPermission()
			{
				return $deleteItem = true;
			}
	}
	class publicator {
		public $addItem;
		public static function hasAddItemPermission()
			{
				return $addItem = true;
			}
		public $editItem;
		public function hasEditItemPermission()
			{
				return $editItem = false;
			}
		public $deleteItem;
		public function hasDeleteItemPermission()
			{
				return $deleteItem = false;
			}
	}

	class Factory {
		public static function createUsers($role){
			if(class_exists($role))
				return new $role;
			else
				echo ("Роль".$role."отсутствует");
		}
	}

	$login = Factory::createUsers($role);
	//var_dump($login);
	
	$addPermission = $role::hasAddItemPermission();
	$editPermission = $role::hasEditItemPermission();
	$deletePermission = $role::hasDeleteItemPermission();

Значение из методов закидываю в переменную и с помощью переменной потом проверяю в условии. Но вопрос в том, насколько это правильно, с точки зрения паттерна фабричного метода.

Answer 1

[Дмитрий Гординский]

Добавь для каждого из поле геттер вида:

public function hasAddItemPermission()
    {
        return $this->addItem;
    }

И используй его для проверок.

А если прямо так критично получать из var_dump, то можно и по рофлить:

class moderator {
    protected $addItem = true;
    protected $editItem = true;
    protected $deleteItem = false;
}
  
$testRole = new moderator();
  
  
ob_start();
var_dump($testRole);
$vraDumpedRole = ob_get_clean();
    
preg_match_all('/bool\((true|false)\)/', $vraDumpedRole, $match);
    
list($addItem, $editItem, $deleteItem) = $match[1];
    
var_dump($addItem);
var_dump($editItem);
var_dump($deleteItem);

Comments

[Артур]

hasAddItemPermission()

Этот метод у нас возвращает булевое значение? Как его можно проверять в условии?
Не понимаю принцип работы

$this->addItem

[Дмитрий Гординский]

Артур, перед изучением паттернов неплохо бы ознакомится с синтаксисом языка:
PHP Objects

[Артур]

DmitriyGordinskiy Подскажете как использовать метод для проверок?

[Кирилл Несмеянов]

Проще было бы allows добавить:

interface PermissionInterface
{
    public function allow(string $permission): bool;
}

///
public function allow(string $permission): bool
{
    return \property_exists($this, $permission) && $this->$permission;
}

Answer 2

[ThunderCat]

Артур,

Но вопрос в том, насколько это правильно, с точки зрения паттерна фабричного метода.

Чет вы сильно путаете теплое с мягким... Внутренняя реализация классов созданных фабрикой НИКАК не влияет на "фабричность". Паттерн относится только к методу создания готовых объектов по определенным условиям. Какие это объекты вообще не суть важно. По сути они могут быть абсолютно разные, просто по логике чаще всего это однотипные объекты, отличающиеся в настройках или внутренней реализации (например все они реализуют один интерфейс, но внутри работают по разному).

Что касается вашего кода - как я уже сказал, для вашего случая(проверки доступа для текущей роли) достаточно задать метод проверки доступа. Как то так:

//$role = $_SESSION['role'];
$role = 'moderator';
  class Role{
     public function checkPermission($action)
     {
         if(isset ($this->$action))
         {
           return $this->$action;
           }
          return false;
   }
}

  class admin extends Role{
    protected $addItem = true;
    protected $editItem = true;
    protected $deleteItem = true;
  }
  class moderator  extends Role{
    protected $addItem = true;
    protected $editItem = true;
    protected $deleteItem = false;
  }
  class publicator  extends Role{
    protected $addItem = true;
    protected $editItem = false;
    protected $deleteItem = false;
  }

  class Factory{
    public static function createUsers($role) : Role{
      if(class_exists($role))
        return new $role;
      else
        echo ("Роль".$role."отсутствует");
    }
  }

  $login = Factory::createUsers($role);
  var_dump($login);
   
// try to use in code:
  if ($login->checkPermission('editItem')){
     echo 'lets edit!';
  };

Comments

[Артур]

Ругается на двоеточие
public static function createUsers($role) : Role{

[ThunderCat]

Артур, значит у вас пых до 7, можете убрать тайпхинтинг

[Артур]

ThunderCat, Понял, обновил до 7.2. Теперь работает