
Sergey
03.10.2018
21:22:59

Maksim
03.10.2018
21:23:02
ничё не понял

Andrew
03.10.2018
21:23:14
Марко описывал такой подход в одном из видео
Пивета который

Google

Sergey
03.10.2018
21:23:27
ну ок, но всеравно непонятно)
как хоть это выглядит?

Andrew
03.10.2018
21:23:53
сек, ща

Maksim
03.10.2018
21:24:04
скинь видео что ли) а то я себя идиотом чувствую... там какие-то chain repository в тренде, а я не в курсе даже)

Andrew
03.10.2018
21:24:40
Видео не найду, не помню
https://paste.ofcode.org/GppaxUMdUa7g9geSEqx3AK
Вот пример, кусок моего кода

Sergey
03.10.2018
21:24:52
может ты о штуках когда у тебя твой репозиторий а внутри общий entity repository?

Maksim
03.10.2018
21:25:31
чёт срань какая-то

Bohdan
03.10.2018
21:25:45

Sergey
03.10.2018
21:25:47
если я правильно понимаю - это chain of responsibility который выбирает имплементацию по фильтру

Maksim
03.10.2018
21:26:10

Sergey
03.10.2018
21:26:14
но специфичная штука как по мне

Google

Andrew
03.10.2018
21:26:26
Пока не использую)
Оставил, может найду где применить

Maksim
03.10.2018
21:28:01
final class ChainUserRepository одобряю) нужно больше подношений богу final'ов)

Andrew
03.10.2018
21:28:16
Все файнал
Только там где надо дать возможность даем на extend

Maksim
03.10.2018
21:29:02
т.е. ни где?)

Andrew
03.10.2018
21:29:12
Нет
Если класс abstract
Его must хоть что то extend
Или если не абстрактный и не финальный, значит так предусмотрено логикой приложения, к примеру класс Фактори, от него идет Фактории разные

Arky
03.10.2018
21:31:32

Andrew
03.10.2018
21:31:37
Хотя тут уже абстрактный опять таки, ибо должен быть метод 1 минимум create(). Мало где не final. Редко когда надо делать extend не абстрактных классов.
)

Maksim
03.10.2018
21:32:12

Arky
03.10.2018
21:32:41

Maksim
03.10.2018
21:33:07

Arky
03.10.2018
21:33:17
да)

Andrew
03.10.2018
21:33:23
у меня все стрикт тайпс)

Maksim
03.10.2018
21:33:26
да)
324 code results in mmasiukevich/service-bus

Arky
03.10.2018
21:33:52
жоско)

Google

Maksim
03.10.2018
21:34:49
жоско)
в темплейт на создание файлов забито) кое кто считает, что типы в рантайме низя проверять, но я сраный параноик)

Bohdan
03.10.2018
21:35:01
@Eraice тебе вот тоже сразу линка

Andrew
03.10.2018
21:35:13
@desper1989 тру стори

Maksim
03.10.2018
21:36:03

Arky
03.10.2018
21:36:05

f4rt~
03.10.2018
21:36:57

Maksim
03.10.2018
21:37:11

f4rt~
03.10.2018
21:37:24
https://github.com/mmasiukevich/service-bus/blob/master/src/EventSourcingSnapshots/Snapshotter.php#L95

Maksim
03.10.2018
21:37:40
а чё не так?)

Bohdan
03.10.2018
21:37:45

f4rt~
03.10.2018
21:37:59
psr2 жи
{ на той же строке

Maksim
03.10.2018
21:38:06
а, срать на пср2)
тут уж кто на что учился)

f4rt~
03.10.2018
21:38:24
если откровенно я тоже так пишу
но честно правлю фиксером после
на пре коммите)

Arky
03.10.2018
21:38:34

f4rt~
03.10.2018
21:38:58

