Dynamische Methode mit Argumenten

Question

Zum Beispiel Aufruf Ich habe Klasse mit zwei Methoden:

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

    end 
    def method_two 

    end 
end 

und Verfahren in der Steuerung, wo ich sie nenne:

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

Aber das Problem kommt, wenn ich versuche method_two anrufen

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

Es geschieht, weil params[:value] kehrt nil. Die einfachste Lösung ist:

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 

Ich frage mich, ob es eine bessere Problemumgehung nicht Argument übergeben, wenn es null ist.

Answer 1

Was Sie versuchen können zu tun wirklich gefährlich, so empfehle ich Ihnen die params[:method] filtern, bevor.

Ich habe einen Hash definiert, der den Namen der Methode einem Lambda zuordnet, der die Methode aufruft, die Argumente und alle Sonderfälle behandelt, die Sie wollen.

Ich bekomme nur ein Lambda, wenn params[:method] in der allowed_methods Hash als Schlüssel ist.

Die &. Syntax ist der neue sichere Navigation Operator in Ruby 2.3, und - kurz - führt die folgende Methode, wenn der Empfänger nicht gleich Null ist (dh das Ergebnis allowed_methods[params[:method]]) Wenn Sie nicht ruby> = 2.3 unter Verwendung von Sie können try statt, die ein ähnliches Verhalten in diesem Fall benutzen:

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

Wenn Sie den Wert von params[:method] nicht filtern, dann ein Benutzer nur :destroy zum Beispiel passieren kann Ihren Eintrag zu löschen, das ist sicherlich nicht was du willst.

Durch den Aufruf ex.send ... umgehen Sie auch die Kapselung des Objekts, die Sie normalerweise nicht sollten.Wenn Sie nur die öffentliche Schnittstelle verwenden möchten, verwenden Sie vorzugsweise .

---

Ein weiterer Punkt auf der großen Sicherheitslücke von Ihnen Code:

eval ist eine private auf Object definierte Methode (eigentlich von Kernel geerbt), so dass Sie es auf diese Weise auf einem beliebiges Objekt verwenden können:

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

Jetzt, mit Ihrem Code, vorstellen, der Benutzer eval als der Wert von params[:method] und einem beliebigen ruby-Code in params[:value] setzt, kann er eigentlich gar whateve Er möchte in Ihrer Anwendung.

Answer 2

Wenn Sie verstehen, was Sie tun, gibt es einfachere Lösungen:

def method_two _ = nil 
end 

oder

def method_two * 
end 

Es funktioniert auch umgekehrt:

def method_one *args 
end 
def method_two * 
end 

und:

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

Nebennote: bevorzugen Sie über send, es sei denn, Sie rufen explizit private Methode.

---

Verwenden splatted params ohne die Methoden Signaturen zu ändern:

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