c# - 如何重写复杂的 C++ 代码行(嵌套三元运算符)

coder 2023-05-03 原文

我一直在查看其他人的代码以进行调试，发现了这一点:

!m_seedsfilter ? good=true : m_seedsfilter==1 ? good=newClusters(Sp) : good=newSeed(Sp);

这是什么意思？是否有一种自动化工具可以将其转化为更易于理解的 if/else 语句？处理像这样复杂的控制结构有什么技巧吗？

编辑说明:我在标题中将其从“不必要的复杂”更改为“复杂”，因为这是一个见仁见智的问题。感谢您到目前为止的所有回答。

最佳答案

如果重写如下，可以改进所写的语句......

good = m_seedsfilter==0 ? true :
       m_seedsfilter==1 ? newClusters(Sp) :
                          newSeed(Sp);

...但总的来说，您应该熟悉三元语句。最初发布的代码、xanatos 的版本或我的版本都没有本质上的邪恶。三元语句并不邪恶，它们是该语言的一个基本特性，一旦你熟悉了它们，你会注意到这样的代码(正如我发布的，而不是你原帖中写的)实际上更容易阅读比一系列 if-else 语句。例如，在这段代码中，您可以简单地阅读以下语句:“变量 good 等于...如果 m_seedsfilter==0 ，则 true ，否则，如果 m_seedsfilter==1 ，则 newClusters(Sp) ，否则， newSeed(Sp) 。”
请注意，我上面的版本避免了对变量 good 进行三个单独的分配，并明确说明该语句的目标是为 good 分配一个值。此外，以这种方式编写，它清楚地表明本质上这是一个“切换案例”构造，默认案例是 newSeed(Sp) 。
应该注意的是，只要 operator!() 类型的 m_seedsfilter 没有被覆盖，我上面的重写就很好。如果是，那么您必须使用它来保留原始版本的行为......

good = !m_seedsfilter   ? true :
       m_seedsfilter==1 ? newClusters(Sp) :
                          newSeed(Sp);

...正如下面 xanatos 的评论所证明的那样，如果您的 newClusters() 和 newSeed() 方法返回彼此不同的类型，并且如果这些类型是用精心设计的无意义转换运算符编写的，那么您将不得不恢复到原始代码本身(尽管希望格式更好，就像在 xanatos 自己的帖子中一样)，以便忠实地复制与原始帖子完全相同的行为。但在现实世界中，没有人会这样做，所以我上面的第一个版本应该没问题。

