我的工作需要开发一个小型Java应用程序，用于解析非常大的XML文件（~300k行）以选择非常特定的数据（使用 Pattern ），因此我正在尝试对其进行一些优化。 我想知道以下两个片段哪个更好：
if (boolean_condition && matcher.find(string)) {
...
}
或者
if (boolean_condition) {
if (matcher.find(string)) {
...
}
}
其他细节：
boolean_condition 是一个布尔值，它在每次迭代中使用外部函数计算而来boolean 设置为 false，我就不需要测试正则表达式是否匹配谢谢您的帮助。
我遵循的一个黄金法则是尽可能"避免嵌套"。但如果以此为代价使我的单个 if 条件语句过于复杂，我不介意进行嵌套。
此外，您正在使用短路&&运算符。因此，如果布尔值为假，则它甚至不会尝试匹配！
所以，
if (boolean_condition && matcher.find(string)) {
...
}
这是正确的方法！
if(x != null && x.isY()); 如果 && 没有短路评估，那么当 x == null 时，这将在你的面前爆炸。 - user以下两种方法：
public void oneIf(boolean a, boolean b)
{
if (a && b)
{
}
}
public void twoIfs(boolean a, boolean b)
{
if (a)
{
if (b)
{
}
}
}
使用第一种写法和第二种写法在方法体中生成的字节码完全相同，因此不会有任何性能差异，这只是一种风格上的选择（个人偏好第一种写法）。
public void noIfs(boolean a, boolean b) {} 也能产生相同的字节码，我不会感到惊讶的。;-) - Axel两种方式都可以，如果第一个条件为假，则不会测试第二个条件。
使用使代码更可读和易懂的方式。对于仅有两个条件的情况，第一种方式更加逻辑清晰和易读。但是如果有5或6个条件使用 &&, || 和 ! 进行连接时，情况可能就不同了。
我建议将您的表达式提取到一个语义有意义的变量中，然后将其传递给您的评估。而不是：
if (boolean_condition && matcher.find(string)) { ... }
将表达式赋值给一个变量，然后计算该变量：
const hasItem = boolean_condition && matcher.find(string)
if (hasItem) { ... }
使用这种方法，即使是最复杂的评估也可以保持可读性：
const hasItem = boolean_condition && matcher.find(string)
const hasOtherThing = boolean_condition || boolean_condition
const isBeforeToday = new Date(string) < new Date()
if (hasItem && hasOtherThing && isBeforeToday) { ... }
Java使用短路运算符处理布尔运算符，因此这两种变体在功能上是相同的。因此，如果boolean_condition为false，则不会继续匹配。
最终，你需要选择更易于阅读和调试的方式，但如果最终括号数量太多，则深度嵌套可能会变得难以控制。
如果条件变得更长，一种改善可读性的方法是将其拆分成多行：
if(boolean_condition &&
matcher.find(string))
{
...
}
我经常看到太多的&&和||被串在一起形成逻辑混乱，往往是微妙错误的源头。
很容易就会认为在正确的位置加入另一个&&或||，从而破坏现有的逻辑。
因此，作为一个通用规则，我尽量不使用它们，以避免随着需求的变化加入更多的内容。
第一个建议是尽量避免这样的if嵌套，我认为这是不好的风格/丑陋的代码，并且&&会短路，只有在布尔值为真时才会测试matcher.find()。
就性能而言，它们是相同的。
在这段代码中几乎可以肯定占用时间的是matcher.find(string)，因为它是一个函数调用。
if (boolean_condition && matcher.find(string)) {
...
}