Как можно использовать Traits для установки переменных классов?

Question

[Андрей Корехов]

Есть идея вынести в трейт методы get и set для установки значений переменных в разных классах. Так сказать создать универсальный метод. Придумал следующую реализацию:

function setVar($name, $value) {
		$this -> $$name = $value;
	}
// вызов 
$key -> setVar('$count', $_GET['count']);

если делаю var_dump($var = $$name) внутри метода то вижу что, обращается к переменной $count b даже выводится ее значение
но непосредственно в скрипте и на странице вылетают notice

Notice: Undefined variable: $count in /media/andrey/_dde_data/projekts/php2/engine/CLSTraits.php on line 7

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

И второй вопрос как это можно обойти или реализовать? все таки сеттеры и геттеры чуть ли не самые однотипные методы в любом классе и логично именно их вынести в трейт

Comments

[Lander]

Может быть вам удобнее будет использовать магический метод __set

[Андрей Корехов]

Спасибо за идею, но удалось реализовать этот вариант

function setVar($name, $value) {
		$this -> $name = $value;
} 
$key -> setVar('count', $_GET['count']);

Всем спасибо

[Дмитрий]

Haotik, используйте стандартный магический метод, ещё можно сделать в нем
return $this;
И писать однострочники
Заведите свойство для багажника, и в него магическими свойствами складывайте и доставайте
Вот писал подобный трэйт
https://annimon.com/code/5186

[Евгений Ромашкан]

Дмитрий, Зачем они нужны? Ужас, мне жаль того кто будет ваш код читать.

[Дмитрий]

Евгений Ромашкан, кто? Трэйты?

[Евгений Ромашкан]

Дмитрий, магические методы для геттеров и сеттеров. Да и трейты тоже

[Дмитрий]

Евгений Ромашкан, это был пример, на коленке написанный, надо было товарищу показать одному. Мой код смотрите в анкете ссылка на гитхаб

[Евгений Ромашкан]

Дмитрий, Не знаю что я там должен увидеть, но надо ненавидеть людей, чтобы советовать им такое, как трейт Magic.

[Дмитрий]

Евгений Ромашкан, чтоб не занимать extends у класса, реализовать какой либо подключённый интерфейс, меньше копипасты

[Евгений Ромашкан]

Дмитрий, Чтобы было меньше копипасты нужно избавиться от копипасты и думать почему она получилась, а не пихнуть её под ковер оставив в виде трейтов.

[Дмитрий]

Евгений Ромашкан, просто надо знать как применить, вы не пользуетесь?

[Евгений Ромашкан]

Дмитрий, Кого, трейты? Они не нужны, т.к. кроме копипасты кода ничего не делают

[Дмитрий]

Евгений Ромашкан, то есть и наследование вы не используете и интерфэйсы и прочее?

[Евгений Ромашкан]

Дмитрий, Наследование это образование подтипов а не только реюз кода, хотя его тоже стоит по возможности избегать(90% юзают его зря), но по другим причинам, а с интерфейсами нет проблем.

Answer 1

[Андрей Корехов]

Спасибо за идеи, но удалось реализовать этот вариант
function setVar($name, $value) {
$this -> $name = $value;
}
$key -> setVar('count', $_GET['count']);

Всем спасибо

Comments

[Евгений Ромашкан]

Задумались бы лучше, зачем вам геттеры и сеттеры.
p.s. Они не нужны, тем более в такой реализации
p.p.s. трейты кстати тоже не нужны.