更新，在原始帖子/答案发布两年半后:
有趣的是，@TimothyShields 和我不时地对此表示赞同，Tim 的答案似乎始终保持在该答案的赞成票的 50% 左右，或多或少(截至本次更新为 43 对 22)。
我想我会添加另一个例子，说明三元语句在明智地使用时可以添加的清晰度。下面的示例是我为调用堆栈使用分析器(一种分析编译的 C 代码的工具，但该工具本身是用 C# 编写的)编写的代码的简短片段。所有三种变体都实现了完全相同的目标，至少就外部可见效果而言。
1. 没有三元运算符:

Console.Write(new string(' ', backtraceIndentLevel) + fcnName);
if (fcnInfo.callDepth == 0)
{
   Console.Write(" (leaf function");
}
else if (fcnInfo.callDepth == 1)
{
   Console.Write(" (calls 1 level deeper");
}
else
{
   Console.Write(" (calls " + fcnInfo.callDepth + " levels deeper");
}
Console.WriteLine(", max " + (newStackDepth + fcnInfo.callStackUsage) + " bytes)");

2. 使用三元运算符，单独调用 Console.Write():

Console.Write(new string(' ', backtraceIndentLevel) + fcnName);
Console.Write((fcnInfo.callDepth == 0) ? (" (leaf function") :
              (fcnInfo.callDepth == 1) ? (" (calls 1 level deeper") :
                                         (" (calls " + fcnInfo.callDepth + " levels deeper"));
Console.WriteLine(", max " + (newStackDepth + fcnInfo.callStackUsage) + " bytes)");

3. 使用三元运算符，折叠为对 Console.Write() 的单个调用:

Console.WriteLine(
   new string(' ', backtraceIndentLevel) + fcnName +
   ((fcnInfo.callDepth == 0) ? (" (leaf function") :
    (fcnInfo.callDepth == 1) ? (" (calls 1 level deeper") :
                               (" (calls " + fcnInfo.callDepth + " levels deeper")) +
   ", max " + (newStackDepth + fcnInfo.callStackUsage) + " bytes)");

有人可能会争辩说，上面三个例子之间的区别是微不足道的，既然微不足道，为什么不更喜欢更简单的(第一个)变体呢？一切都是为了简洁；用“尽可能少的词”表达一个想法，以便听者/读者在我到达想法的结尾时仍能记住想法的开头。当我和 child 子说话时，我使用简单、简短的句子，因此需要更多的句子来表达一个想法。当我与能说流利的语言的成年人交谈时，我会使用更长、更复杂的句子来更简洁地表达想法。
这些示例将单行文本打印到标准输出。虽然它们执行的操作很简单，但很容易将它们想象成更大序列的子集。我越能简洁地表达该序列的子集，该序列中的内容就越能显示在我的编辑器屏幕上。当然，我很容易把这种努力做得太过分，使它更难以理解；目标是在易于理解和简洁之间找到“sweet spot”。我认为，一旦程序员熟悉了三元语句，理解使用它们的代码就会比理解不使用它们的代码更容易(例如 2 和 3 上面的 _x1045679506 上面的 _x1079506 _107920 107920 上面)。
有经验的程序员应该对使用三元语句感到舒服的最后一个原因是在进行方法调用时避免创建不必要的临时变量。作为一个例子，我展示了上述例子的第四个变体，将逻辑浓缩为对 Console.WriteLine() 的单个调用；结果是少可理解和少简明:
4. 没有三元运算符，折叠为对 Console.Write() 的单个调用:

string tempStr;
if (fcnInfo.callDepth == 0)
{
   tempStr = " (leaf function";
}
else if (fcnInfo.callDepth == 1)
{
   tempStr = " (calls 1 level deeper";
}
else
{
   tempStr = " (calls " + fcnInfo.callDepth + " levels deeper";
}
Console.WriteLine(new string(' ', backtraceIndentLevel) + fcnName + tempStr +
                  ", max " + (newStackDepth + fcnInfo.callStackUsage) + " bytes)");

在争论“将逻辑浓缩为对 Console.WriteLine() 的单个调用是不必要的”之前，请考虑这仅仅是一个例子:想象一下调用其他一些方法，一个接受多个参数的方法，所有这些都需要基于其他变量状态的临时变量.您可以创建自己的临时文件并使用这些临时文件进行方法调用，或者您可以使用三元运算符并让编译器创建自己的(未命名的)临时文件。我再次争辩说，三元运算符比没有三元运算符可以实现更简洁和易于理解的代码。但是要使其易于理解，您必须放弃任何先入为主的观念，即三元运算符是邪恶的。

关于c# - 如何重写复杂的 C++ 代码行(嵌套三元运算符)，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/18237432/

c#运算符 code strong c++c

有关c# - 如何重写复杂的 C++ 代码行(嵌套三元运算符)的更多相关文章

ruby - 如何使用 Nokogiri 的 xpath 和 at_xpath 方法 - 2
我正在学习如何使用Nokogiri，根据这段代码我遇到了一些问题:require'rubygems'require'mechanize'post_agent=WWW::Mechanize.newpost_page=post_agent.get('http://www.vbulletin.org/forum/showthread.php?t=230708')puts"\nabsolutepathwithtbodygivesnil"putspost_page.parser.xpath('/html/body/div/div/div/div/div/table/tbody/tr/td/div
ruby - 如何从 ruby 中的字符串运行任意对象方法？ - 2
总的来说，我对ruby还比较陌生，我正在为我正在创建的对象编写一些rspec测试用例。许多测试用例都非常基础，我只是想确保正确填充和返回值。我想知道是否有办法使用循环结构来执行此操作。不必为我要测试的每个方法都设置一个assertEquals。例如:describeitem,"TestingtheItem"doit"willhaveanullvaluetostart"doitem=Item.new#HereIcoulddotheitem.name.shouldbe_nil#thenIcoulddoitem.category.shouldbe_nilendend但我想要一些方法来使用
python - 如何使用 Ruby 或 Python 创建一系列高音调和低音调的蜂鸣声？ - 2
关闭。这个问题是opinion-based.它目前不接受答案。想要改进这个问题？更新问题，以便editingthispost可以用事实和引用来回答它.关闭4年前。Improvethisquestion我想在固定时间创建一系列低音和高音调的哔哔声。例如:在150毫秒时发出高音调的蜂鸣声在151毫秒时发出低音调的蜂鸣声200毫秒时发出低音调的蜂鸣声250毫秒的高音调蜂鸣声有没有办法在Ruby或Python中做到这一点？我真的不在乎输出编码是什么(.wav、.mp3、.ogg等等)，但我确实想创建一个输出文件。
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
ruby-on-rails - 'compass watch' 是如何工作的/它是如何与 rails 一起使用的 - 2
我在我的项目目录中完成了compasscreate.和compassinitrails。几个问题:我已将我的.sass文件放在public/stylesheets中。这是放置它们的正确位置吗？当我运行compasswatch时，它不会自动编译这些.sass文件。我必须手动指定文件:compasswatchpublic/stylesheets/myfile.sass等。如何让它自动运行？文件ie.css、print.css和screen.css已放在stylesheets/compiled。如何在编译后不让它们重新出现的情况下删除它们？我自己编译的.sass文件编译成compiled/t
ruby-on-rails - Rails 编辑表单不显示嵌套项 - 2
我得到了一个包含嵌套链接的表单。编辑时链接字段为空的问题。这是我的表格:Editingkategori{:action=>'update',:id=>@konkurrancer.id})do|f|%>'Trackingurl',:style=>'width:500;'%>'Editkonkurrence'%>|我的konkurrencer模型:has_one:link我的链接模型:classLink我的konkurrancer编辑操作:defedit@konkurrancer=Konkurrancer.find(params[:id])@konkurrancer.link_attrib
ruby - 如何将脚本文件的末尾读取为数据文件(Perl 或任何其他语言) - 2
我正在寻找执行以下操作的正确语法(在Perl、Shell或Ruby中):#variabletoaccessthedatalinesappendedasafileEND_OF_SCRIPT_MARKERrawdatastartshereanditcontinues. 最佳答案 Perl用__DATA__做这个:#!/usr/bin/perlusestrict;usewarnings;while(){print;}__DATA__Texttoprintgoeshere 关于ruby-如何将脚
ruby - 如何在 buildr 项目中使用 Ruby 代码？ - 2
如何在buildr项目中使用Ruby？我在很多不同的项目中使用过Ruby、JRuby、Java和Clojure。我目前正在使用我的标准Ruby开发一个模拟应用程序，我想尝试使用Clojure后端(我确实喜欢功能代码)以及JRubygui和测试套件。我还可以看到在未来的不同项目中使用Scala作为后端。我想我要为我的项目尝试一下buildr(http://buildr.apache.org/)，但我注意到buildr似乎没有设置为在项目中使用JRuby代码本身!这看起来有点傻，因为该工具旨在统一通用的JVM语言并且是在ruby中构建的。除了将输出的jar包含在一个独特的、仅限ruby
ruby - 如何指定 Rack 处理程序 - 2
Rackup通过Rack的默认处理程序成功运行任何Rack应用程序。例如:classRackAppdefcall(environment)['200',{'Content-Type'=>'text/html'},["Helloworld"]]endendrunRackApp.new但是当最后一行更改为使用Rack的内置CGI处理程序时，rackup给出“NoMethodErrorat/undefinedmethod`call'fornil:NilClass”:Rack::Handler::CGI.runRackApp.newRack的其他内置处理程序也提出了同样的反对意见。例如Rack
ruby-on-rails - Rails 源代码 : initialize hash in a weird way? - 2
在rails源中:https://github.com/rails/rails/blob/master/activesupport/lib/active_support/lazy_load_hooks.rb可以看到以下内容@load_hooks=Hash.new{|h,k|h[k]=[]}在IRB中，它只是初始化一个空哈希。和做有什么区别@load_hooks=Hash.new 最佳答案查看rubydocumentationforHashnew→new_hashclicktotogglesourcenew(obj)→new_has

c# - 如何重写复杂的 C++ 代码行(嵌套三元运算符)

有关c# - 如何重写复杂的 C++ 代码行(嵌套三元运算符)的更多相关文章

随机推荐