Телеграмм чат группы symfony

Size: a a a

symfony

2345 membersпожаловаться на группу

2021 April 17

CvekCoder in symfony

Чем? Поглядел в трейт и все увидел. В чем неочевидность?

источник

00:04пожаловаться #1

АВ

Александр Вологжанин... in symfony

Представь что есть интерфейс с двумя методами:

public support(MyClass $obj, MyClassTwo $obj2);
public handle(MyClass $obj, MyClassTwo $obj2);

Реализация support почти везде будет на подобии:

public support(MyClass $obj, MyClassTwo $obj2)
{
    return $obj->name === $obj2->name;
}

Но иногда в 1 из 10 случаев будут встречаться более сложные проверки, я вижу 3 пути:
1. Создаешь абстрактный класс за место интерфейса
2. Копипастишь
3. Создаешь трейт

источник

00:05пожаловаться #2

CvekCoder in symfony

В данном конкретном случае я и хочу его повысить. Потому что каплинг, как и пиво, не только вредно, но и полезно. Надо мне изменить во всех дефолт реализациях поле на nullable - я это изменю в одном месте.

источник

00:05пожаловаться #3

АЯ

Андрей Ява in symfony

1. В том, что в них начинают писать сторонний функционал. Прям функционал не два поля
2. Поля неочевидны. Он сам неочевиден, и его малейшее изменение сразу автоматом применится ко всем сущностям.
Другими словами - трейт нельзя менять никогда.

источник

00:06пожаловаться #4

Шурик in symfony

а часто бывает кейс, что тебе надо прям во всех дефолт сразу что-то поменять?)

Бывало кстати

типа есть 100 классов и для всех 100 надо поменять сраный createdAt?
а что должно произойти, чтоб такое понадобилось?

Это ж тот самый dry

ладн, я пас)

не)

00:07пожаловаться #10

АЯ

Андрей Ява in symfony

Это не dry

источник

00:07пожаловаться #11

CvekCoder in symfony

С другими трейтами было, с createdAt - не припомню

источник

00:08пожаловаться #12

АЯ

Андрей Ява in symfony

Dry это когда у тебя функционал лежит в одном месте, а если ты его скопипастил везде куда только можно, просто приныкал копипасту треем - это не драй, это замаскированная копипаста

источник

00:08пожаловаться #13

CvekCoder in symfony

Пример - раньше был date, а теперь хотим datetime.
Был datetime - хотим datetime_immutable

источник

00:09пожаловаться #14

Sergey Milegov in symfony