java - 搜索并替换没有大括号的 if 语句以包含大括号

Question

我最近一直在使用声纳进行代码分析。当我进行彻底的违规向下钻取时，我发现许多 java 文件的 if 语句定义时没有用大括号(数以千计的地方)。是否有一种简单的方法来替换或添加大括号到 if 语句，或者我可以执行哪些步骤来完成此任务而无需在每个文件中手动执行。

我目前正在使用 intelliJ。

Answer 1

Is there a simple way to replace or add braces to if statements or what are the steps that I can perform to achieve this task without doing it manually in each of the files.

我不知道是否有工具可以自动执行此操作。 (可能有……)但假设存在这样的工具，我不认为这是正确的方法。

退后一步，考虑一下为什么代码分析将此报告为问题。很多人(比如@pst 和我)认为大括号应该始终存在，尽管各种风格指南并不坚持这一点。这是有充分理由的……除了“它看起来很丑”。考虑这些示例代码片段:

if (i == 1) 
    i++;
    doSomething();

while (i < 1)
    i++;
    doSomething();

如果您不仔细阅读这些内容，您的眼睛会误以为 doSomething(); 是有条件地调用的……由于缩进不正确。它发生了。

(旁白:因为误读代码而将某人标记为“无能”是没有帮助的。如果你正拼命修复一个阻碍展示的错误并且你已经连续工作了 14 个小时，那么你很可能会错过这个那种事情。并不是因为你无能。一旦你遇到过几次这种情况，教训就会深入......)

好的，现在假设您运行一个自动工具来添加大括号。你会得到的是:

 if (i == 1) {
    i++;
 }
    doSomething();

 while (i < 1) {
    i++;
 }
    doSomething();

和原来的代码意思完全一样。 但是 ...如果原始代码实际上是一个错误怎么办？如果程序员打算 doSomething() 调用是有条件的怎么办？

简而言之，通过自动添加大括号，我们掩盖了最初程序员的意图，并使这些错误更难追踪。

---

底线 - 我认为谨慎手动检查这些事件中的每一个……而不是自动“修复”它们。事实上，我认为如果您没有时间或耐心手动检查它们，最好不要管代码。最好关闭警告... IMO。