Scala中安全解Option并避免类型不匹配的方法

问题原因 #

你的findBarWithOptions函数出现类型不匹配，核心原因是：Option(f.bars).map(_.find(_.s == barToFind))返回的是Option[Option[Bar]]，但flatMap要求传入的函数必须返回Option[Bar]（而非嵌套的双层Option）。

find方法本身就会返回Option[Bar]，用map包裹它时会再套一层Option，最终导致类型不符合flatMap的要求。

修正方案 #

方案1：替换map为flatMap #

将Option(f.bars)后的map改成flatMap，直接把嵌套的Option[Option[Bar]]扁平化为Option[Bar]，满足flatMap的类型要求：

def findBarWithOptions(foo: Foo, barToFind: String): Option[Bar] = {
  Option(foo).flatMap(f => Option(f.bars).flatMap(_.find(_.s == barToFind)))
}

方案2：使用for推导式（可读性更强） #

Scala的for推导式能让嵌套的Option操作逻辑更清晰，和上面的写法本质等价：

def findBarWithOptions(foo: Foo, barToFind: String): Option[Bar] = {
  for {
    f <- Option(foo)
    bars <- Option(f.bars)
    bar <- bars.find(_.s == barToFind)
  } yield bar
}

方案3：用flatten处理嵌套Option #

也可以保留map，后续用flatten展开双层Option，但这种写法不如前两种符合Scala惯用风格：

def findBarWithOptions(foo: Foo, barToFind: String): Option[Bar] = {
  Option(foo).flatMap(f => Option(f.bars).map(_.find(_.s == barToFind)).flatten)
}

额外优化建议 #

在纯Scala场景中，应尽量避免使用null。可以给Foo的bars字段设置默认空序列，从根源上减少空值处理逻辑：

case class Foo(bars: Seq[Bar] = Seq.empty[Bar])
case class Bar(s: String)

def findBar(foo: Option[Foo], barToFind: String): Option[Bar] = {
  foo.flatMap(_.bars.find(_.s == barToFind))
}

内容的提问来源于stack exchange，提问作者candied_orange