Почему нельзя передавать свойство текущего объекта в замыкание?

Question

[Роман Башарин]

Приветствую коллеги. Помогите разобраться в тонкостях языка. Второй час не могу найти ничего даже близко похожего. Мануал с SO все уже вычитал)) Ошибка следующая:
PHP Parse error: syntax error, unexpected '->' (T_OBJECT_OPERATOR), expecting ')', Судя по всему в этом месте $this->db
Код:

class Orders {
	function __construct(Database $db) {
		$this->db = $db;
	}

	function createOrder(array $data) {
		$this->db->transaction(function($this->db) { // ошибка в этой строке
			$id1 = $this->db->insert('...');
			$id2 = $this->db->insert('...');
			$res = $this->db->update('...');
		});
	}
}
$orders = new Orders($db);


function createOrder(array $data) {
	$db = $this->db;
	$this->db->transaction(function($db) { // Так работает
	$this->db->transaction(function() use ($db) { // И так тоже
		$id1 = $this->db->insert('...');
		$id2 = $this->db->insert('...');
		$res = $this->db->update('...');
	});
}

Это нафиг не удобно вообще, в каждом методе прегонять в локальную переменную $db = $this->db, сейчас их 5, а когда будет 25, это же кабзда.

Comments

[Александр Х]

ай я вас умоляю, коллега... так таки прям "кабзда"? маленькое неудобство, не более... в первый раз чтоли? :)

Ну а если серьезно, в php5 - надо просто смириться... адаптируйся или умри...
в php7 - $this в замыканиях обретает новый смысл (и кстати, не все этому были рады, как оказалось)...

UPD: пардон, попутал... см. комментарии ниже...

[Станислав Б]

Александр Х: не видел ни одного, кто был бы не рад...

[Александр Х]

Станислав Б: да, пардон, попутал... не рады были отмене &$param при вызове функции... и php.ru подсказывает, что это было сильно раньше PHP7.... ладно, удаляюсь из дискуссии по причине обнаружения фактологической каши в воспоминаниях о том, "как это было".... :)

[Роман Башарин]

Александр Х: не в первый, но обычно это решалось, а тут не смог найти ответ самостоятельно)

Answer 1

[Станислав Б]

начиная с пхп 5.4 у тебя $this доступен внутри анонимной функции объявленной внутри класса.
не надо ничего туда передавать.

Comments

[Роман Башарин]

Точно, круто, это работает, но тогда есть доп. вопрос. Так выглядит метод из класса БД для транзакции. Мы должны передать в $actions (в этой переменной же анонимная функция?) (то что в секции try), насколько я понимаю в неё нужно передать объект БД (считай PDO), но мы ничего не передаём. Почему нет ошибки? У анонимной функции же указан аргумент?

public function transaction(Callable $actions)
{
	$this->pdo->beginTransaction();
	
	try {
		$actions($this);
	} catch (Exception $e) {
		$this->pdo->rollBack();
		return $e;
	}

	$this->pdo->commit();
	return true;
}

[miki131]

Роман Башарин: Параметр не нужно передавать, анонимная функция вызывается с параметром, тебе нужно его только объявить.

[Роман Башарин]

miki131: Вы об этой части? try { $actions($this); }
Раз это делается в контексте класс, то $this автоматически появляется при создании объекта и будет передан. Так что отдельно его не нужно объявлять. Об этом речь?

Answer 2

[miki131]

Ты пишешь параметр функции, а не передаешь его туда$this->db

Comments

[Роман Башарин]

Точно! Сутки без не проходят без следа)) Есть ещё небольше продолжение вопроса, но я его задам в параллельной ветке, там более актуально