Нужно ли использовать interface?

Question

[PANOV_DEV]

Есть класс ValutaManager в __construct у меня добавление объектов:

class ValutaManager {
   private $valuta = [];

   public function __construct() {
      $this->valuta["rub"] = new RubMoney();
      $this->valuta["uah"] = new UahMoney();
      $this->valuta["eur"] = new EurMoney();
   }
}

Так же, во всех этих классах (RubMoney, UahMoney, EurMoney) у меня имеется метод getName() - он возвращает мне название валюты.

Так же, у меня сам метод getName() вызывается в ValutaManager.

Нужно ли создавать interface Money и указывать его в каждом классе Uah, Rub, Eur?

Можно ведь его не указывать и будет работать так же, как и запланировано.

Comments

[Сергей delphinpro]

valuta VS Money O_o
Currency & Money
Valuta & Dengi

Вы уж или трусы наденьте, или крестик снимите...

[Сергей delphinpro]

interface IMoney {
  public function getName(): string;
}

class CurrencyManager {
   /** @var IMoney[] */
   private $currencies = [];

   public function __construct(IMoney $rub, IMoney $uah, IMoney $eur) {
      $this->currencies["rub"] = $rub;
      $this->currencies["uah"] = $uah;
      $this->currencies["eur"] = $eur;
   }
}

$cm = new CurrencyManager(
    new RubMoney(),
    new UahMoney(),
    new EurMoney()
);

Answer 1

[Ипатьев]

Нет, ничего указывать не надо.
Тут и без интерфейсов получатся отличные самолёты из соломы и палок.

Comments

[PANOV_DEV]

Я вроде задал адекватный вопрос, а не про твои детские фантазии про самолёт из соломы и палок.

[Adamos]

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

[Дмитрий]

PANOV_DEV, А где вы проверяете что обьект должен быть именно interface Money? Если бы вы в конструкторе плохо названого класса ValutaManger получали какую нибудь валюту имело бы смысл. А без этого какой? Что бы вам IDE подсветил если вы забудете метод getName реализовать в валюте? Ну не знаю. Смысл метода то в каждом классе не понятен ибо он явно одинаков

[Ипатьев]

PANOV_DEV, вы явно понятия не имеете что такое интерфейс, и зачем он нужен.
В этом случае использовать его однозначно не следует.

[Sergei Nazarenko]

PANOV_DEV,

Я вроде задал адекватный вопрос, а не про твои детские фантазии про самолёт из соломы и палок.

Это не его детские фантазии, это реально говнокод. ValutaManager - не, ну - это же дичь, голимый рунглишь такое бывает когда инглиша нотандерстандабле. Оно плохо читаемо и непонятно. Без обид - ты и сам это поймешь со временем когда (если) вникнешь. Советую почитать книжицу "Чистый Код".

[Ипатьев]

А. теперь понятно, почему мои ответы игнорят :)
Ну, не буду зря кнопки давить.

[PANOV_DEV]

Sergei Nazarenko, выучи значение слова "пример"

[PANOV_DEV]

Sergei Nazarenko, и название так-то не влияет на работу способности, генний

[Sergei Nazarenko]

PANOV_DEV, Выучи русский язык, ну и английский не мешало бы.

[PANOV_DEV]

Sergei Nazarenko, Т9 не слышал? Указать ошибку можешь или только говорить умеешь?

Небось, на слово "способности" клюнул? Генний.

[Sergei Nazarenko]

PANOV_DEV, На какие ошибки тебе указать? То, что ты не знаешь английского языка? Ну это очевидно же, потому как если бы знал, то не приводил бы подобных примеров. Ну тебе как минимум было бы "стремно" блистать своим невежеством. Это первое. Во-вторых - T9 устаревшая технология, которая в современных телефонах не используется, а та, что используется ну как то не исправляет правильно написанных слов, а это значит, что ты изначально писал неправильно. И в-четвертых, что ты мне пытаешься тут доказать? Что ты офигенный программист, а все остальные ничего не понимают? Человек сделал тебе замечание по делу. Но ты этого не в состоянии понять, потому что опыта и знаний у тебя не хватает.

