Использование идентификатора юзера в моделях. Корректно ли?

Question

[Roquie]

Суть: В каждой модели, по 5 - 10 методов, которым как воздух необходим `$user_id`. Вызывая каждый в контроллере, надо всегда передавать идентификатор юзера - прям мучение какое-то, незнаю чтоли :).
Корректно ли задать свойство в базовой модели:

public function __construct()
{
     $this->user = \Auth::getUser();
}

...а затем использовать его по необходимости в остальных.

Или это потом может обернуться "плачевными последствиями"?

Answer 1

[Алексей]

Как по мне так вполне корректно, другое дело что лучше на хардкодить, а передать параметром в конструктор.

Comments

[Roquie]

при использовании тайп-хинтинга передавать параметр в конструктор проблематично )

[Алексей]

Что-то не понятно в чем проблема, покажите какой-нибудь абстрактный пример когда проблематично.

[Roquie]

Алексей Анонимоус:

//это то как есть сейчас. Detail - та самая модель.
public function index(Authenticatable $user, Detail $d)
{
	$statistic = $d->stat($user->getAuthIdentifier(), 'year'); // пока user_id передается в методы модели
	return view('index');
}

переходить на

public function index(Authenticatable $user)
{
        $d = new Detail($user->getAuthIdentifier()); // сейчас это одна модель, а вырастет:
        $two = new Foo($user->getAuthIdentifier());
        $three = new Bar($user->getAuthIdentifier()); //  и т.д.
	
        $statistic = $d->stat('year');
	return view('index');
}

не хочется :)

Answer 2

[Сергей Гладковский]

Плачевными последствиями может обернуться наличие (и их дальнейшее увеличение) 5-10 методов в модели. :) Используйте паттерн "репозиторий" или CQRS.

Если же всё нормально, то наличие определение userId в модели - норм.

Comments

[Roquie]

ну дык, модели должны быть жирными, не?) это все методы обработки данных, причем часть вынесена в хелперы и т.п.

[Сергей Гладковский]

Roquie: зачем их делать жирными?:) пусть будут в меру упитанными той логикой, которая отвечает за передачу и связывание данных...

[Roquie]

Сергей Гладковский: ну в этом плане, они в меру упитанные :)

[vasIvas]

Roquie: я, как и Вы, считаю, что пять методов в модели, это более чем нормально.
И хранить id, тоже нормально. А остальные пусть учатся.

Answer 3

[Максим Кудрявцев]

Для начала 5-10 методов в модели... Чего у Вас там за модель такая?

Насчет поля id_user: если Ваша логика подразумевает, что с одним экземпляром модели будет работать только 1 юзер - объявляйте доп.поле и инициализируйте его в конструкторе.

Кстати, как один из вариантов, если Вам нужно быть уверенным, что этот самый id_user в процессе выполнения метода модели по какой-то причине не изменится и не потянет за собой все снежным комом, используйте его как параметр по умолчанию

public function Roquie($id_user = $this->_id_user) { .... }

В самом методе работайте не с $this->_id_user, который у Вас в контролере инициализируется, а с $id_user, являющийся параметром именно этого метода.

Comments

[JhaoDa]

$this в параметрах?

[Максим Кудрявцев]

JhaoDa: да. При выполнении появляется ошибка?

[JhaoDa]

Максим Кудрявцев: вы, видимо удивитесь, но да: syntax error, unexpected '$this' (T_VARIABLE)