Question

这是我之前的question：

的后续内容

假设我正在重构这样的函数：

def check(ox: Option[Int]): Unit = ox match {
  case None => throw new Exception("X is missing")
  case Some(x) if x < 0 => throw new Exception("X is negative")
  case _ => ()
}

我正在编写一个新的纯函数doCheck来返回Unit或异常。

case class MissingX() extends Exception("X is missing")
case class NegativeX(x: Int) extends Exception(s"$x is negative")

import scalaz._, Scalaz._

type Result[A] = Excepiton \/ A

def doCheck(ox:Option[Int]): Result[Unit] = for {
  x <- ox toRightDisjunction MissingX()
  _ <- (x >= 0) either(()) or NegativeX(x)
} yield ()

然后从check

调用它

def check(ox:Option[Int]): Unit = doCheck(ox) match {
  case -\/(e) => throw e
  case _ => ()
}

有意义吗？像这样实施doCheck会更好吗？

def doCheck(ox:Option[Int]): Result[Int] = for {
  x1 <- ox toRightDisjunction MissingX()
  x2 <- (x1 >= 0) either(x1) or NegativeX(x1)
} yield x2

如何使用cats实现它？

Answer 1

你会在猫身上做同样的事情，只有猫本身没有Boolean => Xor[A, B]语法，就像来自scalaz的either () or ()一样。

import cats.data.Xor
import cats.implicits._

def doCheck(ox: Option[Int]): Xor[Exception, Unit] =
  ox.toRightXor(MissingX()).flatMap(x => if(x > 0) ().right else NegativeX(x).left)

您可以使用mouse，它为猫提供类似的语法助手：

import com.github.benhutchison.mouse.boolean._

ox.toRightXor(MissingX()).flatMap(x => (x > 0).toXor(NegativeX(x), ()))

Xor也有方法ensure来执行此类操作，但如果谓词不成立，它也无法访问该元素。如果您不需要x NegativeX，那么您可以写下：

ox.toRightXOr(MissingX()).ensure(Negative())(_ > 0).void

如何重构一个抛出Scalaz或Cats异常的函数

1 个答案: