Как использовать локальные переменные и переменные экземпляра для улучшения дизайна приложения в программе

Этот вопрос, похоже, больше связан с дизайном приложения, чем с производительностью.

Вместо того, чтобы рассматривать производительность локальных переменных против переменных, обдумайте назначение каждой переменной и спросите себя Does it belong here?, Классы могут стать довольно большими. В конце концов, вы хотите хранить столько информации, сколько необходимо.

Переменные экземпляра (также известный как класс «свойство»)

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

Локальные переменные

В объектно-ориентированных приложениях локальные переменные обычно ограничены областью действия функции (иначе метод), в которых они находятся. Эти переменные обычно уничтожаются или больше не используются после завершения выполнения функции.

Доменные объекты

Ваше приложение содержит объект домена с именем PersonalInformation, Доменный объект имеет одну цель: хранить информацию о thing или же abstract thing (например, личная информация). Следовательно, он должен содержать только такие свойства, как name, surname, birthday, ...и метод, который напрямую манипулирует этими свойствами, например, методы получения и установки.

Доменный объект также находится там, где вы должны разместить свою логику проверки. Если вы используете PersonalInformation объект в нескольких контроллерах, вы будете переписывать одни и те же правила проверки ввода для многих файлов. Этого можно избежать, устанавливая значения в свойствах объекта, а затем проверяя их внутренне через validate() метод.

Таким образом, ваша логика проверки может быть удалена из вашего контроллера.

Контроллеры

Контроллеры предназначены для приема запросов, указания другим частям приложения обрабатывать данные POST / DELETE / PUT, а затем либо визуализировать представление (например, шаблон HTML или строку JSON), либо перенаправить в другую часть приложения.

В большинство случаев, ваши контроллеры не следует обрабатывать логику проверки для ваших доменных объектов.

Рефакторинг вашего кода

Я реорганизовал ваш код, чтобы соответствовать этим принципам, и добавил комментарии, чтобы объяснить цель внесенных изменений.

PersonalInformationController.php

<?php

class PersonalInformationController
{
/**
* Though this controller interacts with one or more PersonalInformation objects, those objects
* do not describe this controller. Nor do they provide any additional information that the
* controller MUST know about.
*
* This variable should be local, not a class property.
*/
//private $personalInformation;

/**
* Because $personalInformation is no longer a class property,
* it does not need to be set in the constructor.
*/
function __construct() {}

function doPost()
{
/**
* The try/catch block will catch any validation exceptions thrown by the validate()
* method withing the $personalInformation object.
*/
try {
/**
* This instance of the PersonalInformation object is only being used within the scope of
* this function, so it should be a local variable.
*
* @var PersonalInformation personalInformation
*/
$personalInformation = new PersonalInformation();
$personalInformation->setName(filter_input(INPUT_POST, "name"));
$personalInformation->setSurname(filter_input(INPUT_POST, "surname"));
$personalInformation->setGender(filter_input(INPUT_POST, "gender"));
$personalInformation->setBirthday(filter_input(INPUT_POST, "birthday"));
$personalInformation->setNationality(filter_input(INPUT_POST, "nationality"));
/** Set other properties of personalInformation ... */

/**
* This validate() method will check the integrity of the data you passed into the setter
* methods above. If any of the domain object's properties are invalid, an exception will be thrown.
*/
$personalInformation->validate();

/** save the object / show view / other controller logic */

} catch(RequiredFieldException $e) {
/** A field was empty. Error handling logic here. */
} catch(InvalidFieldException $e) {
/** A field value was incorrect. Error handling logic here. */
}
}
}

PersonalInformation.php (доменный объект)

/**
* Class PersonalInformation
*
* This class is a domain object. Notice that it only contains properties that describe PersonalInformation.
* All of the logic contained in this class manipulates the properties, and is "unaware" of any outside entities.
*/
class PersonalInformation {

private $name;
private $surname;
private $gender;
private $birthday;
private $nationality;

/**
* Try to avoid duplicate code. If all the validation for your domain object is the same,
* use a method, such as this, to store all of the validation logic.
*
* @throws \InvalidFieldException
* @throws \RequiredFieldException
*/
public function validate() {
if(empty($this->name) || empty($this->surname)) {
throw new RequiredFieldException("Name and surname can't be empty");
} else if(!is_string($this->name) || !is_string($this->surname)) {
throw new InvalidFieldException('Name and surname must be a strings!');
}

if(empty($this->gender) || !is_string($this->gender)) {
throw new InvalidFieldException('Gender must be a string!');
}

if(empty($this->birthday) || !is_string($this->birthday)) {
throw new InvalidFieldException('Birthday must be a string!');
}

if(empty($this->nationality) || !is_string($this->nationality)) {
throw new InvalidFieldException('Nationality must be a string!');
}

/** other validation rules **/
}

/**
* @return mixed
*/
public function getName()
{
return $this->name;
}

/**
* @param mixed $name
*/
public function setName($name)
{
$this->name = $name;
}

/**
* @return mixed
*/
public function getSurname()
{
return $this->surname;
}

/**
* @param mixed $surname
*/
public function setSurname($surname)
{
$this->surname = $surname;
}

/**
* @return mixed
*/
public function getGender()
{
return $this->gender;
}

/**
* @param mixed $gender
*/
public function setGender($gender)
{
$this->gender = $gender;
}

/**
* @return mixed
*/
public function getBirthday()
{
return $this->birthday;
}

/**
* @param mixed $birthday
*/
public function setBirthday($birthday)
{
$this->birthday = $birthday;
}

/**
* @return mixed
*/
public function getNationality()
{
return $this->nationality;
}

/**
* @param mixed $nationality
*/
public function setNationality($nationality)
{
$this->nationality = $nationality;
}
}

Обратите внимание, что приложение теперь использует меньше переменных и функций. Он хранит только столько информации, сколько необходимо в каждом классе. Следовательно, ваше приложение будет работать лучше и использовать меньше памяти. Код чистый, простой и простой в обслуживании.

Надеюсь это поможет!