将代码语句放在大括号中是一种好的编程习惯吗？

Question

在我一生中读过的所有源代码中，我从未见过这样做的。如果它被认为是不好的编程实践，那么一定有一个我无法理解的原因。另外，我认为它有时会提高可读性而不是恶化它。以下是我在 ruby​​ 代码中完成的几个地方。

@pushButton.connect(SIGNAL :clicked) do (@select_file ||= Qt::FileDialog.new).show end

和

(tmr=Qt::Timer.new).connect SIGNAL :timeout do 
  @label.text = Qt::Application.translate("MainWindow", "The time right now is #{Time.now}", nil, Qt::Application::UnicodeUTF8)
end 
tmr.start(1000)

Answer 1

尽可能力求简单总是一个好主意，为此，最好以直截了当的方式陈述事情。像这样的声明使得很难确定变量的来源，因为它们相当彻底地嵌入到语句中。

在括号内声明作用域变量通常被认为是可以接受的：

if (found = MyModel.find_by_pigeon_id(params[:pigeon_id]))
  # Variable 'found' used only within this block
end

# Ruby variables will persist here, but in many languages they are out of scope

更详细的版本实际上具有含义：

found = MyModel.find_by_pigeon_id(params[:pigeon_id])
if (found)
  # Variable 'found' can be used here
end

# Implies 'found' may be used here for whatever reason

能够扫描整个程序并非常清楚地看到声明的所有变量总是很好。隐藏事情除了让人们感到沮丧之外没有任何目的。

就你可以逃脱的程度而言，Ruby 比许多其他语言要轻松得多。有些语言会因为使事情复杂化而严厉惩罚你，因为声明或转换中的微小错误可能会产生巨大的后果。这并不意味着您应该抓住每一个机会充分利用这一点。

以下是我建议如何实现您的第一个示例：

# Ensure that @select_file is defined
@select_file ||= Qt::FileDialog.new

@pushButton.connect(SIGNAL(:clicked)) do
  # Block is split out into multiple lines for clarity
  @select_file.show
end

第二个：

# Simple declaration, variable name inherited from class name, not truncated
timer = Qt::Timer.new

timer.connect(SIGNAL(:timeout)) do 
  # Long parameter list is broken out into separate lines to make it clear
  # what the ordering is. Useful for identifying accidentally missing parameters.
  @label.text = Qt::Application.translate(
    "MainWindow",
    "The time right now is #{Time.now}",
    nil,
    Qt::Application::UnicodeUTF8
  )
end

timer.start(1000)

我发现最复杂的程序通常看起来最简单，因为它们是由经验丰富的人编写的，他们知道如何以简单的方式表达事物。

有趣的是，一些最简单的程序往往是最复杂的，因为它们是由新手编写的，这些新手要么是在哗众取宠、炫耀，要么是自掘深渊，不断地向问题扔代码以期解决问题。

Answer 2

我觉得读起来比较困难。我希望它稍微详细一些，但更清晰：

tmr = Qt::Timer.new
tmr.connect SIGNAL :timeout do 
  @label.text = Qt::Application.translate("MainWindow", "The time right now is #{Time.now}", nil, Qt::Application::UnicodeUTF8)
end 
tmr.start(1000)

Answer 3

一行中塞满太多逻辑语句的代码很难阅读，因为读者本质上必须在大脑中将其展开为类似于 troelskn 的答案的语句。

它的可读性较差的其他原因是：

• 它使行长度超出了大多数文本编辑器窗口的边框。
• 它掩盖了您在调试时可能需要看到的各个步骤。

Answer 4

我想你的意思是父母？

Brace: {

Paren: (

如果您使用大括号而不是 do/end，您的第一个示例会变得更具可读性——按照惯例，如果块位于单行上，则使用大括号，而 do/end 用于多行块。

@pushButton.connect(SIGNAL :clicked) { (@select_file ||= Qt::FileDialog.new).show }

第二个我在 java/c 中见过很多例子，但实际上我在 ruby​​ 中看到了它的更多功能：

Qt::Timer.new.connect SIGNAL :timeout do 
  @label.text = Qt::Application.translate("MainWindow", "The time right now is #{Time.now}", nil, Qt::Application::UnicodeUTF8)
end.start(1000)

但是在启动它之后你就​​没有对该对象的引用。