Rails-Formular-Validierung

Question

Ich habe eine Rails-App, mit der ein Benutzer eine Datenbankabfrage erstellen kann, indem er ein umfangreiches Formular ausfüllt. Ich fragte mich, die beste Praxis für die Überprüfung der Formularparameter in Rails. Zuvor habe ich meine results Methode (diejenige, an die das Formular gesendet wird), tun Sie Folgendes:

if params[:name] && !params[:name].blank?
  @name = params[:name]
else
  flash[:error] = 'You must give a name'
  redirect_to :action => 'index'
  return
end

Aber bei mehreren Formularfeldern wurde es lästig, dies für jedes einzelne Feld zu wiederholen. Ich konnte sie nicht einfach alle in eine Schleife stecken, um jedes Feld zu überprüfen, weil die Felder unterschiedlich eingerichtet sind:

• eine einzige Taste: params[:name]
• einen Schlüssel und einen Unterschlüssel: params[:image][:font_size]
• nur erwarten, dass einige Formularfelder ausgefüllt werden, wenn ein anderes Feld gesetzt wurde

Etc. Dies war auch wiederholend, weil ich die flash[:error] für jeden fehlenden/ungültigen Parameter und die Weiterleitung an dieselbe URL für jeden einzelnen. Ich bin zur Verwendung einer before_filter die alle erforderlichen Formularparameter überprüft und nur dann true zurückgibt, wenn alles in Ordnung ist. Dann wird die my results Methode wird fortgesetzt, und die Variablen werden einfach zugewiesen, ohne dass eine Überprüfung stattfindet:

@name = params[:name]

In meinem validate_form Methode, habe ich Codeabschnitte wie den folgenden:

if (
  params[:analysis_type][:to_s] == 'development' ||
  params[:results_to_generate].include?('graph')
)
  {:graph_type => :to_s, :graph_width => :to_s,
   :theme => :to_s}.each do |key, sub_key|
    unless params[key] && params[key][sub_key]
      flash[:error] = "Cannot leave '#{Inflector.humanize(key)}' blank"
      redirect_to(url)
      return false
    end
  end
end

Ich frage mich nur, ob ich dies am besten angehe oder ob ich etwas Offensichtliches übersehe, wenn es um die Parametervalidierung geht. Ich befürchte, dass dies immer noch nicht die effizienteste Technik ist, denn ich habe mehrere Blöcke, in denen ich einen Wert zuweisen flash[:error] , dann auf dieselbe URL umleiten und dann false zurückgeben.

Edit zur Klarstellung: Der Grund, warum ich diese Validierung derzeit nicht in den Modellen vornehme, ist zweierlei:

• Ich versuche nicht, Daten vom Benutzer zu sammeln, um eine Zeile in der Datenbank zu erstellen oder zu aktualisieren. Keine der Daten, die der Benutzer eingibt, wird nach der Abmeldung gespeichert. Sie werden direkt bei der Übermittlung verwendet, um die Datenbank zu durchsuchen und einige Daten zu generieren.
• Das Abfrageformular nimmt Daten auf, die sich auf mehrere Modelle beziehen, und es nimmt andere Daten auf, die sich überhaupt nicht auf ein Modell beziehen. So sind z. B. der Diagrammtyp und das Thema, wie oben gezeigt, nicht mit einem Modell verbunden, sondern vermitteln lediglich Informationen darüber, wie der Benutzer seine Ergebnisse anzeigen möchte.

Bearbeiten, um eine verbesserte Technik zu zeigen: Ich verwende jetzt anwendungsspezifische Ausnahmen, dank Jamis Buck's Den richtigen Ausnahmeartikel aufziehen . Zum Beispiel:

def results
  if params[:name] && !params[:name].blank?
    @name = params[:name]
  else
    raise MyApp::MissingFieldError
  end

  if params[:age] && !params[:age].blank? && params[:age].numeric?
    @age = params[:age].to_i
  else
    raise MyApp::MissingFieldError
  end
rescue MyApp::MissingFieldError => err
  flash[:error] = "Invalid form submission: #{err.clean_message}"
  redirect_to :action => 'index'
end

Answer 1

Sie könnten es mit active_form ( http://github.com/cs/active_form/tree/master/lib/active_form.rb ) - einfach ActiveRecord ohne den Datenbankkram. Auf diese Weise können Sie alle AR's Validierung Zeug verwenden und behandeln Sie Ihr Formular wie Sie jedes andere Modell würde.

class MyForm < ActiveForm
  validates_presence_of :name
  validates_presence_of :graph_size, :if => # ...blah blah 
end

form = MyForm.new(params[:form])
form.validate
form.errors

Answer 2

Sieht aus wie Sie tun die Validierung in der Steuerung, versuchen Sie es in das Modell, es ist besser geeignet, um diese Art von Sache.

Answer 3

Wenn Sie das Problem heute noch einmal angehen würden, könnten Sie ein Modell für den Abfrageparametersatz erstellen und die in Rails eingebauten Validierungen verwenden. Rails 3 macht dies mit ActiveModel::Validations viel einfacher, siehe diese Stelle .

Answer 4

Modell

class Person 
    include ActiveModel::Validations
    include ActiveModel::Conversion
    extend ActiveModel::Naming

    attr_accessor :name
    attr_accessor :email

    validates_presence_of :name,:message => "Please Provide User Name"
    validates_presence_of :email,:message => "Please Provide Email"
end

Beachten Sie, dass Sie das Modell nicht unbedingt speichern/bestehen lassen müssen, um es zu validieren.

Controller

@person.name= params["name"]
@person.email= params["email"]
@person.valid?

Wenn Sie die Methode .valid? für das Modell aufrufen, werden die Fehler in das @person-Objekt eingefügt. Daraus folgt,

Siehe

<%if @person.errors.any? %>
     <%@person.errors.messages.each do|msg| %>
          <div class="alert alert-danger">
            <%=msg[0][1]%>
          </div>
     <%end%>
<%end%>