Arky
03.10.2018
21:39:18
всегда писал без пробела, хз зачем он нужен(

Google

Maksim
03.10.2018
21:39:21
но честно правлю фиксером после
я так пишу много лет и переучиваться только потому, что толпа хомячков решила, что так сейчас "более лучше" мне лично фпадлу)
пускай на питон идут писать) там нет проблем со скобками)
я в большинстве случаев одобряю стандарты на интерфейсы, но вот код стаил - это ещё 1, 101й стандарт. всё равно в каждой тиме он свой. хоть 1 пробелом, да отличается

f4rt~
03.10.2018
21:45:53
я до сих пор не могу получить конфиг для фиксера, у меня пздц все разименования, пробелы, куча всякой штуки специфичной, вплоть что стриктайпсы на второй строке
а я привык писать на одной с <?php

Maksim
03.10.2018
21:46:18
забавно, но у меня в конторе фиксером никто не пользуется)
и у всех всё одинаково

knopkod4v
03.10.2018
21:46:44
на предыдущем месте работы у товарищей правило с выравниванием =, => внутри блока было и разделения на блоки (правда они не рассказали мне как на эти блоки разделять =\)

f4rt~
03.10.2018
21:46:48
ну я не могу в голове столько удержать, в плане считать что бы multiline syntax
был 4 пробела
а не 8м

Maksim
03.10.2018
21:47:01
в шторме реформат есть

f4rt~
03.10.2018
21:47:08
дык
он и форматирует по дефолту 8
это не большая проблема, я ручками что то настроил что то приноровился проверять

Andrew
03.10.2018
21:49:34

Sergey
03.10.2018
21:50:09
мне немного не хватает претира для php, пока обхожусь php-cs-fixer-ом. Что до шторма и его реформата - не особо он помогает. Не все можно настроитть и невозможно сильно под себя кастомайзить

Maksim
03.10.2018
21:50:37

knopkod4v
03.10.2018
21:50:39
а надо ли под себя кастомайзить?

Sergey
03.10.2018
21:51:05
ну вот с претиром не надо, потому что там и так все хорошо (ну как, есть вопросы - но зато всегда все одинаково)

Google

Sergey
03.10.2018
21:51:14
а php-cs-fixer по умолчанию все довольно примитивно делает
ну мол, хочу например автоматические переносы если аргументы в вызове метода приводят к строке больше чем на 100 символов - ищем плагин. Нету плагина - пишем

knopkod4v
03.10.2018
21:52:30
стыдно признаться, но я никогда не пользовался именно fixer-ом, только php-cs =\ И как-то не ощущаю необходимости

f4rt~
03.10.2018
21:52:34
ну я ленивая задница, фиксером спасаюсь;
был бы единый кодстайл, было бы другое дело, причем я не сторонник тащить cs на CI
хочется что бы пошарил один конфиг и все юзают его на прекоммитах

Maksim
03.10.2018
21:52:45
автопереносы у нас люди ставят)

Sergey
03.10.2018
21:52:54
это как "я не сторонник того что бы внутри команды стиль кода был единообразен"

Maksim
03.10.2018
21:53:15
а если забывают, то в угол)
но чёт на моей памяти давно никто ничего не забывал) уже много-много лет)

Sergey
03.10.2018
21:53:17
"а как же индивидуальность!"

f4rt~
03.10.2018
21:53:30

Sergey
03.10.2018
21:53:30

f4rt~
03.10.2018
21:53:45
а не когда мне CI фиксер запустит и все сделает

Sergey
03.10.2018
21:53:52
не не
CI должен убедиться что никто не забыл - фиксить он не должен
CI не должен менять код

Maksim
03.10.2018
21:54:11

f4rt~
03.10.2018
21:54:12

Sergey
03.10.2018
21:54:32
могу объяснить почему код стайл это важно, но объяснять (или документировать) где пробелы ставить - нет уж

Maksim
03.10.2018
21:55:28
ну я не сидел и каждый кейс не разбирал, если ты об этом)

f4rt~
03.10.2018
21:55:48