Как создать свойство внутри метода?

Question

[Max Ba]

Устал работать с массивом. Хочу заменить метод

public function setData(array $data):void{
  $this->data = $data;
}

на

public function setData(array $data):void{
  foreach($data as $k => $v){
    $this->$k = $v;
  }
}

1. Делают ли так?
2. Как это сделать?) Пробовал через __set, что-то не разобрался. Первый раз столкнулся с такой необходимостью.

Comments

[Антон Р.]

А что непонятного с __set? https://www.php.net/manual/ru/language.oop5.overlo...

[Дмитрий]

Да можно и так, без set

Answer 1

[ThunderCat]

1. Делают ли так?

Делают, но это не хорошая практика. Во первых имеете шанс затереть какие-то настроечные данные объекта и получить непредсказуемость поведения. Во вторых, если у вас там значения из бд или другого хранилища в формате ключ-значение, лучше хранить их в отдельном поле, том же дата, но в виде stdClass, чтобы не путать их при работе с хранилищем с остальными полями, например так:

public function setData(array $data):void{
  $this->data = (object)$data;
}

в идеале у вас должен быть DTO (data transfer object), но в актив рекорд движках он обычно не используется.

Comments

[Max Ba]

А ну понял. Да, дабы не затереть данные можно проверить exists property. Но я согласен, бардак какой-то получится. Тогда поработаю с массивами.
Спасибо.

[ThunderCat]

Max Ba, не обязательно же, можете внутри этого свойства вполне иметь себе объект, ну будет не $obj->id, a $obj->data->id, все равно лучше.

[Максим Федоров]

только DTO, а не TDO... :) хотя какая разница :)

[ThunderCat]

Максим Федоров, очепятка блин )