【问题标题】:Scala collections: Is there a safe map operation?Scala 集合:有安全的地图操作吗?
【发布时间】:2012-04-26 21:00:40
【问题描述】:

假设我有一个列表(或地图中的值),我想对每个项目执行操作。但不幸的是,无论出于何种原因,这个值列表都可能包含空值。

scala> val players = List("Messi", null, "Xavi", "Iniesta", null)
players: List[java.lang.String] = List(Messi, null, Xavi, Iniesta, null)

为了避免被 NPE 炸毁,我需要执行以下操作:

scala> players.filterNot(_ == null ).map(_.toUpperCase)
res84: List[java.lang.String] = List(MESSI, XAVI, INIESTA)

有没有更好的方法?

最好是这样的:

players.safeMap(_.toUpperCase)

scala-language mailing list,Simon 提出了这个:

players.filter ( null !=).map(_.toUpperCase )

这是我最初拍摄的较短版本,并且在没有专用方法的情况下尽可能短。

更好的是,Stefan 和 Kevin 提出了 withFilter 方法,该方法将返回一个惰性代理,因此可以合并两个操作。

players.withFilter ( null !=).map(_.toUpperCase )

【问题讨论】:

  • 如果星期二有任何迹象,球员名单根本不是类型安全的,更不用说在 Scala 中要避免 null ;-)
  • 这可能适用于您的用例,也可能不适用,但是:如果您永远不会使用该列表中有 null 值的事实,我建议您删除他们一劳永逸地您将其分配给val players。之后你永远不需要考虑它们;没有collect,没有隐含,没有NPE。
  • @Debilski 有时您的调用代码可能格式不正确。在混合 Java + Scala 应用程序中尤其如此。
  • @virtualeyes,null 的身份已被伪装以保护他们的形象:P

标签: scala scala-collections


【解决方案1】:

如果您无法避免 nulls(例如,如果您从 Java 代码中获取列表),另一种选择是使用 collect 而不是 map

scala> players.collect { case player if player != null => player.toUpperCase }
res0: List[java.lang.String] = List(MESSI, XAVI, INIESTA)

【讨论】:

  • 它的优点是一次通过,所以目前看来是一个不错的解决方案。但是,它会稍长一些。有什么看起来更接近我建议的 safeMap(f) 的吗?
  • @fracca - 如果类型注释比空测试短,您可以{ case player: String => player.toUpperCase }。对于匹配类型的东西,它不能是null
  • @fracca 你可以使用withFilter避免两次通过,如players.withFilter(null!=).map(_.toUpperCase)
【解决方案2】:

我会这样做:

players flatMap Option map (_.toUpperCase)

但这比collect 更糟糕。 filter + map 总是用collect 做得更好。

【讨论】:

  • hmmm,这不能用 2.9.1 编译
  • @LuigiPlinge 是的,类型参数有些问题。 players flatMap (Option(_)) map (_.toUpperCase) 工作。
【解决方案3】:

你可以转换成Option[String]的列表:

scala> val optionPlayers = players.map(Option(_))
optionPlayers: List[Option[java.lang.String]] = List(Some(Messi), None, Some(Xavi), Some(Iniesta), None)

Option 普遍优于null,它在如何安全处理数据方面为您提供了很大的灵活性。以下是获得所需结果的简单方法:

scala> optionPlayers.collect { case Some(s) => s.toUpperCase }
res0: List[java.lang.String] = List(MESSI, XAVI, INIESTA)

scala> optionPlayers.flatMap(_.map(_.toUpperCase))
res1: List[java.lang.String] = List(MESSI, XAVI, INIESTA)

scala> optionPlayers.flatten.map(_.toUpperCase)
res2: List[java.lang.String] = List(MESSI, XAVI, INIESTA)

您可以在其他 StackOverflow 问题中或通过网络搜索找到有关 Option 的更多信息。

或者,您始终可以将您想要的 safeMap 方法定义为 List 的隐含方法:

implicit def enhanceList[T](list: List[T]) = new {
  def safeMap[R](f: T => R) = list.filterNot(_ == null).map(f)
}

所以你可以这样做:

scala> players.safeMap(_.toUpperCase)
res4: List[java.lang.String] = List(MESSI, XAVI, INIESTA)

虽然如果您定义一个隐式,您可能希望使用 CanBuildFrom 样式,就像基本集合所做的那样,使其不仅仅适用于 List。您可以在其他地方找到更多相关信息。

【讨论】:

  • 然后我会为相同的输出做这一切:val optionPlayers = players.map(Option(_))optionPlayers.map(_.map(_.toUpperCase))optionPlayers.map(_.map(_.toUpperCase)).flatten
  • @fracca,请参阅我的建议,以从您的问题中获得相同的输出。
  • 选项确实是可取的,但是,在某些情况下您已经有了空值(例如,与编写不佳的 Java 代码交互)。转换为选项然后应用操作太冗长了。我想要一些真正简洁的东西来隐藏所有的噪音。
  • @fracca,请参阅我关于仅定义隐式 safeMap 方法的建议。这绝对是你的代码中看起来最干净的东西。
  • 我认为你是对的,隐式方法可能是最干净的解决方案。但是,我希望库中已经定义了这样的解决方案。可能是添加到库中的一个很好的候选操作?
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2013-07-06
  • 2010-11-18
  • 2011-06-19
  • 2012-01-30
  • 1970-01-01
相关资源
最近更新 更多