【问题标题】:Scalastyle Boolean expression can be simplifiedScalastyle 布尔表达式可以简化
【发布时间】:2016-03-29 12:01:05
【问题描述】:

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

我只是不明白我应该如何让它变得更简单,是否有一些奇怪的 Option[Boolean] 比较我错过了?

【问题讨论】:

  • 我遇到了同样的问题。似乎是 ScalaStyle 中的一个错误。在这种情况下,更改为中缀必须避开他们的正则表达式。对于这种情况,使用 contains 似乎是最简单的方法。
  • 有一个新的问题与相关主题:Simplifying Option[Boolean] expression in Scala

标签: scala scala-collections scalastyle


【解决方案1】:

似乎建议您与方法调用用法保持一致。中缀形式的所有内容:

(t contains true) && (f contains false)

或常规方法调用形式的所有内容:

t.contains(true).&&(f.contains(false))

【讨论】:

    【解决方案2】:

    使用您的值,t.contains(true).&&(f.contains(false)) 始终返回 true。因此,您可以通过简单地编写true 来简化它,即只执行print 而没有if 条件。

    【讨论】:

    • 抱歉,实际代码不使用编译时已知的值。我添加了另一个示例来澄清
    猜你喜欢
    • 1970-01-01
    • 2016-07-13
    • 2016-02-20
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多