- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
module AuthenticatedSystem
protected
def logged_in?
!!current_user
end
def current_user
@current_user ||= login_from_session unless @current_user == false
@current_user
end-
Ruby / Говнокод #7532
−96
Запостил:
bebesuk,
12 Августа 2011
Follow us!
логика !!current_user на руби особенно ущербна, с тем же успехом можно возвращать тупо current_user, т.к. все равно че угодно кроме nil и false сойдет за true в булевом контексте
однако, @current_user ||= login_from_session unless @current_user == false, имеет право на жизнь, но только в случае если login_from_session возвращает false как флаг ошибки или вроде того ( си стайл епта )
то есть код current_user может быть не говном - но только в случае, если код login_from_session таки им является
говно порождает говно, однако
nil && true вернет nil, а не false. А вот это уже далеко не всегда желательно
если же тупо проверять something в булевом контексте ( if; unless ), то разницы между nil и false никакой нет
в данном случае, скорее всего, имеет место быть "оптимизация", чтобы не дергать login_from_session лишний раз после фейла оного, но это я тоже считаю говном, т.к. гораздо логичнее использование какого-нибудь AnonymousUser
И всё обернуть if-ами. В case-е.