草庐IT

c++ - 没有 "if(...) return ...;"的 "else"是否被认为是好的风格？

coder 2023-05-31 原文

这段代码:

if( someCondition )
    return doSomething();

return doSomethingElse();

对比这段代码:

if( someCondition )
    return doSomething();
else
    return doSomethingElse();

本质上，它们是相同的，但是什么是最好的风格/性能/...(当然，如果答案中有任何非自以为是的话)？还要考虑多个“if else's”的情况:

if( someCondition )
    return doSomething();
else if( someOtherCondition )
    return doSomethingDifferently();
//...
else
    return doSomethingElse();

谢谢!

最佳答案

当函数中有多个返回语句时，这称为“提前返回”。如果你做 Google search对于“提前返回”，您会发现一个接一个的链接表明它很糟糕。

我说的是废话。

人们声称早退是不好的，有两个主要原因和一个次要原因。我将通过它们并按顺序给出我的反驳。请记住，这是我的全部意见，最终您必须自己决定。

  1. 原因:提前返回很难清理。

    反驳:就是这样RAII是为了。一个设计良好的程序不会以这样一种方式分配资源，即如果执行提前离开范围，这些资源就会泄漏。而不是这样做:

...

int foo()
{
  MyComplexDevice* my_device = new MyComplexDevice;
  // ...
  if( something_bad_hapened )
    return 0;
  // ...
  delete my_device;
  return 42;
}

你这样做:

int foo()
{
  std::auto_ptr<MyComplexDevice> my_device(new MyComplexDevice);
  if( something_bad_hapened )
    return 0;
  // ...
  return 42;
}

并且提前返回不会导致资源泄漏。在大多数情况下，您甚至不需要使用 auto_ptr 因为您将创建数组或字符串，在这种情况下您将使用 vector、string 或类似的东西。

您应该像这样设计您的代码以保证稳健性，因为可能会出现异常。异常是一种提前返回的形式，就像显式的 return 语句一样，您需要准备好处理它们。 foo() 中的异常你可能不会处理，但 foo() 无论如何都不应该泄漏。

  1. 原因:提前返回会使代码更加复杂。 反驳:早期返回实际上使代码更简单。

函数应该有一个责任是一种普遍的哲学。我同意这一点。但是人们对此太过分了，并得出结论，如果一个函数有多个返回，它必须有多个责任。 (他们通过说函数的长度不应超过 50 行或其他任意数字来扩展这一点。)我说不。仅仅因为一个函数只有一个职责，并不意味着它不需要做很多事情来履行这个职责。

以打开数据库为例。这是一项责任，但它由许多步骤组成，每个步骤都可能出错。打开连接。登录。获取一个连接对象并返回它。 3 个步骤，每个步骤都可能失败。您可以将其分解为 3 个子步骤，但不要使用这样的代码:

int foo()
{ 
  DatabaseObject db = OpenDatabase(...);
}

你最终会拥有:

int foo()
{
  Connection conn = Connect(...);
  bool login = Login(...);
  DBObj db = GetDBObj(conn);
}

因此，您实际上只是将假定的多个职责移到调用堆栈中的更高位置。

  1. 原因:多个返回点不是面向对象的。 反驳:这实际上只是另一种说法“每个人都说多次返回不好，尽管我真的不知道为什么。”

换一种说法，这实际上只是试图将所有东西都塞进一个物体形状的盒子里，即使它不属于那里。当然，也许连接是一个对象。但是是登录吗？登录尝试不是(IMO)对象。这是一个手术。或者算法。尝试采用这种算法并将其塞进一个对象形状的盒子中是对 OOP 的无端尝试，只会导致代码更复杂、更难维护，甚至可能效率更低。

关于c++ - 没有 "if(...) return ...;"的 "else"是否被认为是好的风格？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/4238063/

amp34sectioncodereturnc++if-statement

