ну если легко читается, то вай нот

практика подсказывает, что: 1. это придется расширять 2. расширяя, придется всю метомагию понимать

или если сломается.. ух помню коллега до меня пилил mongo-like query language для SQL базы

или так

вот это было как в истории про батю и борщ

одним словом, я перестраховываюсь и пишу явно

практика подсказывает, что: 1. это придется расширять 2. расширяя, придется всю метомагию понимать

Я думал об этом у меня была достаточно узкая задача и расширению никак не мешпла

или копирую

практика подсказывает, что: 1. это придется расширять 2. расширяя, придется всю метомагию понимать

Это как везде, если хорошо не умеешь( а расширение, это 2 принцип солида) то лучше не делать. С этим соглашусь

И все было Реал ясно просто был модуль и ты его подключал к модели

а еще любой такой гениальный код, если он действительно гениальный и очень сложный, должно быть легко выкинуть в случае необходимости

Типа сортаьл и сервабл

Серчабл и сортабл*

че че о чем трёп

че че о чем трёп

О том как я ранее изобретал

Помнишь?

да

и я тебе говорил что говна навернёшь, помнишь?

так всё ок же

не навернул

просто обсуждаем типа норм ли впринципе метапрограммирование

я то там всё сделал чтоб не навернуть

типа моя штука ну вообще никак не мешает другим методам работы с таблицами

надо просто доку нормальную оставлять

типа какие методы генерятся

хотя бы в виде комментариев в коде

ДА

да*

я обычно если сильная метамагия с define_method - справа в виде комментария построчно пример набрасываю

чтоб хоть что-то одно нагрепалось =)

ДА

Ну что ты решил задачу? Если нет, заново поставь вопрос в стиле Дудя или Познера. Просто интересным показался твой вопрос.

решил конечно

пользуемся

ток фронт не доделан был

поэтому проект пока отложили

пушо дедлайн пройден

поэтому опыта в проде этой штуки нет

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

плюс прослоечка в аппликейшн_контроллер которая парсила входящие параметры если они шли на индекс и автоматически вызывала нужный сгенерирвоаный метод поиска или сортировки

мы с фронтом договорились что когда на индекс идёт типа можно чтоб было два массива параметров - сёрч парамс и сорт парамс

в сёрч парамс могли приходить названия столбца и тип поиска

помню пилил шутку, которая фильтровала данные как в гитхабе

т.е. author:name label:test search text

типа если приходило айди - возвращали с нужным айди если айдис - то возвращали айди с от и до нужных если айдис_каунт - то количество с таким то айди и тд

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

ну и объект, который все это оборачивает

а у меня в итоге 2 модуля и пара методов в апликейшн контроллере

что изменил бы ща - обернул бы методы в апликейшн контроллере тоже

тогда чёт не дошло в голову

да и вообще вопрос да

надо ли такому быть в апликейшн контроллере

типа по сути это был не экшн на который шлётся запрос

не думаю

а логика парса параметров

я бы отдельный сделал объект с логикой

как было бы правильнее?

ну вот сделал бы объект

и его в экшен пихал

а потом как?

подключал бы его к апликешйн контроллеру?

не было ли бы это считай тем же самым?

я вот так делал и то, считаю, что это не правильно https://github.com/davydovanton/octostar/blob/master/apps/web/controllers/repos/index.rb#L12

А пушо это шо ?

потому что

ребята, подскажите пожалуйста, почему если генерю xlsx - он не открывается. сделал как http://railscasts.com/episodes/362-exporting-csv-and-excel?view=asciicast ругается на формат файла. а если делаю xls - опять предупреждение : действительный формат отличается от того что указано в имени файла (xls). подскажите пожалуйста, как избавиться от предупреждения?

:)

я вот так делал и то, считаю, что это не правильно https://github.com/davydovanton/octostar/blob/master/apps/web/controllers/repos/index.rb#L12

ну а вот если бы я хотел это сделать дефолтным поведением у всех ээкшнов индекс

во все бы и передавал бы

явно

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

ну да

вот тоже хороший вопрос

наверное лучшим вариантом тогда

было бы делать как ты

но

сделать генератор контроллера кастомный

зачем?

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

явно пишешь контроллеры все и все

ну просто у меня в црм почти все контроллеры это круды

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

с повторяющейся логиков

просто вызов объекта + обработка результата

ну он и не будет толстым

скопируешь 4 строчки и все, в чем беда?

я же говорю

там просто в индексе вместо def index render Model.all end

ребята, подскажите пожалуйста, почему если генерю xlsx - он не открывается. сделал как http://railscasts.com/episodes/362-exporting-csv-and-excel?view=asciicast ругается на формат файла. а если делаю xls - опять предупреждение : действительный формат отличается от того что указано в имени файла (xls). подскажите пожалуйста, как избавиться от предупреждения?

Файл как заканчивается (тип какой)?

будет типа def index render Searchable.search(Model, search_params)

и типа так бы и генерировал бы их сразу

ну по сути реал ток 2 строчки поправить руками

но можно же и автоматизировать

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

так почему не сделал бы вот так def index result = Searchable.search(Model, search_params) if result # success case else # failure case end end

и все

да можно и так я не против

надеюсь возьмут

постулаты ооп... нихуя себе

ну это реал самая база на самом деле

просто теор часть не так часто же в голове держишь

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