jjzjj

c# - 清理病态嵌套 "if { } else { if { } else { if { ... } } }"

coder 2024-06-03 原文

我目前不幸在处理其他人的 C# 代码，这让我大吃一惊。我不知道我之前的人是如何维护这段代码的，因为它的各种病症已经使 IDE、编译器、运行时环境崩溃......

我今天面临的问题涉及一个 15 兆字节的源文件，该文件具有真正令人惊叹的病态嵌套程度。代码如下:

if(var == 0) {
  // do stuff
}
else {
  if(var == 1) {
    // do stuff
  }
  else {
    if(var == 2) {
      // do stuff, identical word for word to the `var == 1` case
    }
    else {
      // etc.
    }
  }
}

在最好的情况下，这是一个有问题的风格选择。然而，这与代码的另一种病态相结合:其中一些 block 将近一千级深。 (我费心测量的最深的值远远超过 700。)我真诚地希望我面前的人，作为他们在被迫与此代码分离之前的最后行为之一，运行了一个样式工具，导致了我面前的可憎。我无法想象他们可能会像现在这样编写这段代码，尤其是因为对代码进行的每三次或四次编辑都会使 IDE 崩溃。 (有时会删除我的源文件副本，作为奖励。)

我写了一个简单的基于正则表达式的工具来尝试压缩更简单的案例，但它似乎半处理然后破坏了这个特定的代码。 (我不确定它是否会失败，因为这段代码有时也会使用预处理器条件，或者因为最长的匹配项将近 10MB 长而 Lua 的正则表达式匹配器无法应对。)我希望有可以扭转这个问题的广泛使用的工具或技术。我已经不得不使用 astyle 来清理代码的其他一些风格“问题”。 astyle 的 --remove-brackets 选项 几乎 做我想要的，但要求括号内的语句是一行上的单个语句，这非常不是这里的案例...(只是为了越过我的“t”，我检查了；astyle 没有造成这个特殊问题。)

编辑:对问题代码的深入检查揭示了这样的事情:

#if OneThing
int num2296 = otherThing();
#endif
#if AnotherThing
int num44 = otherThing()
int num45 = 0;
#endif
int num72 = 0;
#if OneThing
int num45 = 0; // note: multiple equivalent declarations of num45
#endif
#if OneThing
for(int num2297 = 0; num2297 < num2296; ++num2297) {
  num45 = doSomething(num2297);
#endif
#if AnotherThing
for(int num43 = 0; num43 < num44; ++num43) {
  num45 = doSomething(num43);
#endif
  if(somethingElse(num45)) {
    ++num72;
  }
} // note: only one closing brace for the two protected by #ifs

此代码的两个版本针对不同目的进行了编译，一个定义了 OneThing，一个定义了 AnotherThing。不过两者的区别大部分只是变量名而已，逻辑完全相同。 (大部分，不是全部。)

上面代码段末尾的大括号等案例解释了为什么我的简单工具坏了。这看起来也越来越像是设计出来的工作保障，而不是无辜的无能。 (如果代码曾经在反编译器会生成像 num2276 这样的变量名的位置，那么当前不在那个位置。)

不幸的是，这意味着自动化工具可能无法独自解决问题。我只需要艰难地完成，慢慢地消除上一个程序员造成的损害。我把这个问题留在这里是因为有一个神奇的工具，我不知道，可以将两个版本转换为 SSA 并识别和折叠它们的逻辑等价物，然后将它们转换回来......

最佳答案

您可以使用 Roslyn 重写代码。将源代码修改为文本不是一个好方法。使用 Roslyn，您可以将其修改为语法树。

也许它可以帮助您将所有内容展平？

if (a)
 if (b) F2()
 else F3();
else
 F4();

可能变成:

if (a && b) F2();
else if (a && !b) F3();
else F4();

这样源代码就变成了一个平面列表，并且在什么条件下进入分支会更加明显。

关于c# - 清理病态嵌套 "if { } else { if { } else { if { ... } } }"，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/37531723/

c#elsenum的codeif-statementnestedindentation

有关c# - 清理病态嵌套 "if { } else { if { } else { if { ... } } }"的更多相关文章

  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 - 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

  4. ruby - 将散列转换为嵌套散列 - 2

    这道题是thisquestion的逆题.给定一个散列，每个键都有一个数组，例如{[:a,:b,:c]=>1,[:a,:b,:d]=>2,[:a,:e]=>3,[:f]=>4,}将其转换为嵌套哈希的最佳方法是什么{:a=>{:b=>{:c=>1,:d=>2},:e=>3,},:f=>4,} 最佳答案 这是一个迭代的解决方案，递归的解决方案留给读者作为练习:defconvert(h={})ret={}h.eachdo|k,v|node=retk[0..-2].each{|x|node[x]||={};node=node[x]}node[

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

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

  6. ruby-on-rails - 迷你测试错误 : "NameError: uninitialized constant" - 2

    我遵循MichaelHartl的“RubyonRails教程:学习Web开发”，并创建了检查用户名和电子邮件长度有效性的测试(名称最多50个字符，电子邮件最多255个字符)。test/helpers/application_helper_test.rb的内容是:require'test_helper'classApplicationHelperTest在运行bundleexecraketest时，所有测试都通过了，但我看到以下消息在最后被标记为错误:ERROR["test_full_title_helper",ApplicationHelperTest,1.820016791]test

  7. ruby-on-rails - 相关表上的范围为 "WHERE ... LIKE" - 2

    我正在尝试从Postgresql表(table1)中获取数据，该表由另一个相关表(property)的字段(table2)过滤。在纯SQL中，我会这样编写查询:SELECT*FROMtable1JOINtable2USING(table2_id)WHEREtable2.propertyLIKE'query%'这工作正常:scope:my_scope,->(query){includes(:table2).where("table2.property":query)}但我真正需要的是使用LIKE运算符进行过滤，而不是严格相等。然而，这是行不通的:scope:my_scope,->(que

  8. 使用 ACL 调用 upload_file 时出现 Ruby S3 "Access Denied"错误 - 2

    我正在尝试编写一个将文件上传到AWS并公开该文件的Ruby脚本。我做了以下事情:s3=Aws::S3::Resource.new(credentials:Aws::Credentials.new(KEY,SECRET),region:'us-west-2')obj=s3.bucket('stg-db').object('key')obj.upload_file(filename)这似乎工作正常，除了该文件不是公开可用的，而且我无法获得它的公共(public)URL。但是当我登录到S3时，我可以正常查看我的文件。为了使其公开可用，我将最后一行更改为obj.upload_file(file

  9. ruby - 安装 Ruby 时遇到问题(无法下载资源 "readline--patch") - 2

    当我尝试安装Ruby时遇到此错误。我试过查看this和this但无济于事➜~brewinstallrubyWarning:YouareusingOSX10.12.Wedonotprovidesupportforthispre-releaseversion.Youmayencounterbuildfailuresorotherbreakages.Pleasecreatepull-requestsinsteadoffilingissues.==>Installingdependenciesforruby:readline,libyaml,makedepend==>Installingrub

  10. Ruby——嵌套类和子类是一回事吗？ - 2

    下面例子中的Nested和Child有什么区别？是否只是同一事物的不同语法？classParentclassNested...endendclassChild 最佳答案 不，它们是不同的。嵌套:Computer之外的“Processor”类只能作为Computer::Processor访问。嵌套为内部类(namespace)提供上下文。对于ruby​​解释器Computer和Computer::Processor只是两个独立的类。classComputerclassProcessor#Tocreateanobjectforthisc

随机推荐