ruby - Ruby 最佳实践：如果不为空，则在一个运算符中执行其他操作

Question

1.我找不到写这段代码的优雅方式：

if array.empty?
  # process empty array
else
  array.each do |el|
    # process el
  end
end

我想要一个循环，而不是写array两次。我读了这个，但没有足够好的解决方案。

---

2. 我实际上是在一个 HAML 模板中。同样的问题。

- if array.empty?
  %p No result
- else
  %ul
  - array.each do |el|
    %li el

Answer 1

关于什么？

array.each do |x|
  #...
  puts "x",x
end.empty? and begin
  puts "empty!"
end

Answer 2

我在 HAML（不是普通的 Ruby）中看到的最干净的方法是：

- array.each do |item|
    %li
        = item.name
- if array.empty?
    %li.empty
        Nothing here.

正如其他答案所提到的，不需要该else子句，因为这已经隐含在其他逻辑中。

即使您可以在一条干净的行中执行每个其他操作，您也无法实现您想要实现的标记（<p>如果 array.empty？，<ul>如果 array.present？）。此外，您在问题中显示的 HAML 是讲述代码背后故事的最佳方式，这意味着它对其他开发人员来说更具可读性和可维护性，所以我不知道您为什么要重构为更神秘的东西.

Answer 3

我认为没有比这更优雅或更易读的方式了。任何以某种方式将迭代与条件结合起来的方式都只会产生黑盒代码，这意味着：条件很可能会隐藏在Array扩展中。

Answer 4

如果array为空，则不会被迭代，因此该each块不需要条件化。由于 的返回值each是接收者，因此您可以将each块放在empty?条件内。

if (array.each do |el|
  # process el
end).empty?
  # process empty array
end

Answer 5

假设“处理空数组”在处理后将其留空，您可以省略其他：

if array.empty?
  # process empty array 
end
array.each do |el|
  # process el
end

或在一行中：

array.empty? ? process_empty_array : array.each { |el| process_el } 

Answer 6

如果数组为 nil，那么我们可以强制清空数组

if (array || []).each do |x|
  #...
  puts "x",x
end.empty?
  puts "empty!"
end

Answer 7

我看到有人问如何处理 nil 的情况。

诀窍是将其转换为字符串。所有转换为字符串的 nil 都变为空字符串，所有空情况继续为空。

nil.to_s.empty?
"".to_s.empty?

两者都将返回 true