ООП. хранить ли данные в объекте?

Question

[Andy91]

Прошу прощения наверное за глупый вопрос, но трудно его сформулировать, поэтому не знаю даже как googl-ить эту тему. Подскажите новичку как правильней относительно ООП хранить данные. Трудно пока обяснить поэтому привожу два примера:

class Class{
  function FUNC(){
    // код для получения DATA
    return Data;
  }
}

obj = new Class();
d = obj->FUNC();

или так

class Class{
  private DATA

  function FUNC(){
    if( this->DATA ){
      return this->DATA;
    } else {
      // код для получения DATA;
      this->DATA = DATA;
      return DATA;
    }
  }
}

obj = new Class();
d = obj->FUNC();

Comments

[Sayonji]

Если результат вычислений DATA всегда одинаков, то лучше, конечно, закешировать (вариант 2).

Answer 1

[xfg]

Не оба.

class Class {
  private $data;

  public function __construct($data) {
    $this->data = $data;
  }
  public function data() {
    return $this->data;
  }
}

Код для получения data живет за пределами класса. Чтобы понять, почему так, переименуйте Class например в Auto, а data например в color. В общем, суть в том, что у объекта должно быть состояние (свойства объекта) и может быть поведение (методы объекта), которые это состояние меняют. Это должно быть очевидно. Иногда бывают исключения. Очень редко. Почти всегда, можно сделать тру ООП.

Сейчас у вас, к сожалению, процедурный код, обернутый в номенклатуру ООП :(

Comments

[Andy91]

Спасибо за ответ! Но я видимо плохо поставил вопрос. То что Вы написали я прекрасно понимаю и только подобным образом поступаю.
Просто в моём случае DATA это параметры которые не определяют объект, а генерируются за "базовых" свойств объекта, которые были определены в конструкторе.
Эти данные могут понадобится (могут и не понадобится) и однозначно перегружать ими объект при создании класса не стоит.

А вопрос в следующем: если вдруг эти данные понадобятся их лучше хранить в объекте (и обращаться к ним через объект) или же в записать их в новую переменную и обращаться к ним уже через неё?

[xfg]

Нужно видеть реальную задачу. Не понятно, что вы хотите сделать. В любом случае, данные в объекте храниться не должны, для данных есть хранилища (база/файл/удаленный сервис и т.д.). Из данных собирают объект или вложенную иерархию объектов (агрегат). Скорее всего, то что вы хотите хранить в $this->data можно разложить на дочерние объекты, которые будут жить внутри класса Class.

Напишите какую вы задачу решаете.