有关c++ - 没有 "if(...) return ...;"的 "else"是否被认为是好的风格？的更多相关文章

  1. ruby-on-rails - rails : "missing partial" when calling 'render' in RSpec test - 2

    我正在尝试测试是否存在表单。我是Rails新手。我的new.html.erb_spec.rb文件的内容是:require'spec_helper'describe"messages/new.html.erb"doit"shouldrendertheform"dorender'/messages/new.html.erb'reponse.shouldhave_form_putting_to(@message)with_submit_buttonendendView本身，new.html.erb，有代码:当我运行rspec时，它失败了:1)messages/new.html.erbshou

  2. ruby-on-rails - 由于 "wkhtmltopdf"，PDFKIT 显然无法正常工作 - 2

    我在从html页面生成PDF时遇到问题。我正在使用PDFkit。在安装它的过程中，我注意到我需要wkhtmltopdf。所以我也安装了它。我做了PDFkit的文档所说的一切......现在我在尝试加载PDF时遇到了这个错误。这里是错误:commandfailed:"/usr/local/bin/wkhtmltopdf""--margin-right""0.75in""--page-size""Letter""--margin-top""0.75in""--margin-bottom""0.75in""--encoding""UTF-8""--margin-left""0.75in""-

  3. ruby-on-rails - 如何验证 update_all 是否实际在 Rails 中更新 - 2

    给定这段代码defcreate@upgrades=User.update_all(["role=?","upgraded"],:id=>params[:upgrade])redirect_toadmin_upgrades_path,:notice=>"Successfullyupgradeduser."end我如何在该操作中实际验证它们是否已保存或未重定向到适当的页面和消息？ 最佳答案 在Rails3中，update_all不返回任何有意义的信息，除了已更新的记录数(这可能取决于您的DBMS是否返回该信息)。http://ar.ru

  4. ruby - 难道Lua没有和Ruby的method_missing相媲美的东西吗？ - 2

    我好像记得Lua有类似Ruby的method_missing的东西。还是我记错了？ 最佳答案 表的metatable的__index和__newindex可以用于与Ruby的method_missing相同的效果。 关于ruby-难道Lua没有和Ruby的method_missing相媲美的东西吗？，我们在StackOverflow上找到一个类似的问题： https://stackoverflow.com/questions/7732154/

  5. ruby-on-rails - 如果为空或不验证数值，则使属性默认为 0 - 2

    我希望我的UserPrice模型的属性在它们为空或不验证数值时默认为0。这些属性是tax_rate、shipping_cost和price。classCreateUserPrices8,:scale=>2t.decimal:tax_rate,:precision=>8,:scale=>2t.decimal:shipping_cost,:precision=>8,:scale=>2endendend起初，我将所有3列的:default=>0放在表格中，但我不想要这样，因为它已经填充了字段，我想使用占位符。这是我的UserPrice模型:classUserPrice回答before_val

  6. ruby - 检查 "command"的输出应该包含 NilClass 的意外崩溃 - 2

    为了将Cucumber用于命令行脚本，我按照提供的说明安装了arubagem。它在我的Gemfile中，我可以验证是否安装了正确的版本并且我已经包含了require'aruba/cucumber'在'features/env.rb'中为了确保它能正常工作，我写了以下场景:@announceScenario:Testingcucumber/arubaGivenablankslateThentheoutputfrom"ls-la"shouldcontain"drw"假设事情应该失败。它确实失败了，但失败的原因是错误的:@announceScenario:Testingcucumber/ar

  7. ruby-on-rails - rails 目前在重启后没有安装 - 2

    我有一个奇怪的问题:我在rvm上安装了ruby​​onrails。一切正常，我可以创建项目。但是在我输入“railsnew”时重新启动后，我有“程序'rails'当前未安装。”。SystemUbuntu12.04ruby-v"1.9.3p194"gemlistactionmailer(3.2.5)actionpack(3.2.5)activemodel(3.2.5)activerecord(3.2.5)activeresource(3.2.5)activesupport(3.2.5)arel(3.0.2)builder(3.0.0)bundler(1.1.4)coffee-rails(

  8. ruby-on-rails - 如何优雅地重启 thin + nginx？ - 2

    我的瘦服务器配置了nginx，我的ROR应用程序正在它们上运行。在我发布代码更新时运行thinrestart会给我的应用程序带来一些停机时间。我试图弄清楚如何优雅地重启正在运行的Thin实例，但找不到好的解决方案。有没有人能做到这一点？ 最佳答案 #Restartjustthethinserverdescribedbythatconfigsudothin-C/etc/thin/mysite.ymlrestartNginx将继续运行并代理请求。如果您将Nginx设置为使用多个上游服务器，例如server{listen80;server

  9. ruby - 检查数组是否在增加 - 2

    这个问题在这里已经有了答案:Checktoseeifanarrayisalreadysorted?(8个答案)关闭9年前。我只是想知道是否有办法检查数组是否在增加？这是我的解决方案，但我正在寻找更漂亮的方法:n=-1@arr.flatten.each{|e|returnfalseife

  10. ruby - 在没有 sass 引擎的情况下使用 sass 颜色函数 - 2

    我想在一个没有Sass引擎的类中使用Sass颜色函数。我已经在项目中使用了sassgem，所以我认为搭载会像以下一样简单:classRectangleincludeSass::Script::FunctionsdefcolorSass::Script::Color.new([0x82,0x39,0x06])enddefrender#hamlengineexecutedwithcontextofself#sothatwithintemlateicouldcall#%stop{offset:'0%',stop:{color:lighten(color)}}endend更新:参见上面的#re

随机推荐