DG
а в чем смысл под ковер убрать говно?
Size: a a a
2019 November 28
DG
Так я буду знать что там есть проблема. А если директивами обрамлю и правило для пути отключу то я просто закопаю проблему
AI
а в чем смысл под ковер убрать говно?
Объяснить линтеру что это не гавно
DG
Хотя это и не проблема на самом деле
AI
Линтер тупой, ты умный
k
Как вот такую ссанину можно отрефакторить красиво?
def scope
scope = AppCreationLog.all
scope = scope.bundle_like(query) if query
scope = scope.greater_than_limit if greater_than_limit
scope
end
можно использовать ламбда композицию, правда это трудно читаться будет
DG
та мложно было и public_send заюзать. Но тоже решение сомнительное
VL
Как вот такую ссанину можно отрефакторить красиво?
def scope
scope = AppCreationLog.all
scope = scope.bundle_like(query) if query
scope = scope.greater_than_limit if greater_than_limit
scope
end
Этот метод очень много на себя берет, может имеет смысл сделать много супер тупых методов и потом уже вызывать цепочками Класс.мини_метод1.мини_метод2
В другом месте Класс.мини_метод2.еще_что_то_там
В другом месте Класс.мини_метод2.еще_что_то_там
МВ
Мне вообще кажется что там фичур енви
2019 December 02
M
Как вот такую ссанину можно отрефакторить красиво?
def scope
scope = AppCreationLog.all
scope = scope.bundle_like(query) if query
scope = scope.greater_than_limit if greater_than_limit
scope
end
Можно начать с такого - линтер должен быть доволен -
def scope
return scope.bundle_like(query) if query
return scope.greater_than_limit if greater_than_limit
AppCreationLog.all
end
def scope
return scope.bundle_like(query) if query
return scope.greater_than_limit if greater_than_limit
AppCreationLog.all
end
DG
А если два условия соблюдены
M
вообще я не прав. так не правильно
M
def scope
scope = AppCreationLog.all
return scope.bundle_like(query) if query
return scope.greater_than_limit if greater_than_limit
scope
end
Вот так должно работать.
scope = AppCreationLog.all
return scope.bundle_like(query) if query
return scope.greater_than_limit if greater_than_limit
scope
end
Вот так должно работать.
DG
Не, так тоже не будет. Нет возможности сразу 2 фильтра применить
M
Не, так тоже не будет. Нет возможности сразу 2 фильтра применить
Тогда можно как Alex предлагал - без последнего scope, хоть линтер успокоится.
def scope
scope = AppCreationLog.all
scope = scope.bundle_like(query) if query
scope.greater_than_limit if greater_than_limit
end
def scope
scope = AppCreationLog.all
scope = scope.bundle_like(query) if query
scope.greater_than_limit if greater_than_limit
end
AA
Тогда можно как Alex предлагал - без последнего scope, хоть линтер успокоится.
def scope
scope = AppCreationLog.all
scope = scope.bundle_like(query) if query
scope.greater_than_limit if greater_than_limit
end
def scope
scope = AppCreationLog.all
scope = scope.bundle_like(query) if query
scope.greater_than_limit if greater_than_limit
end
У тебя Нил вернётся
M
ага, точно
DG
Для меня это оверинжиниринг получится