present? - это обертка над !empty?

сорян, попутал с утра. Везде, где написано empty?, имелся в виду blank?

Люди как правильно сделать вот такое В контроллере почти в каждом action должен быть переменная из сессии вобще логика такая в макете сверху есть селект, и это селект устанавливает определенное значение в сессии и каждый action в контроллерах должен получать значение этой переменной но проверять в каждом след образом if session[:myvar].nil? return end myvar = session[:myvar] как то муторно

return ?

а где это, в отдельной функции где-то?

или во всех экшенах??

Сделай в Application Controller хук на before action, который это делает, а там, где он не нужен делай скип

заибись

делать хуки в Application Controller

а потом придумывать, как обойти действие этого хука

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

То проще описать исключения

а потом будет надо почти не везде

а осадочек останется

Ну надо будет - отрефакторит

В чем проблема?

в костылях в перспективе

ты, кажется, себя позиционировал как "не веб-макаку"

Предложи решение лучше

а легче сказать про макаку, чем предложить решение)

не правда ли? )

а потом придумывать, как обойти действие этого хука

для этого есть skip_before_action, непонятно что нужно придумывать

можно сделать пустой контроллер с коллбэком before_action. все контроллеры в которых нужен данный функционал наследовать от него.

Предложи решение лучше

не использовать хук, выделить код проверки

а легче сказать про макаку, чем предложить решение)

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

)

несложно думаю)

можно сделать пустой контроллер с коллбэком before_action. все контроллеры в которых нужен данный функционал наследовать от него.

ООП головного мозга

не использовать хук, выделить код проверки

И разве хук это не выделенный код?

И разве хук это не выделенный код?

выделенный. В хук. Не надо выделять код в хук, котрый неявно вызывается

Окей, он выделил метод, дальше что?

а дальше его использует

Вызывает в каждом экшне?

да

Т.е. ты предлагаешь заменить DRY rails решение на копипаст кода?

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

а на вызов метода

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

И чем это лучше чем написать хук, для которого описать исключения?

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

где-то там

Т.е. по твоей логике вообще нельзя пользоваться хуками, потому что у них якобы неявный вызов?

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

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

+1 за явный вызов метода set_var_if_available!

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

хотя, если там речь только об одном контроллере, то можно и экшеном

если нужно в нескольких контроллерах, то лучше явным вызовом метода

иначе потом в этот метод добавятся всякие проверки, чтобы оно не валилось на nil

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

то бишь стандартный devise'овский before_action :authenticate_user! лучше заменять на явный вызов метода, например?

нет, зачем?

точнее, зависит от контекста

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

выше речь как раз о before_action'ах, которые должны выполняться практически во всех контроллерах

если аутентификация нужна только в 3х экшенах, из 10, то можно и руками)

:authenticate_user! как раз так и работает, например

Но в сообщении то и сказано, что у него почти во всех экшнах

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

так что с before_action :authenticate_user! делать? Менять на явный вызов?

Т.е. по твоей логике вообще нельзя пользоваться хуками, потому что у них якобы неявный вызов?

я вроде ясно сказал, что хук в контроллере, от которого наследуются остальные - э то нехорошо

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

жиза

так что с before_action :authenticate_user! делать? Менять на явный вызов?

оу, это в сторону @vskopuk должно было быть, промазал

И лучше писать явный вызов метода везде, где он используется, даже если этот метод вызывается в 80% экшнов в разных контроллерах?

80% это как пример

так что с before_action :authenticate_user! делать? Менять на явный вызов?

в контроллере, агась

И лучше писать явный вызов метода везде, где он используется, даже если этот метод вызывается в 80% экшнов в разных контроллерах?

именно

ERROR: S client not available

Окей

Но я все таки продолжу делать на хуках

И скипать их там, где не нужно

да хоть на жаваскрипте пиши, хехехехех

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

Чем много раз вызывать этот же код вручную

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

если before_action :authenticate_user! написано в ApplicationController, то это не очень хорошо и даже дерьмово

вроде об этом же речь

а не о том, что before_action нельзя использовать

Чем много раз вызывать этот же код вручную

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

если before_action :authenticate_user! написано в ApplicationController, то это не очень хорошо и даже дерьмово

согласен. я как минимум на каждый неймпспейс создаю base_controller.rb и в них уже добавляю необходимое. App Controller по минимуму трогаю

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

дада, "просто отрефакторю"

been there

Окей, чувак. Я не буду с тобой спорить. Попробуй в следующий раз не переходить на личности и использовать аргументы.

это же рор чат

вот так

Окей, чувак. Я не буду с тобой спорить. Попробуй в следующий раз не переходить на личности и использовать аргументы.

иззинити, про веб-макак - это же не я придумал

отвлекитесь посмейтесь https://ebanoe.it/2017/04/02/wasted-life-in-it/

)

отвлекитесь посмейтесь https://ebanoe.it/2017/04/02/wasted-life-in-it/

@ruby_talks

бггг

))

не ждали? )

привет всем)

Ребят, подскажите для еженедельных отчетов, есть смысл использовать Date.commercial для определяния времени начала и конца периода? или проще стандартными манипуляциями с датой воспользоваться

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

недельные — пн-вс или по номеру недели в году?

я думал хранить данные в виде номеров недель так проще будет