Schrumpfen Ruby on Rails IF-ELSE-code

Wie kann ich verkleinern Sie den folgenden code.. Kann es getan werden in einer Zeile statt?

<% if pos_count < 0 %>
  <% pos_name = "SHORT" %>
  <% else %>
  <% pos_name = "LONG" %>
<% end %>

Dank!

InformationsquelleAutor Twiddr | 2012-09-27
  1. 14

    Kann man das in einer Zeile mit dem Bedingungsoperator (es ist ein ternärer operator):

    <% pos_name = (pos_count < 0) ? "SHORT" : "LONG" -%>
    • Perfekt.. genau das, was ich neeeded.. danke..
    InformationsquelleAutor KARASZI István
  2. 1

    Sieht so aus Sicht der Logik und der besten Praxis zu minimieren. Ich würde vorschlagen, extrahieren diese in ein view-helper (oder Dekorateur, wenn Sie bevorzugen).

    Als Istvan vorgeschlagen, können Sie den ternären operator, aber für die Lesbarkeit würde ich einfach lassen, wie es if-else-Anweisung und verschieben Sie es in eine Methode, etwa so:

    app/helpers/my_view_helper.rb

    module MyViewHelper
  def pos_name(pos_count)
    if pos_count < 0
      "SHORT"
    else
      "LONG"
    end
  end
end
    • +1 auf das extrahieren der anzeigen von Logik in den view Helfern. Als das Projekt wächst, anzeigen, Moderatoren können auch verwendet werden: railscasts.com/episodes/286-draper?autoplay=true
    • Die klingen wie gute Praxis.. Kann ich dann einfach anrufen, die Methode, wie dies in der Ansicht: pos_count.pos_name ?
    • Würden Sie das nennen wie pos_name(pos_count) aus der Sicht. Ich habe aktualisiert, der code oben angepasst.
    InformationsquelleAutor gylaz
Schreibe einen Kommentar