Я помню, как читал, что в моделях Doctrine 2 я не должен устанавливать общедоступные свойства/поля. Как тогда вы выставляете эти поля? В песочнице использовались методы get*() и set*(). Это лучшая идея? Это очень громоздко. Использование магических методов __get() __set() сделает вещи похожими на настройку полей общедоступными?
Какова ваша рекомендация?
Вот почему вы не можете использовать общедоступные свойства: Как можно общедоступные поля «прерывают ленивую загрузку» в Doctrine 2?
Вы правы, что __get() и __set() могут облегчить доступ к полям protected/private.
Вот простой пример:
public function __get($name)
{
if(property_exists($this, $name)){
return $this->$name;
}
}
Конечно, это дает доступ ко всем свойствам. Вы можете поместить это в класс, расширенный всеми вашими объектами, а затем определить неоцениваемые поля как private. Или вы можете использовать массив, чтобы определить, какие свойства должны быть доступны: $this->accessable = array('name', 'age')
Существует множество способов защитить все свойства и при этом иметь достаточно простой способ их получения/установки.
...We therefore urge you to map only private and protected properties on entities and use getter methods or magic __get() to access them. doctrine- project.org/docs/orm/2.0/en/reference/
- person Petr Peller; 20.05.2011
$function = 'get' . ucfirst($name); $this->$function(); Я знаю, что это не поможет вам НЕ писать код... но нет никаких причин, по которым вы не могли бы проверить, работает ли функция не существовало, а затем вместо этого извлеките свойство. Это позволит вам перегружать эти геттеры и сеттеры, не делая прямого сопоставления там, где вам это нужно, и поддерживать красивый короткий синтаксис $object->property.
- person General Redneck; 12.02.2018
Лично мне не нравится шаблонный код с тривиальной целью — он делает код уродливым и утомительным для чтения. Поэтому я настоятельно предпочитаю __get/__set. Тем не менее, у них есть несколько недостатков:
__get/__set вызывается только тогда, когда поле не видно; если вы обращаетесь к свойствам в коде класса сущности, они не вызываются, и у прокси нет возможности загрузить себя. (Doctrine пытается избежать этого, мгновенно загружая прокси-сервер, как только вызывается один из его общедоступных методов, но есть некоторые исключения, такие как __construct или __wake, где это не имеет смысла, поэтому вы можете попасть в беду, например, прочитав поле в конструкторе.)empty($entity->field) не будет вызывать __get (и, таким образом, это нарушит поведение прокси-сервера, если оно используется)empty($entity->field) вызовет магический метод __isset(), который предполагается использовать в документации.
- person feeela; 01.06.2016
Если какая-то информация должна быть обнародована, определите для нее геттер. Если это поддается изменению, добавьте сеттер (еще лучше, добавьте свободный сеттер!).
Таким образом, API чище, без магии. Я не люблю магию в своем коде.
Просто мои два цента :)
Под "свободным сеттером" я подразумевал тот, который реализует шаблон свободного интерфейса.
return $this;, поэтому вы можете выполнять цепочку, подобную jQuery: $object->setName('name')->setCode('code');. Хорошая идея. Было бы неплохо добавить это в сеттеры, сделанные инструментом orm:generate-entities.
- person rjmunro; 19.12.2011
Да, методы получения и установки - это способ доступа к вашим данным. Они немного громоздки, поэтому некоторым людям не нравится доктрина2 или спящий режим. Но вам нужно сделать это только один раз для каждой сущности, и тогда они будут очень гибкими для создания форматирования вывода, на которое вы надеетесь. Вы можете использовать cli, чтобы сделать это за вас. Но когда вы заставляете их вращаться, я не нахожу это большим делом. Тем более, что вы делаете это только с теми свойствами, которые вам нужны.
Ваше здоровье
Вместо того, чтобы иметь отдельные геттеры и сеттеры или даже использовать магические функции. Есть ли проблема с наличием чего-то подобного в классе
public function Set($attrib, $value)
{
$this->$attrib = $value;
}
public function Get($attrib)
{
return $this->$attrib;
}
Это упрощает доступ к атрибутам и означает, что они динамически устанавливаются из массивов пар ключей.. есть комментарии? или альтернативные предложения?
Doctine 2 предоставляет [инструмент командной строки][1] для создания базовых классов сущностей.
используйте следующее, чтобы получить базовое определение класса Entity из вашего сопоставления, дополненное функциями получения/установки для каждого свойства:
path/to/doctrine_cli orm:generate-entities --generate-methods=true path/to/entities/
Вы по-прежнему несете ответственность за изменение каждого метода получения/установки, чтобы гарантировать, что они являются правильным типом данных, поскольку методы получения/установки, сгенерированные для Entity, не выполняют приведение/преобразование типов.
