Wie Codes zu verbessern, dass die Zitate alle array-Elemente mit ` " und gibt eine Zeichenfolge zurück, die alle diese notiert und durch Komma getrennte Elemente?

Ich bin mit Schienen 3.2.2, und ich möchte zitieren, die alle array-Elemente mit ' und geben eine Zeichenfolge zurück, die alle diese notiert und durch Komma getrennte Elemente. In dieser Zeit bin ich mit

['a', 'b', 'c'].collect {|x| "'#{x}'"}.join(", ")
# => "'a', 'b', 'c'"

aber ich glaube ich kann verbessern Sie den obigen code (vielleicht mithilfe eines mir unbekannten Ruby-Methode, wenn es vorhanden ist). Ist es möglich?

  • Was haben Sie scheint gut zu mir.
  • Es gibt nichts zu optimieren, bu Sie können Aliase: map statt collect und * statt join wenn Sie hungern, um Ihren code kleiner: %w[a b c].map{|x|"'#{x}'"}*', '.
  • Was passiert, wenn einer der strings im array enthält eine ' von selbst, z.B. ["a", "b'cd"]?
  • Was Sie haben, ist gut. Auch dies ist pure ruby. Es gibt nichts "Schienen" über es.
  • Nur - Ops!
  • Vielleicht solltest du konvertieren, die auf eine Antwort um diese Frage beantworten?

InformationsquelleAutor Backo | 2012-05-18
  1. 11

    Benutze ich

    "'#{%w{a b c}.join("', '")}'"

    Hier ist die erweiterte version:

    ' # Starting quote
%w{a b c}.join("', '") # Join array with ', ' delimiter that would give a', 'b', 'c
' # Closing quote
    • Es scheint, deine version ist besser.
    • Hier ist, warum: es ist Zunächst zu Fuss array einmal statt zweimal(map und join). Zweitens schafft Sie weniger ruby-Objekte - eine wrapper-string für das aktuelle Ergebnis und eine Zeichenfolge, die als join-Ergebnis. Initiale Lösung erzeugt neues array für die map-Ergebnis-und die neue Zeichenfolge für jedes element im array. Es könnte zeigen, verschlechtern sich die Leistung auf größeren array seit der GC würde länger dauern.
    • Sie zu, dass in Ihrer Antwort zu erklären, warum Ihr code ist vorzuziehen. Auch, du meinst die garbage collection mit der 'GC'?
    InformationsquelleAutor Sigurd
  2. 6

    Können Sie ersetzen collect mit seinem alias map und .join mit den entsprechenden *. Schließlich können Sie mit die Tastenkombination für das schreiben ein array von strings, %w(...), und Sie können verwenden Sie einfache Anführungszeichen für das argument der .join/* da es nicht die Verwendung von string-interpolation (auch wenn es vielleicht fragwürdig ist, wenn es besser wenn es zur Leistung kommt).

    %w(a b c).map {|x| "'#{x}'"} * ', '

    Scheint es, gibt es keine performance-Unterschied zwischen dieser version und das original, sondern, dass Sigurd ' s version besser:

    Original 3.620000 0.000000 3.620000 ( 3.632081) 
Diese 3.640000 0.000000 3.640000 ( 3.651763) 
Sigurd ist 2.300000 0.000000 2.300000 ( 2.303195)

    Code für benchmark:

    require 'benchmark'

n = 1000000

Benchmark.bm do |x|
  x.report("Original") { n.times do
      ['a', 'b', 'c'].collect {|x| "'#{x}'"}.join(", ")
    end}
  x.report("This") { n.times do
      %w(a b c).map {|x| "'#{x}'"} * ', '
    end}
  x.report("Sigurd's") { n.times do
      "'#{%w{a b c}.join("', '")}'"
    end}
end
    InformationsquelleAutor N.N.
Schreibe einen Kommentar