rubocop -a

и указать какой cop юзать

кстати да, или rubocop.yml отредактировать

посоны, кстати, насчет экранирования все таки. (" ? ", some) vs "#{some}"

что лучше и почему?

через ? очевидно же

чем? проще писать?

#{} это не экранирование, это интерполяция

а рельса при наличии плейсхолдеров ? санитайзит параметры

окак. т.е это таки РАЗНЫЕ вещи?

да

? придуман от sql injection

тоесть одно впиливается в строку как есть, второе чистится от говна и тоже влетает в строку?

и интерполяцией я sql не защищу, получается

” a + #{foo}” это просто синтаксический сахар к “a + ” << foo.to_s

ххм @article = policy_scope(Article).find_by(id: "?", params[:id])

получается так

нет

тут ничего делать не надо

это касается только мест где ты собираешь сырой sql

потом оно так и не сработает

интерполяция строк это синтаксический сахар к конкатенации?

ооок

не, конкатенация дольше

ооок

вообще, да, посмотри байткод

кек

не, так это у меня в set_article в его поиске по /article/"drop table"

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

в before_action set_article, only [:edit, :new]

не, так это у меня в set_article в его поиске по /article/"drop table"

нет, `where`/`find`/и прочее при указании параметров хешем всё само экранирует

str1 + str2 + str3 это ж дольше, чем "#{str1}#{str2}#{str3}"

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

да

потому в первом случае руби возвращает str1+ str2, потом добавляет к этому str3 и возвращает

спасибо, бать

нет, `where`/`find`/и прочее при указании параметров хешем всё само экранирует

тоесть я могу не морочиться и оставить как есть, и мой типа патчик на введение экранирования в дефолтный генератор и set_model ... before ... получается не нужен?

пиши ещё

RubyVM::InstructionSequence.dissam

отетот вот - policy_scope(Article).find_by("id: ?", params[:id])

policy_scope(Article).find_by(id: params[:id])

и всё будет ок

https://v1.std3.ru/4a/60/1462382038-4a603f3dda3d1038951332d7f1c84131.png https://v1.std3.ru/c7/5b/1462382038-c75bd3ab629436bd4999b077857ca211.png

да откуда эта херь влетает то

https://v1.std3.ru/4a/60/1462382038-4a603f3dda3d1038951332d7f1c84131.png https://v1.std3.ru/c7/5b/1462382038-c75bd3ab629436bd4999b077857ca211.png

а теперь сравни примеры про str1, str2, str3

https://gist.github.com/ojab/b2cadf2b30ad18f74f959c2ff252e581

ой, блядь

этот срач

нет, не хочу

твой бенчмарк говно, бтв

чойто?

https://www.refheap.com/118673

потому что ты там не учитываешь три миллиона вещей

например?

размер строки, например

вон смотри выше

https://www.refheap.com/118673

тут даже строки не заморожены

там рассово верный бенчмарк

о чём дальше можно говорить?

лол

COUNT.times

о реальной производительности

да-да

можешь на ips переписать

там форк есть

будет ещё более рассово верно

замораживать строки это нечестно

ERROR: S client not available

а, строки заморожены, да

чото я проглядел

блин. хочу layout в зависимости от логина запилить, а мне "undefined method `user_signed_in?' for ApplicationController:Class"

блин. хочу layout в зависимости от логина запилить, а мне "undefined method `user_signed_in?' for ApplicationController:Class"

Devise?

угу

кстати охуительная задача, динамически _объявить_ в биндиге lvar

без eval

(можно и eval)

did u restart ur server ?

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

так чтоб оно ниже по скопу было доступно

у меня сходу не получилось

угу

сделай layout devise.html.что-либо и devise его заюзает для незалогиненных

lvar это что

local variable?

там рассово верный бенчмарк

так это, что в нём расово верного окромя размера строк?

количество операндов например

а фиг

сделай layout devise.html.что-либо и devise его заюзает для незалогиненных

эт как

эт как

app/views/layouts/devise.html.erb

размер строки, операнды, работа с lvar

потому что константу ещё резолвить надо

layout 'guest' unless user_signed_in?

а lvar резолвить не надо, вот он в биндинге прямо лежит

layout 'guest' unless user_signed_in?

оно само подхватит

зачем вы в бенчмарке строки фризите7

оно само подхватит

оно метод найти не может

какой подхватит

undefined method `user_signed_in?' for ApplicationController:Class

зачем вы в бенчмарке строки фризите7

чтоб время аллокации не мерять

оно метод найти не может

убери нафиг это из ApplicationController'a и сделай app/views/layouts/devise.html.erb, devise будет показывать страницы в этом layout'e для незалогиненных пользователей