Question

我编写了一个逻辑/方法，该方法返回两个不同的对象（整数和字符串），例如，返回的值将为5000.00 Dollars。

所以我写了一个方法来达到我的期望。参见下面的逻辑：

s = x.currency # This assigns the string `dollarpounds` to s
a = s.slice(6..11) # This slice off 6 to 11 and returns just pounds to variable a
price_value = x.price # This is integer (price)
volume_volume = x.volume # This is integer (volume)
value = price_value * volume_volume # This multiplies price and volume and returns the value
value + "#{a}" # Now this throws TypeError-String can't be coerced into BigDecimal

因此，为了解决这个问题，我重构了我的方法，但是我认为它非常侮辱性的代码片段被认为是Ruby的高手。 如何在下面重新编写此重构逻辑以使其足够聪明，如Ruby代码？

这就是我所做的：

重构逻辑。它会按预期返回5000.00 Dollars

s = x.currency # This assigns the string `dollarpounds` to s
a = s.slice(6..11) # This slice off 6 to 11 and returns just pounds to variable a
price_value = x.price # This is integer (price)
volume_volume = x.volume # This is integer (volume)
value = price_value * volume_volume # This multiplies price and volume and returns the value
[[value].join(' '), "#{a}"].split(',').join(' ') # This returns 5000.00 Dollars

尽管我的re-factored代码可以正常工作，但我仍然觉得这对ruby社区是一种侮辱，并且可以做得更好。任何帮助您将其做得更好的方法都将受到赞赏。

Answer 1

使用插值：

"#{value} #{a}"

或串联：

value.to_s + ' ' + a

Answer 2

非常有趣的是，我如何在重构[[value].join(' '), "#{a}"].split(',').join(' ')的最后一行中使用插值法，并且我从来不会调暗它适合仅使用插值法。除了在答案线程中建议的插值方式之外，我还能够使代码更简单，更小，更快。

s = x.currency
a = s.slice(6..11)
value = x.price * x.volume
"#{value} #{a}" # Thanks to @PavelPuzin for this suggestion in this line.

关于实现此目标的最佳方法，我们可以考虑的另一件事是研究Interpolation和我使用Benchmark确定其算法复杂度的Join：

require "benchmark"

numbers = (1..1000).to_a

n = 1000
Benchmark.bm do |x|
  x.report { n.times do   ; numbers.each_cons(2) {|a, b| "#{a} #{b}"}; end }
  x.report { n.times do   ; numbers.each_cons(2) {|a, b| [a, b].join(" ")}; end }
end 

###############################Result###################################
    user     system      total        real
   0.467287   0.000731   0.468018 (  0.468641)
   1.154991   0.001563   1.156554 (  1.157740)

避免不能将String强制转换为BigDecimal。

2 个答案: