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

ну правильно для первого роута будет всегда 404 если в базе нету page у которой slug = nil

!params[:category].nil? ? все тело метода заключено в блок if ? unless @category ?

!params[:category].nil? ? все тело метода заключено в блок if ? unless @category ?

можешь пожалуйста предложить свой вариант? про unless @category то даже не знаю что не так

@category = find_by_slug!(params[:category]) if params[:category].present?

!params[:category].nil? и unless @category - по сути то же двойное отрицание

хотя с unless спорный варинт, у меня в мозгу он звучит "если не категория" по моему гораздо проще понять условие "если категория есть" (if category.present?)

def define_category return if params[:category].nil? @category = ... render '404' if @category.blank? end самое простое что можно сделать

вообще рендера не должно быть в методе define_category либо метод должен называется render_defined_category

def define_category return if params[:category].nil? @category = ... render '404' if @category.blank? end самое простое что можно сделать

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

@category = find_by_slug!(params[:category]) if params[:category].present?

вот самое простое

я бы написал @category = find_by_slug!(params[:category) все

вообще рендера не должно быть в методе define_category либо метод должен называется render_defined_category

ну тут хитро, рендер стопорит исполнение основного екшна должно быть типа Category.find_by! который будет выбрасывать NotFound ексепшн а уже выше это будет ловиться и рендеить 404

ну значит стопор должен возникать в основном экшене

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

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

ну значит стопор должен возникать в основном экшене

вообще есть и нету категории это 2 разных случая которые почему то скрестили в 1 экшн. я уверен там еще и в теле экшна есть if @category ...

вот именно ?

о боже

что это за паскаль

турбо

спасибо Никите) @category = PortfolioCategory.find_by!(slug: params[:category]) if params[:category].present?

ну нельзя же так

1) метод который определяет категорию должен определять категорию, даже если нет параметра по которому нужно определять категорию 2) если такой параметр nil/blank/whatever то категории просто не будет - это условие не нужно 3) метод который определяет категорию не должен ничего знать об операциях которые нужно произвести, это не его обязанность - не нужно вносить путанницу 4) в потребляющем методе, которому нужна эта категория нужно написать return render 404 unless category, и это будет его законное место

Ух, опять пришел когда дьячук годные вещи говорит

ojab просто ленится сегодня

1) согласен 2) нет 3) согласен 4) вовсе не обязательно

я бы написал @category = find_by_slug!(params[:category) все

это не эквивалентно

можем вытащить записи, где значение поля удовлетворяет условию IS NULL

в то время как проверка present? такую возможность исключит

4 - зачем делать render 404, если есть механизм исключений?

можем вытащить записи, где значение поля удовлетворяет условию IS NULL

неконсистентность?

4 - зачем делать render 404, если есть механизм исключений?

ну хочет аффтар 404

это другой вопрос

Разве ActiveRecord::RecordNotFound не рендерится в 404?)

можем вытащить записи, где значение поля удовлетворяет условию IS NULL

published.find_by_slug так будет консистентнее

Разве ActiveRecord::RecordNotFound не рендерится в 404?)

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

аналогично, но на 95% уверен, что рендерится в 404)

значит вообще ничего не нужно писать кроме find_by!

обработка исключений работает медленнее чем render 404, это по сути главная разница )

подумаеш, куплю еще сервер ?

Да неужели)

И насколько же медленнее?)

где-то видел сравнение одинакового кода, где выбрасывался экспешен и ретурн по условию

разница там в сотни раз

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

просто вспомнилось

Замерю прямо сейчас

быть такого не может никак)

тут сравнивается с throw/catch, но суть ясна - https://jacopretorius.net/2012/01/catch-and-throw-in-ruby.html

разница менее чем в десять раз

по данным, приведенным в статье

я говорил про return а не про throw

+ ессно чем больше стектрейс тем медленее

1) метод который определяет категорию должен определять категорию, даже если нет параметра по которому нужно определять категорию 2) если такой параметр nil/blank/whatever то категории просто не будет - это условие не нужно 3) метод который определяет категорию не должен ничего знать об операциях которые нужно произвести, это не его обязанность - не нужно вносить путанницу 4) в потребляющем методе, которому нужна эта категория нужно написать return render 404 unless category, и это будет его законное место

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

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

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

4 - зачем делать render 404, если есть механизм исключений?

это на любителя

Не согласен

Если есть механизм для удобства, надо пользовать

так можно оправдать что угодно, любой неэффективный подход

не всегда эффективно == производительно

рельсы не для производительности

let's holiwar begin?))

*руби

ага

пока не выйдет 3 версия

Замерю прямо сейчас

замерил, никакой разницы

Имхо, есть механизм исключений, он очень гибкий. Не пользовать его в рельсах глупо. В конце концов, все эти find!, find_by!, find_by_field! сделаны специально для того, чтобы все было красиво и лаконично.

пока не выйдет 3 версия

Пока есть небольшие сомнения - уж больно амбициозные цели поставлены.

Подскажите, пожалуйста как лучше всего оформить тест, если у меня для разных моделей похожая валидация, только разный describe блок с ассоциациями?

ну там же изначально было что х3 по сравнению с 2 версией

Подскажите, пожалуйста как лучше всего оформить тест, если у меня для разных моделей похожая валидация, только разный describe блок с ассоциациями?

shared что то там

shared_examples_for ?

Подскажите, пожалуйста как лучше всего оформить тест, если у меня для разных моделей похожая валидация, только разный describe блок с ассоциациями?

может не стоит валидацию тестировать?

https://relishapp.com/rspec/rspec-core/docs/example-groups/shared-context

Стоит

может не стоит валидацию тестировать?

искушаете?)

для валидаций крутая штука shoulda matchers есть

а что за валидация такая, которую нельзя протестировать отдельно от модели?

один фиг непонятно, для чего предполагается юзать гемсеты.

Некрореплай ;) У нас гемсеты на билд машинах используются, чтобы держать разные версии инструмента, написанного на Ruby.

искушаете?)

да, каждый раз искушаюсь когда кто-то тестирует рельсовые валидации

А зачем отдельно? Сейчас в разных моделях валидации дублируются, потому что люди любят писать валидаторы отдельно, вот и нужно проверить

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

Пойдут люди по такой дорожке как вы предлагаете

И сиди потом разгребай

я тогда что-то не понимаю

нужно проверить что у модельки есть валидация? протестировать валидацию? или что?

ну вдруг рельсовая валидация не работает!

Простетировать что валидация выводит нужный текст в нужном количестве

Это так важно?

Ладно, только время тратить, одни тролли какие-то Спасибо, @vasivas

Некрореплай ;) У нас гемсеты на билд машинах используются, чтобы держать разные версии инструмента, написанного на Ruby.

И что мешает использовать bundler вместо гемсетов?

Уважаемые доны тебе помочь всей душой хотят, а ты в троллинге обвиняешь. Не хорошо.

И что мешает использовать bundler вместо гемсетов?

bundler умеет как rvm-exec? Запустить конкретную версию гема и при этом учесть версии гемов-зависимостей?

вродь как да

bundler умеет как rvm-exec? Запустить конкретную версию гема и при этом учесть версии гемов-зависимостей?

он для этого и создан, версии вместе с зависимостями в Gemfile.lock лежат

Всем привет, какой-то затуп. Можно как-то получить одним запросом, допустим, всех юзеров с уникальными именами?

UNIQUE?