Как работать с class?

Question

[John]

Всем привет!
У меня вопрос, есть класс:
DeleteTarget.php
pastebin.com/jkvV9LbY
Как мне правильно передать туда target id что бы все работало??
делаю вот так:
delete.php
pastebin.com/JD3n6vCy

только в самом классе немного меняю:

class DeleteTarget{

	//Server Keys
	private $access_key; // ставил public но ошибка не меняется
	private $secret_key;
	
	private $targetId;
	private $url 			= "https://vws.vuforia.com";
	private $requestPath 	= "/targets/";
	private $request;

но при вызове получаю ошибку:


delete.php <--- Сюда обращаюсь с помощью Ajax думал что в нем проблема но когда обращаюсь напрямую то все тоже

а когда в самом классе указываю таргет и ключи то все работает отлично!

{
result_code: "Success",
transaction_id: "7f0882c3d76b47bb9a87c0ddf339a83f"
}

Помогите решить проблему!
Спасибо!

Answer 1

[Максим Хан-Магомедов]

У вас в классе используется устаревшая форма конструктора (вместо __constructor()), которая к тому же не принимает ни одного аргумента.

В конструкотре удалять — грешновато. Поэтому для удаления лучше отдельный метод сделать, который потом вызывать (или после инициализации, или статичный метод).

Comments

[John]

а как тогда мне занести в класс ключи и таргет из вне??

[Максим Хан-Магомедов]

Вы можете их передать в констркутор (public function __cunstruct($access_key, $secret_key, $id) { $this->access_key = $access_key; $this->secret_key = $secret_key; $this->id = $id; } ) или использовать сеттеры (public function setId($id) { $this->id = $id; } )

[John]

ozgg: сделал так: pastebin.com/WBgkJXaX передаю как и раньше в delete.php но ошибка как была так и осталась =(

[Максим Хан-Магомедов]

John: потому что вы в констуркторе только присваиваете значения и ничего с этим дальше не делаете.
Логика такая:
В Delete.php создаётся экземпляр класса DeleteTarget с каким-то состоянием (установленными ключами и идентификатором). Чтобы произошло удаление, нужно с этим что-то сделать (позвать $del->execDeleteTarget(), например).

[Максим Хан-Магомедов]

John: плюс в новом конструкторе потерялась строка "$this->requestPath = $this->requestPath . $this->targetId;"

[John]

ozgg: спасибо! заработало! Теперь мне всегда делать конструктор в других классах? почему тогда vuforia.com сразу в своих SDK так не сделали??

[Максим Хан-Магомедов]

John: вроде как с седьмой версии PHP старая форма, когда название метода совпадает с названием класса, больше не будет работать как конструктор, поэтому да, конструктор в классе всегда должен называться __constructor (можете просто переименовать в остальных классах).
Почему не сделали — сложно сказать, я их код не изучал. Скорее всего «по историческим причинам» или в попытках обратной совместимости (тогда бы вместо public/protected/private в названиях свойств было бы просто var, а имена методов все были бы без области видимости).

[Максим Хан-Магомедов]

ozgg: то есть __construct(), конечно же.