Передача параметров в конструктор аргументами функции или массивом?

Question

[Тарас Сереванн]

Параметры передаются аргументами:
new Class($param1, $param2, $paramN)

Параметры передаются массивом
new Class(array $params)

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

Comments

[V Sh.]

Зачем передавать синглтон, если он хранит параметры? Сделайте его публичным и обращайтесь к нему из классов) Вообще мне кажется, что это зависит от количества параметров и от типа функции. Если это агрегирующая функция, то нужен динамический список параметров, если нет, то вполне хватит и обычных.

[Тарас Сереванн]

V Sh.: >обращайтесь из классов
но ведь это снизит переносимость кода, нет>?

[trevoga_su]

V Sh.: > Зачем передавать синглтон, если он хранит параметры?
> Сделайте его публичным и обращайтесь к нему из классов)

Это плохая идея. Читайте про инверсии зависимостей.

Answer 1

[trevoga_su]

Так, первое.
Передавать в виде массива аргументы - просто отвратительно. Потому что придется знать/заучивать ключи массива, что не очень хорошо, а точнее - ужасно.
Представь, если бы все функции в PHP принимали только один массив! Каково бы было? Ты бы смог помнить туеву хучу ключей для каждой функции? Нет.
Далее, редактор не подскажет/покажет, что должен принимать метод или функция. Осложняется phpDoc-ументирование.

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

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

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

Comments

[Тарас Сереванн]

Полностью согласен. Просто увидел такое в одном популярном классе и решил тут уточнить. А что на счет синглтона?

[trevoga_su]

Тарас Сереванн: тема сингелтона тут вообще ни к месту. он к вопросу отношения не имеет

[trevoga_su]

Тарас Сереванн: я понял. отписал выше. читай про инверсии зависимостей. про принципом единой обязанности и Внедрение зависимости

[Тарас Сереванн]

Слушайте, trevoga_su, вы на мои вопросы тут отвечаете, я иногда на ваши, может можно вас в друзья добавить ВКонтакте чтобы иногда задавать глупые вопросы по ООП и паттернам (я только недавно начал в это вникать)? ;)

Answer 2

[Максим Кузнецов]

Массив используется там, где нужна гибкость. Т.е. там, где разработчик не может предугадать, сколько параметров придёт при вызове метода, какие типы данных будут иметь передаваемые значения и т.п. За такую гибкость разработчик должен платить, создавая код обработки массива, что может быть весьма нетривиальной задачей. Кроме трудоёмкости и сложности кода получите ещё и потерю производительности. Излишней гибкости следует избегать.

В общем случае лучше передавать либо несколько параметров, либо строго типизированный объект с заданными полями. Причём при передаче большого количества параметров тоже могут быть просадки с производительностью.

Comments

[BoShurik]

Полезный компонтент для работы с массивом параметров: OptionsResolver Component