Динамический метод вызова с аргументами

Question

Для примера у меня есть класс с двумя методами:

class Example < ActiveRecord::Base 
    def method_one(value) 

    end 
    def method_two 

    end 
end 

и метод в контроллере, где я называю их:

def example 
    ex = Example.find(params[:id]) 
    ex.send(params[:method], params[:value]) if ex.respond_to?(params[:method]) 
    end 

Но проблема возникает, когда я пытаюсь вызвать method_two

ArgumentError (wrong number of arguments (1 for 0)) 

Это происходит потому, что params[:value] возвращается nil. Самое простое решение:

def example 
    ex = Example.find(params[:id]) 
    if ex.respond_to?(params[:method]) 
     if params[:value].present? 
     ex.send(params[:method], params[:value]) 
     else 
     ex.send(params[:method]) 
     end 
    end 
    end 

Интересно, есть ли какие-либо лучше обойти, чтобы не передать аргумент, если это нуль.

Answer 1

То, что вы пытаетесь сделать, может быть очень опасно, поэтому я рекомендую вам фильтровать params[:method] раньше.

allowed_methods = { 
    method_one: ->(ex){ex.method_one(params[:value])} 
    method_two: ->(ex){ex.method_two} 
} 
allowed_methods[params[:method]]&.call(ex) 

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

Я получаю только лямбда, если params[:method] находится в хеше allowed_methods.

&. синтаксиса является новым безопасным оператором навигации в рубине 2.3, и - для краткости - выполняет следующий метод, если приемник не ноль (т.е. результата allowed_methods[params[:method]]) Если вы не используете рубиновый> = 2.3 , вы можете использовать вместо try, которые имеют подобное поведение в этом случае:

allowed_methods[params[:method]].try(:call, ex) 

Если вы не фильтровать значение params[:method], то пользователь может просто передать :destroy, например, чтобы удалить запись, которая конечно, не то, что вы хотите.

Также, позвонив по номеру ex.send ..., вы обходите инкапсуляцию объекта, которой обычно не требуется.Чтобы использовать только общедоступный интерфейс, предпочитайте использовать public_send.

---

Другая точка на большой изъян безопасности вас код:

eval является частным методом, определенным на Object (фактически унаследованной от Kernel), так что вы можете использовать его таким образом на любом объекте:

object = Object.new 
object.send(:eval, '1+1') #=> 2 

Теперь, с кодом, представьте пользователь помещает eval как значение params[:method] и произвольный код рубина в params[:value], он может на самом деле whateve он хочет в вашей заявке.

Answer 2

Если вы понимаете, что вы делаете, есть более простые обходные пути:

def method_two _ = nil 
end 

или

def method_two * 
end 

Он работает, а наоборот:

def method_one *args 
end 
def method_two * 
end 

и:

ex.public_send(params[:method], *[params[:value]]) \ 
    if ex.respond_to?(params[:method]) 

Sidenote: предпочитает public_send над send, если вы явно не вызываете метод private.

---

Использование splatted Params без изменения сигнатуры методов:

ex.public_send(*[params[:method], params[:value]].compact)