Ruby: verschachtelte if-Anweisungen

War ich ein wenig code schreiben, und es endete als viel zu hässlich für meinen Geschmack. Gibt es trotzdem, ich kann es umgestalten, so dass ich don ' T verwenden Sie verschachtelte if-Anweisungen?

def hours_occupied(date)
  #assuming date is a valid date object    
  availability = get_work_hours(date)
  focus = "work"

  if availability.nil
    availability = get_family_hours(date)
    focus = "family"

    if availability.nil
      availability = get_friend_hours(date)
      focus = "friends"
    end
  end
end

Ich weiß, ich werde in der Lage sein, etwas zu tun, wie dies für die Verfügbarkeit

availability = get_work_hours(date) || get_family_hours(date) || get_friend_hours(date)

aber wie kann ich die Fokus-variable entsprechend?

  • Können wir annehmen, dass Sie nicht zeigen, die komplette Methode? ansonsten macht es keinen Sinn zu ordnen diesen ungenutzten Variablen. Wenn das der Fall ist, fügen Sie "..." als Platzhalter an der Unterseite der Methode.
  • Finde ich mich völlig unfähig zu understande das Thema und den Zweck seines code, das macht mich unfähig zu reagieren.
InformationsquelleAutor rlhh | 2013-06-19
  1. 1

    Eine weitere Möglichkeit ist, nur Werte zuweisen, wenn es notwendig ist:

    def hours_occupied(date)
  availability, focus = get_work_hours(date), "work"
  availability, focus = get_family_hours(date), "family" unless availability
  availability, focus = get_friend_hours(date), "friend" unless availability
end

    oder über einen iterator:

    def hours_occupied(date)
  availability = focus = nil
  %w(work family friend).each {|type| availability, focus = self.send(:"get_#{type}_hours", date), type unless availability}
end
    InformationsquelleAutor trushkevich
  2. 5

    Ich würde etwas tun, wie die folgenden es, macht deutlich, dass jeder Fall ist gegenseitig:

    def hours_occupied(date)
  if availability = get_work_hours(date)
    focus = "work"
  elsif availability = get_family_hours(date)
    focus = "family"
  elsif availability = get_friend_hours(date)
    focus = "friends"
  end
end
    • Ruby-Bedingungen sind Ausdruck, so dass keine Notwendigkeit zu wiederholen focus 3 mal
    InformationsquelleAutor Don Cruickshank
  3. 3

    Ich würde schreiben:

    def hours_occupied(date)
  focus = if (availability = get_work_hours(date))
    "work"
  elsif (availability = get_family_hours(date))
    "family"
  elsif (availability = get_friend_hours(date))
    "friends"
  end
  # I guess there is more code here that uses availability and focus.
end

    Aber ich bin nicht sicher, mit verschiedenen Methoden für die verschiedenen Typen ist eine gute Idee, es macht den code schwerer zu schreiben. Ein anderes Konzept mit Enumerable#map_detect:

    focus, availability = [:work, :family, :friends].map_detect do |type|
  availability = get_hours(date, type)
  availability ? [type, availability] : nil
end
    • Ich nie sehen, wie Zuordnungen im bedingten test. Es ist zu viel, wie C oder Perl und gibt mir Albträume. Vielen Dank. 🙂
    • Du bist herzlich willkommen 🙂 Einige Leute würden sagen, dass, wenn die Sprache ermöglicht es (zum Beispiel Python nicht), warum soll man es meiden? kann man nicht auch sagen a == aus a =? Aber das werde ich nicht 🙂 ich mag diese Bauweise, weil es vereinfacht den code, mit weniger verschachtelten Ausdrücken. Beachten Sie, dass die Klammern machen es mehr conspicous.
    InformationsquelleAutor tokland
  4. 1

    Einen Fall, wenn ist auch eine option:

    focus = case availability
when get_work_hours(date)
  "work"
when get_family_hours(date)
  "family"
when get_friend_hours(date)
  "friends"
end
    • entweder das ist falsch oder case anders funktioniert als ich dachte 🙂
    • Beispiel mit dummy-Daten hier
    • aber availability muss die erste nicht-null Ergebnis, der die Methode aufruft, es ist vorher nicht bekannt.
    • Verwenden Sie einfach case, nicht case availability
    • +1 @Stefan, das sieht aus wie der richtige Weg ist.
    • Jungs, ich bin ein bisschen verwirrt durch Kommentare, prüfen Sie die Frage, availability ist nicht bekannt! oder bin ich etwas fehlt?
    • das ist, warum Sie haben zu verwenden case ohne parameter: focus = case when get_work_hours...
    • Ja Stefan, aber mit einer Zuordnung in die einzelnen when, so ist es nicht absolut gleichwertig zu einer if-Kette?
    • Du hast Recht, die OP muss beide Variablen.

    InformationsquelleAutor steenslag
Schreibe einen Kommentar