Как завести связи между объектами?

Question

[ozornick]

Здравствуйте. Где почитать или объясните кто знает о таком случае.
Имеем

class Operation
        {
            public $id;
            public $date;
            public $user_id;
            
            // Далее сеттеры, геттеры
        }

Теперь нужно вывести все операции совместно с информацией некоего пользователя

class Operation {
// ...
public $user

    // Добавить User    $user = new User::find($this->$user_id)
    public setUser(User $user) {
        $this->user = $user;
    }
}

По сути вопрос можно перефразировать "Как работать с вложенными сущностями?" Стоит ли при создании класса внедрять инъекцию или есть лучший вариант? В основе работа по принципу Data Mapper
UPD: Поправил пост

Answer 1

[dmitriy]

1. по логике если operation не может существовать без user то при создании объекта он должен быть обязателен new Operation(new User("username"));
2. public setUser() - неправильно реализован, так нужно: setUser(User $user)
3. Работа с вложенными объектами тема правильная но нужно понимать что для передачи такого объекта в слой хранения нужен будет писать datamapper который будет трансформировать объект в формат хранения бд. Как правило ORM сразу предоставляет инструментарий для этого.

Comments

[ozornick]

1. По логике требуется user_id. Какие то данные нужны для банального вывода, далее посмотрим.
№ Пользователь Операция Дата
24 ИмяКакоеТо Читает 12-04-17
2. Реализацию писал на коленке. Не суть важно. Показал, что требуется зависимость $user
3. Изначально стандартно выбираю Operation. А дальше?

$operation = OperationModel::findAll();
echo $operation->$user->getName();
// вопрос что делать внутри
static function findAll()
{
    $operation = Db::execute("SELECT * FROM operation");
    $operation->setUser(Db::execute("SELECT * FROM users WHERE user_id = $operation->getUserId()"));
}

[dmitriy]

https://github.com/dimaxz/code-example

[ozornick]

Что там интересного? DataMapper?

[dmitriy]

может вам не изобретать велосипед а взять Propel2 ORM?

[ozornick]

diamond: может мне не пытаться кодить, а просто купить готовый скрипт?

Answer 2

[GreatRash]

Я может чего не понимаю, но:

class Operation {
  // ...
}

class UserOperation extends Operation {
  // ...
}

Comments

[ozornick]

Тоже вариант, но если потребуется еще расширить Operation добавив зависимость?

[GreatRash]

ozornick: Вроде PHP позволяет делать так:

class One {
  public function testOne() {
    echo 'I am One';
  }
}

class Two extends One {
  public function testTwo() {
    echo 'I am Two';
  }
}

class Three extends Two {
  public function testThree() {
    echo 'I am Three';
  }

  public function test() {
    self::testOne();
    echo '<br />';
    self::testTwo();
    echo '<br />';
    self::testThree();
  }
}

$obj = new Three();
$obj->test();

Answer 3

[Максим Тимофеев]

Наверное Вам будет интересно почитать про AR: codengineering.ru/post/3

Comments

[ozornick]

Нет, не интересно. Как раз таки настраиваюсь на Data MApper

[dmitriy]

mysql_query - deprecated
php.net/manual/ru/function.mysql-query.php

[Максим Тимофеев]

ozornick: тогда надо бы это в вопросе указать.

Answer 4

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

Если правильно понимаю, Вас интересует вопрос как именно связанные данные из реляционного представления преобразовать в объектное. В этом случае рекомендую почитать про паттерн Foreign Key Mapping. Почитать можно у Фаулера, (ну либо просто погуглить)

Comments

[ozornick]

Возможно. Ушел в гугл...

[ozornick]

https://youtu.be/B-muCmHHofo?t=4m10s Как я и предполагал, через внедрение добавить объект User

Answer 5

[trevoga_su]

Вариант 1.
Как выше сказали - строго необходим объект пользователя, что бы знать его ID.
При сохранении пишем ID объекта пользователя в таблицу операций или, в отдельную таблицу связей, если предполагается связь один ко многим.
DM при сохранении модели Operation имеет в методе сохранения код, который отвечает за связь операции и пользователя.
Грубо говоря, метод operation::save() имеет реализацию, в которой сохраняется связь.
Модель Operation имеет lazy load методы, которые позволяют выцепить объект пользователя:
$operation->getUsers();

Вариант 2.
Реализуем систему, при которой любые свзяи между объектами храним в ОДНОЙ таблице:
id объекта который привязан
id объекта к которому привязан
модель (например, имя класса) которая привязана
модель (например, имя класса) к которому привязан
- тут можно связывать любые объекты в системе
за счет ID объекта модели и имени класса достигается уникальность связей:

234 | 567 | \my\namespace\user | \my\namespace\operation

Comments

[ozornick]

я так и размышляю. Важнее работа с этим $operation->getUsers(); Как его ввести и получить

[trevoga_su]

class operation {
    private $user = null;

    public function getUser()
    {
        if ($this->user == null) {
            // поиск пользователя по $this->user_id в какой-то там таблице связи...
            $this->user = (new YouDataMapperUser2Operation)->findByUserId($this->user_id);
        }

        return $this->user;
    }
}

[ozornick]

То есть класс сущности должен знать о маппере? Что если просто при создании operation инициализировать user? Или это затратно в плане памяти и быстродействия? Вообщем нужно размышлять... Пока ответ неопределен

[trevoga_su]

То есть класс сущности должен знать о маппере?

без этого не обойтись. по моей практике. я тоже писал систему с DM и моделями. класс сущности может не знать о мэппере, но должна быть возможность быстро получить необходимый маппер.

[ozornick]

trevoga_su: реализовал пока банально. Теперь думаю, должен ли Operation что то менять в User, или ограничить только информацией?

// в operation_mapper::find(1)
$data['user'] = user_mapper::find(1);
self::create($data);
// сам operation тоже имеет метод создания себя
create($data){
    setUserId($data['user_id']);
    setUser($data['user']);
}