Scalastyle 布尔表达式可以简化
Scalastyle Boolean expression can be simplified
Scalastyle(intellij 2016.1 默认值)说这个布尔表达式可以简化
val t = Option(true)
val f = Option(false)
if(t.contains(true) && f.contains(false)) {
println("booop")
}
我可以通过将 if 更改为:
来摆脱这个问题
if(t.contains(true).&&(f.contains(false)))
或者将 && 更改为 &
但并没有真正看到这是如何简化它的,谁能解释一下这是怎么回事?
更新
它似乎与编译时是否已知 val 或它们是在本地定义无关。下面的代码也得到了表达式可以被简化的警告:
object TestFoo {
def bar(t: Option[Boolean]) = {
val f = Option(scala.util.Random.nextBoolean)
if (t.contains(true) && f.contains(false)) println("booop")
}
def main(args: Array[String]) = bar(t = Option(scala.util.Random.nextBoolean))
}
我只是不明白我应该如何让它变得更简单,是否有一些奇怪的选项[布尔]比较我错过了?
根据您的价值观,t.contains(true).&&(f.contains(false))
总是 returns true
。所以你可以通过简单地写 true
来简化它,即只执行 print
而没有 if
条件。
似乎建议您与方法调用用法保持一致。中缀形式的所有内容:
(t contains true) && (f contains false)
或常规方法调用形式的所有内容:
t.contains(true).&&(f.contains(false))
Scalastyle(intellij 2016.1 默认值)说这个布尔表达式可以简化
val t = Option(true)
val f = Option(false)
if(t.contains(true) && f.contains(false)) {
println("booop")
}
我可以通过将 if 更改为:
来摆脱这个问题if(t.contains(true).&&(f.contains(false)))
或者将 && 更改为 &
但并没有真正看到这是如何简化它的,谁能解释一下这是怎么回事?
更新 它似乎与编译时是否已知 val 或它们是在本地定义无关。下面的代码也得到了表达式可以被简化的警告:
object TestFoo {
def bar(t: Option[Boolean]) = {
val f = Option(scala.util.Random.nextBoolean)
if (t.contains(true) && f.contains(false)) println("booop")
}
def main(args: Array[String]) = bar(t = Option(scala.util.Random.nextBoolean))
}
我只是不明白我应该如何让它变得更简单,是否有一些奇怪的选项[布尔]比较我错过了?
根据您的价值观,t.contains(true).&&(f.contains(false))
总是 returns true
。所以你可以通过简单地写 true
来简化它,即只执行 print
而没有 if
条件。
似乎建议您与方法调用用法保持一致。中缀形式的所有内容:
(t contains true) && (f contains false)
或常规方法调用形式的所有内容:
t.contains(true).&&(f.contains(false))