Как вы передаете данные в шаблоны?

Question

[AlikDex]

Собственно вопрос. Просто отдельными кусками, или собираете 1 массив например. и передаете?
Ну вот например, создаем страничку редактирования профиля, собираем в контроллере нужные модели, и в шаблон их:

return $this->render('profile', [
	'user' => $user,
	'profile' => $profile,
	'passwordForm' => $passwordForm,
	'avatar' => $avatar,
]);

А можно подругому:

$data = [
	'user' => $user,
	'profile' => $profile,
	'passwordForm' => $passwordForm,
	'avatar' => $avatar,
];
return $this->render('profile', [
	'data' => $data,
]);

Ну и интересно, почему тот или иной вариант лучше?

Answer 1

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

А есть разница? Выбор того или иного метода должен быть оправдан чем-то. Что быстрее, что проще читается? Что удобней. На мой взгляд в массива смысла нет, это менее гибко и требует лишнего кода, тогда зачем? Но в Вашей архитектуре это может быть оправдано.

Comments

[AlikDex]

Не знаю насчет есть или нет разницы, интуитивно чувствую что нада второй вариант, т.к. придает чутка стандартизации чтоли. И в шаблонах проще проверять есть переменная или нет. Точнее даже не на существование, а empty(). В случае с isset ведь может быть и существует, но пустая, потенциально приведет к ошибке.

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

AlikDex: Как раз стандартнее первый. Так как оперируете объектами, а не массивом с объектами. "проверять есть переменная или нет" - первый тоже проще

[vyachin]

AlikDex: мой вариант

$data = [
  'user' => $user,
  'profile' => $profile,
  'passwordForm' => $passwordForm,
  'avatar' => $avatar,
];
for($i=0;$i<10;++$i)
{
    $data = [
        'data' => $data
   ];
}
return $this->render('profile', [
  'data' => $data,
]);

[AlikDex]

vyachin: смишшно апхахочешься.

[vyachin]

AlikDex: я просто пошутил)) надеюсь вы поняли что нет во втором варианте никакой стандартизации, а вот в моем есть)))

[AlikDex]

vyachin: просто иногда данных надо набирать довольно много. к примеру список новостей каких-то. пагинашка, сортировка, страница, три цифры для "показано с 51 по 61 из 150", какие рекламные материалы и еще что-то может быть.

И вот тут момент настает. Допустим первая страница, пагинашки нет. Получается проверям перед отправкой в шаблон переменную, и второй раз при выводе виджета. Т.к. виджет не принимает нулл. И так с любыми данным.

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

[vyachin]

ну тогда действительно удобно объединять данные, из вашего примера было не очень понятно

Answer 2

[Валерий]

Согласен с постом выше. В этом конкретном случае нужно исходить или из Вашей архитектуры или из простого удобства.

Как по мне вариант первый - прощу и удобочитаемей.
Складывать все в массив стоило бы при передаче нескольких логических блоков, например:

$profile = [
  'user' => $user,
  'profile' => $profile,
  'passwordForm' => $passwordForm,
  'avatar' => $avatar,
];
$friends = [
  'list' => $list,
  'blocked' => $blocked
];
return $this->render('profile', [
  'profile' => $profile,
  'friends' => $friends
]);

Answer 3

[Stanislav]

return $this->render('profile', [  'user' => $user,  'profile' => $profile,  'passwordForm' => $passwordForm,  'avatar' => $avatar,]);