Wenn die Zeichenfolge leer ist, geben Sie einen Standardwert zurück

Oft muss ich überprüfen, ob ein Wert leer ist, und Folgendes schreiben: "Keine Daten vorhanden":

@user.address.blank? ? "We don't know user's address" : @user.address

Und wenn wir ungefähr 20-30 Felder haben, die wir auf diese Weise verarbeiten müssen, wird es hässlich.

Was ich gemacht habe, ist eine erweiterte String-Klasse mit orMethode

class String
  def or(what)
    self.strip.blank? ? what : self
  end
end

@user.address.or("We don't know user's address")

Jetzt sieht es besser aus. Aber es ist immer noch roh und rau

Wie wäre es besser, mein Problem zu lösen. Vielleicht wäre es besser, die ActiveSupport classHilfsmethode oder Mixins oder irgendetwas anderes zu erweitern oder zu verwenden. Welche Ruby-Idealogie, Ihre Erfahrung und Ihre Best Practices mir sagen können.

ActiveSupport fügt presenceallen Objekten eine Methode hinzu , die ihren Empfänger zurückgibt, wenn present?(das Gegenteil von blank?), und nilansonsten.

Beispiel:

host = config[:host].presence || 'localhost'

Phrogz gab mir die Idee in PofMagicfingers Kommentar, aber was ist mit dem Überschreiben von | stattdessen?

class String
  def |(what)
    self.strip.blank? ? what : self
  end
end

@user.address | "We don't know user's address"

Da Sie dies in Ruby on Rails tun, scheint es, als würden Sie mit einem Modell arbeiten. Wenn Sie überall in Ihrer App einen angemessenen Standardwert wünschen, können Sie beispielsweise die addressMethode für Ihr UserModell überschreiben .

Ich kenne ActiveRecord nicht gut genug, um dafür guten Code bereitzustellen. in der Fortsetzung wäre es so etwas wie:

class User < Sequel::Model
  def address        
    if (val=self[:address]).empty?
      "We don't know user's address"
    else
      val
    end
  end
end

... aber für das obige Beispiel scheint dies, als würden Sie die Ansichtslogik in Ihr Modell mischen, was keine gute Idee ist.

Ihre oder Methode kann einige unerwünschte Nebenwirkungen haben, da der alternative (Standard-) Wert immer ausgewertet wird, auch wenn die Zeichenfolge nicht leer ist.

Beispielsweise

@user.address.or User.make_a_long_and_painful_SQL_query_here

würde zusätzliche Arbeit machen, selbst wenn die Adresse nicht leer ist. Vielleicht könnten Sie das ein bisschen aktualisieren (Entschuldigung, dass Sie den Einzeiler verwirrt haben und versuchen, ihn kurz zu halten):

class String
  def or what = ""
    self.strip.empty? ? block_given? ? yield : what : self
  end
end

@user.address.or "We don't know user's address"
@user.address.or { User.make_a_long_and_painful_SQL_query_here }

Es ist wahrscheinlich besser, ActiveRecord oder einzelne Modelle anstelle von String zu erweitern.

Aus Ihrer Sicht bevorzugen Sie möglicherweise ein expliziteres Muster wie

@user.attr_or_default :address, "We don't know the user's address"

Rubin:

unless my_str.empty? then my_str else 'default' end

RoR:

unless my_str.blank? then my_str else 'default' end