ну мол у нас процессы выстроены таким образом, что мы сначала прописываем полностью как должно работать и уже потом делаем)

бывают очень сложные кейсы что сразу и не сообразишь

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

бывают очень сложные кейсы что сразу и не сообразишь

не ну это понятно, у меня проекты по проще все ж

но ведь не факт что без устранения дублирования получилось бы лучше)

пока выходит что было бы лучше.

ну и плюс типичный проект который скатывается в болото где решения принимаются по факту появления проблем и потом через месяц уже забывают что такое было и оформляют это как багу)

)

пока из кода не достанешь коммит и привязанный к нему тикет откуда пришло "это"

хорошо если разработчик еще глянет гит блэйм

а то бывает тупо выкашивают то что нужно было

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

а то бывает тупо выкашивают то что нужно было

если это в коде было не очевидно, тогда это уже проблемы того кто это писал)

есть такое

@fes0r почему наследование = рак? часто прослеживаю последнее время такое мнение.

https://habrahabr.ru/post/177447/

@fes0r почему наследование = рак? часто прослеживаю последнее время такое мнение.

потому что перегрузки методов нету

приходится раковать - общий класс с методом-оберткой + классы для использования внутри с реализациями

люто не люблю так делать, но приходится

либо массив пихать

тогда сидишь без тайпхинтинга

Перегрузка методов - этож Class User { public function getFullName(){ return 'atata';} } Class AwesomeUser extends User { public function getFullName(){ return 'awesome atata';} }

что с ней не так?

https://habrahabr.ru/post/177447/ спасиб, почитаю

перегрузка метода - это когда в метод передается разное количество переменных или разные их типы или и то и другое

и в зависимости от этих условий, в методе выполняется то или иное действие

значит напутал, буду курить

как тут view сделать для кода?

`

короче три апострофа

и в конце три апострофа

вначале и вконце листинга

class test { //todo smt }

спс

потому что перегрузки методов нету

чушь, она и не нужна

перегрузка метода - это когда в метод передается разное количество переменных или разные их типы или и то и другое

перегрузка методов нужна только в языках со статической системой типов, потому что надо табличку виртуальных методов сделать в момент компиляции. В языках с динамической системой типов и переменным числом аргументов ad-hoc полиморфизм есть из коробки и надо всего-лишь в теле метода сделать if. Ну и надо это обычно не так часто, чаще лучше вооружиться double dispatch и только изредка возникают необходимости проверять типы

в целом наследование типов - норм (если юзать implements мол), а вот наследование классов в 90% случаев не нужно и оверюзится.

делаем все классы final и если надо вводить абстрактные типы - интерфейсы

>надо всего-лишь в теле метода сделать if

а если у тебя 12 вариантов?

12 if?

А у тебя может быть 12 вариантов?)

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

или хотя бы 5

да че уж, хотя бы 3

есть классический пример с принтерами и фигурами, где нужно что-то типа double dispatch + перегрузка методов

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

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

да че уж, хотя бы 3

любая внешняя либа где тупой автор додумался завязаться на конкретном классе в методе

laravel eloquent

самый последний случай

> где тупой автор давай не будем рассматривать ситуации где кто-то тупой

а они все такие

ты всегда все можешь завернуть в свои объекты со своими интерфейсами

а они все такие

ну как бы язык программирования тут не поможет) и фичи тоже)

так у меня уже eloquent завернут в мои объекты

но его-то надо расширять

кого "его"?

элоквент? зачем?

ну и что значит "расширять" в твоем понимании

грамматику свою добавлять

> грамматику свою добавлять я не в теме, это как?

ну типа drop cascade в либе нету

для postgres очень полезная фича

ээм...

и... причем тут перегрузка методов?)

ERROR: S client not available

а ща покажу

а вообще вместо того что бы делать эти вещи через eloquent и расширять его, сделал бы table gateway и не парился бы

и юзал бы dbal + sql

а ща покажу

жду)

интересно стало)

воо, соседний пример в коде нашел

есть в eloquent distinct

/** * Force the query to only return distinct results. * * @return $this */ public function distinct() { $this->distinct = true; return $this; }

ну то есть ты решил понаследовать классы)

для "расширения"

который в select обрабатывается очень примитивно

/** * Compile an aggregated select clause. * * @param \Illuminate\Database\Query\Builder $query * @param array $aggregate * @return string */ protected function compileAggregate(Builder $query, $aggregate) { $column = $this->columnize($aggregate['columns']); // If the query has a "distinct" constraint and we're not asking for all columns // we need to prepend "distinct" onto the column name so that the query takes // it into account when it performs the aggregating operations on the data. if ($query->distinct && $column !== '*') { $column = 'distinct '.$column; } return 'select '.$aggregate['function'].'('.$column.') as aggregate'; }

ну ок ладно

а в postgres есть такая штука как distinct on

и причем тут перегрузка методов?

и eloquent не работает когда есть агрегационная функция + distinct

и причем тут перегрузка методов?)

первый случай

/** * Compile the "select *" portion of the query. * * @param Builder $query * @param array $columns * @return string|null */ protected function compileColumns(Builder $query, $columns) { // If the query is actually performing an aggregating select, we will let that // compiler handle the building of the select clauses, as it will need some // more syntax that is best handled by that function to keep things neat. if (!is_null($query->aggregate)) { return; } $select = $query->distinct ? $query->distinctColumns ? 'select distinct on (' . $this->columnize($query->distinctColumns). ') ' : 'select distinct ' : 'select '; return $select . $this->columnize($columns); }

моя обработка compileColumns

на самом деле туда приходит MyBuilder $query

который наследуется от Builder

но написать я так не могу

потому что пхп будет ругаться

потому что идиоты завязались не на интерфейсе

а зачем тебе так писать?)

чтобы мои методы подсвечивались в тайпхинтинге

вторая часть

про distinct

/** * Force the query to only return distinct results. * * @return $this */ public function distinct() { $this->distinct = true; return $this; }

пришлось добавлять огород с другой функцией