короче творите что хотите - лишь бы на продакшен это не попадало

и нужно больше методов возвращающих void :)

fluent interface уже выше упоминался. Его польза весьма сомнительна.

короче творите что хотите - лишь бы на продакшен это не попадало

ну если на продакшен попадёт код не по psr, то всё, ялда рулю. Да?

честно признаюсь: я в жизни не видел ни одного проекта, который явно следует psr code-style

и сомневаюсь, что увижу

ну если так подходить к вопросу - то лучше сразу "жить в хлеву"

при чём тут хлев, если пср - говно?

было 100 разных стандартов, сделали 1 универсальный и стало 101

Да норм стандарт

главное обьеденить всех

.env files support

поставил, занятная штука. Но тут нюанс есть 1. в самом сервис-басе ключи ногами не забиты и env файла нет в поставке (и не может быть). Тут по сути на откуп отдано

ну да. библиотечка ж

хоть и явная зависимость есть

от dotenv'а но не самая худшая, к.м.к

но за плагин в любом случае спасибо) в хозяйстве пригодится)

главное обьеденить всех

ну хз. Имхо, пср следуют ток его авторы. И то явно не все те, кто его принимал

если смотреть именно на cs

ну да. библиотечка ж

а если вкусовщины не касаться, есть что криминального накинуть по архитектуре?

надо смотреть :)

ну в любом случае это не говнокод. на первый взгляд

на мой взгляд, говнокод)

я позже гляну

но эт нормально) оно у меня так всегда) спасибо

ну у меня более снисходительная планка наговнокод

мне там выборочно из ооп чатика уже накидали от души, есть чем заняться. Но хочется услышать более приземлённые мнения ещё)

Пока нет хотя бы 100 звездочек - говнокод и не нужно.

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

https://github.com/mmasiukevich/service-bus/blob/0f0491b082cd52098a59c1bbac534ee6cb039824/src/Extensions/Logger/ServiceBusLogger.php https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-3-logger-interface.md

а лучше взять готовый

Как говаривал древнегреческий философ Эсхил - хорош не тот кодер, который делает многое, а тот, который делает нужное ?

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

https://github.com/mmasiukevich/service-bus/blob/master/src/Task/CompletedTask.php#L96 почему? в чем смысл?

именованных конструкторов?

Да походу)

ну смысл в именованных конструкторах и в тех возможностях, которые они открывают. Собсна, всё, наверное.

https://github.com/mmasiukevich/service-bus/search?utf8=%E2%9C%93&q=AbstractBootstrap&type= он вообще используется?

https://github.com/mmasiukevich/service-bus-demo/blob/master/src/Application/Bootstrap.php

ну смысл в именованных конструкторах и в тех возможностях, которые они открывают. Собсна, всё, наверное.

у тебя 1 конструктор в классе, больше похоже не фетишь чем на практическую необходимость

кто сказал, что он 1? и почему он должен быть одним?

создание класса Bootstrap обернуто в try catch?

кто сказал, что он 1? и почему он должен быть одним?

код класса CompletedTask

ну если пользователь не поленится, то обернёт. Это вне компетенции данного проекта

ему как бы референс по тому, что падает, предоставлен. Не вижу проблемы

проблемы нет, просто нет целесобразности писать лишний метод

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

ты просил поругать - я ругаю, критику нужно уметь принимать.

я принимаю, но это чистой воды вкусовщина. Думаю, ты-то должен сие понимать.

код не по пср - это не критика.

Пример @Adelf32 хороший был выше. Он накинул, что постфиксы *Interface говно и всё, успокоился) холиварная тема, начинать не резон.

ответ должен быть - у меня принят другой стандрат форматирования кода, отличный от PSR. а начинается какая то "истерика" PSR говно.

в вашей компании нет практики corereview?

в нашей компании нет фанатов psr cs) и вообще таких не видел. а наезд в стиле "на дворе 2018й, а вы не по psr пишите" - чушь, уж извини

а что там не по пср?

Скобки, например)

говносуффиксы для интерфейсов - есть :)

пффф

о даааа

да у нас с тобой одинаковаярелигиянасчетскобок!

а что там не по пср?

Opening braces for control structures MUST go on the same line, and closing braces MUST go on the next line after the body.

блин. пробел мой :)

кароч скобки эти - говно

надо как у него. и как у меня :)

с новой строки скобку.

о, нас уже двое))

а те кто слепо копирует пср - у васчто? экраны маленькие?

коду нужен простор. а не лепить подряд строки друг к другу

эстетика епт!

а те кто слепо копирует пср - у васчто? экраны маленькие?

самое забавное, почему MUST? без этого код иначе работает, или читается труднее? нипанимать в общем

коду нужен простор. а не лепить подряд строки друг к другу

? мы бы с тобой сработались) я коллег уже давно дрессирую не лепить)

да читается треднее

а те кто слепо копирует пср - у васчто? экраны маленькие?

это защита от дурака, смотри что происходит при комментировании //if () { //if {

дураков надо из команды гнать :)

эм, там ещё шторм красненьким подсветит

дурак должен быть клиническим идиотом

и я кстати комментирую if так иногда. иногда надо

и я кстати комментирую if так иногда. иногда надо

зачем?

ну протестить код без условия

букваьно на время

да читается треднее

да вот читается-то как раз легче) у тебя на 1 границе и открывается блок и закрывается. Ты это явно видешь. А не додумываешь сверху ещё 1 скобку

Show must go on

ау тебя наверно сишарповское прошлое тоже, да?

у мну?

угу

да вот читается-то как раз легче) у тебя на 1 границе и открывается блок и закрывается. Ты это явно видешь. А не додумываешь сверху ещё 1 скобку

Читается удобнее когда у всех код одинаково выглядит

там такойстандарт :)

со скобками

я не успел в нём особо упороться

Читается удобнее когда у всех код одинаково выглядит

он никогда не будет у всех одинаково выглядеть (глобально). а так, в проекте принимается cs, настраивается fixer и щасце есть

Я не возвожу все в абсолют

Просто многие думают что у них ОСОБОЕ виденье

а у меня такого не имеется, а читать приходится

ну у создателей любого стандарта ОСОБОЕ мнение)

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

давайте попросим заменить скобки на do/end

берите богомерзкий питон

там такой проблемы нет

Ы

Питон легче пхп