Agregar registro si el valor de retorno es ninguno

Question

Supongamos que hay dos funciones findUser(id:String):Option[User] y findAddress(user:User):Option[Address] Invocado de la siguiente manera:

for(user <- findUser(id); address <- findAddress(user)) println(address)

Ahora me gustaría agregar un registro de errores a esto for-comprehension. Me gustaría llamar a un log(msg:String) función si cualquiera user o address no se encuentra.

for(user <- findUser(id) ifNone log("user not found"); 
    address <- findAddress(user) ifNone log("address not found")) 
       println(address)

¿Puedo hacerlo sin cambiar las firmas de función?

Answer 1

Quizás

implicit def withIfNone[A](o: Option[A]) = new {
  def ifNone(action: => Unit) = { if (o == None) action; o }
}

También puede considerar usar en lugar de opción (o convertir sus opciones a cualquiera). Que no funcionaría con un foreach (a para sin rendimiento), pero podría hacerlo

for(
  a <- option1.toRight("option1 missing").right; 
  b <- option2.toRight("option2 missing").right)
yield f(a,b)

Luego puede coincidir con el resultado con el resultado con

case Left(error) => log (error)
case Right(result) => // use result

Answer 2

Ascensor Box es una clase más apropiada para su caso de uso. A Box es como un Option, pero con dos estados vacíos: OK y Error. Podrías usarlo así:

val addr = for {
  user <- findUser(id) ?~ "user not found"
  address <- findAddress(user) ?~ "address not found"
} yield address

address match {
  case Full(addr) => println(addr)
  case oops: Failure => println(oops.msg) // see Failure for more details
}

Ver este blog Para varias sugerencias relacionadas con su problema.

Answer 3

Puede ser una exageración, pero se parece bastante a lo que querías;)

object Extensions {
  // You need a wrapper since Option is sealed
  class OptionWrapper[E](option: Option[E]) {
    def foreach[U](f: E => U) {
      option foreach f
    }
    def isEmpty = option.isEmpty
  }

  // Modification trait for OptionWrapper
  trait ErrorLogging[E] extends OptionWrapper[E] {
    abstract override def foreach[U](f: E => U) {
      if (isEmpty)
        println("error")
      else
        super.foreach(f)
    }
  }

  // Accessor for the new mixin
  def log[E](option: Option[E]) = new OptionWrapper(option) with ErrorLogging[E]
}

object TestingLogger extends App {
  case class User(address: String)
  def findUser(id: Int): Option[User] = if (id == 1) Some(User("address")) else None
  def findAddress(user: User): Option[String] = Some(user.address)

  import Extensions._

  for {
    user <- log(findUser(1)) // prints out address
    address <- log(findAddress(user))
  } println(address)

  for {
    user <- log(findUser(2)) // prints out error
    address <- log(findAddress(user))
  } println(address)
}

Si no tienes idea de lo que acaba de pasar, lee este.