[PANOV_DEV]

Sergei Nazarenko, тебе пишешь одно, а ты о другом. Кстати, грамматику иди учи. У тебя знаки препинания не все и стоят там, где не нужно, неуч.

[Sergei Nazarenko]

PANOV_DEV, ОК

Answer 2

[Newto]

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

Comments

[Adamos]

Это вопрос сложности.
Структурирование сложного кода его упрощает.
Структурирование элементарного - усложняет.

[Newto]

Adamos, Вы абсолютно правы. К подобным выводам разработчик приходит с опытом.

Answer 3

[Григорий Васильков]

Вы когда на работу устраиваетесь - заключаете контракт. Контракт описывает что вы должны уметь. Это вот интерфейс "на вас".
Тогда как вы сами - это "класс", который что-то уже умеет.

Если ваши классы ничего ЧТО ТРЕБУЕТСЯ ДРУГИМ не делают - интерфейс на них не обязателен.

Интерфейс выполняет две задачи. Даёт классу назначение (имя в рамках задачи) и контроллирует, что класс что-то умеет.

* Ещё он умеет константы хранить, но этим редко пользуются (а зря, интерфейс меняется от проекта-к-проекту, а реализация - от задачи-к-задаче, стало быть объявленные константы вполне себе могут быть в интерфейсе, т.к. реализацию могут удалить)

Назначение чаще всего применяется в контейнерах зависимостей (см. Psr\ContainerInterface), там два интерфейса можно (и нужно) использовать, чтобы два одинаковых объекта положить в разные "коробки" и достать одну в одном классе, а другую - в другом.

Вторая функция применяется для проверки входных данных, когда вы в функции указываете тип данных для параметра, и программа упадет в ошибку, если тип данных неверный. Этот же принцип используется для организации ПОДМЕНЫ одного класса на другой. Если смотреть на вашу задачу (хотя здесь интерфейс не нужен) , то вместо того, чтобы на входе писать RurMoney вы сможете написать MoneyInterface, если вам не важно, какая валюта придет в функцию, но они все умеют в ваш ->getName()

В некоторых программах есть классы-данные, в которых нет методов (либо есть только геттеры и сеттеры - это вот ваш случай). Эти классы называют ValueObject. Для них интерфейс писать не надо. Чаще всего для них даже несколько классов делать не надо, просто один класс с разными свойствами. Но вот если методы есть - то нужно сделать АБСТРАКТНЫЙ КЛАСС, от него наследовать конкретные классы, тогда на входе проверять не интерфейс, а абстрактный класс. Абстрактный класс объединяет в себе "действия по-умолчанию" и "проверку умения" (через абстрактные методы). Они похожи на те, что в вашей задаче - getName() есть у всех - отличный способ положить этот метод в абстракцию, и не писать интерфейс, т.к. у вас уже есть абстрактный класс.

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

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

Конкретно вашу задачу лучше решать разобравшись как настраивать вот этот пакет:
https://github.com/moneyphp/money

Comments

[Ипатьев]

Как всегда, у вас ооооочень путанный ответ.
Я бы от него оставил только первый абзац.

[Григорий Васильков]

Ипатьев, вы правы. Я пытаюсь описать свои мысли с 3-4 разных позиций, чтобы человек нашел в нем именно то предложение, которое предполагает его текущее "настроение" (если так можно назвать). Текста много, но какое предложение ему зайдет - я не знаю, и поэтому пишу одно и то же с разных точек зрения.

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

[Григорий Васильков]

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

[Adamos]

Григорий Васильков,

Вы когда на работу устраиваетесь - заключаете контракт

А когда договариваетесь с другом, что он возьмет пива, а вы ему скинете денег на карточку - нет.

[Ипатьев]

Григорий Васильков, я знаю только один способ - перечитывать и контролировать последовательность изложения. Лишнее убирать
А так-то есть хайповая книга как раз про это, "Пиши, сокращай" называется вроде.

[Григорий Васильков]

Ипатьев, понял, почитаю