Domanda

In sostanza sto cercando per la maggior scala-come modo per fare quanto segue:

def sum(value1: Option[Int], value2: Option[Int]): Option[Int] = 
  if(value1.isDefined && value2.isDefined) Some(value1.get + value2.get)
  else if(value1.isDefined && value2.isEmpty) value1
  else if(value1.isEmpty && value2.isDefined) value2
  else None

Questo dà uscita corretta:

sum(Some(5), Some(3))  // result = Some(8)
sum(Some(5), None)     // result = Some(5)
sum(None, Some(3))     // result = Some(3)
sum(None, None)        // result = None

Ancora sommare più di due opzioni che avrei dovuto usare troppi ifs o utilizzare una sorta di loop.

Modifica-1:

Mentre scrivevo la questione mi si avvicinò con una sorta di risposta:

def sum2(value1: Option[Int], value2: Option[Int]): Option[Int] = 
  value1.toList ::: value2.toList reduceLeftOption { _ + _ }

Questo sembra molto idiomatica al mio occhio inesperto. Questo potrebbe anche lavorare con più di due valori. è ancora possibile fare lo stesso senza la conversione in liste?

Modifica-2:

Ho finito con questa soluzione (grazie a ziggystar ):

def sum(values: Option[Int]*): Option[Int] = 
  values.flatten reduceLeftOption { _ + _ }

Modifica-3:

Un altro grazie alternativi per Landei :

def sum(values: Option[Int]*): Option[Int] = 
  values collect { case Some(n) => n } reduceLeftOption { _ + _ }
È stato utile?

Soluzione

Come su:

scala> def sum(values: Option[Int]*): Option[Int] = values.flatten match {
     | case Nil => None                                                   
     | case l => Some(l.sum)                                              
     | }
sum: (values: Option[Int]*)Option[Int]

scala> sum(Some(1), None)
res0: Option[Int] = Some(1)

scala> sum(Some(1), Some(4))
res1: Option[Int] = Some(5)

scala> sum(Some(1), Some(4), Some(-5))
res3: Option[Int] = Some(0)

scala> sum(None, None)                
res4: Option[Int] = None

Modifica

Forse sarebbe sano per restituire 0 se tutti gli argomenti erano Nessuno. In questo caso la funzione si ridurrebbe a values.flatten.sum.

Altri suggerimenti

scala> def sum(a: Option[Int], b: Option[Int]) = (a,b) match {
     |   case (Some(x), Some(y)) => Some(x + y)
     |   case (Some(x), None) => Some(x)
     |   case (None, Some(y)) => Some(y)
     |   case _ => None
     | }
sum: (a: Option[Int],b: Option[Int])Option[Int]

scala> sum(Some(5), Some(3))
res0: Option[Int] = Some(8)

scala> sum(Some(5), None)
res1: Option[Int] = Some(5)

scala> sum(None, Some(3))
res2: Option[Int] = Some(3)

scala> sum(None, None)
res3: Option[Int] = None

Un'altra soluzione è:

def sum(values: Option[Int]*): Int = values.collect{case Some(n) => n}.sum

Mentre nel caso attuale flatten è chiaramente più conveniente, la versione collect è più flessibile, in quanto consente di eseguire mappature e di avere condizioni di filtro aggiuntive o modelli complessi. Per esempio. Immaginate di voler avere la somma dei quadrati di tutti i numeri pari in valori:

values.collect{case Some(n) if n mod 2 == 0 => n*n}.sum

Si può rendere molto conciso con il fatto che ci sia un'istanza Semigroup per Option che fa esattamente quello che vuoi. È possibile utilizzare scalaz o gatti . Ecco un esempio utilizzando cats:

import cats.std.option._
import cats.syntax.semigroup._
import cats.std.int._

Option(1) |+| Option(2) // Some(3)
Option(1) |+| None      // Some(1)
None      |+| Option(2) // Some(2)

Così il vostro sum diventa:

def sum(v1: Option[Int], v2: Option[Int]): Option[Int] = v1 |+| v2

soluzione ridotto di michael.kebe con un po 'un'occhiata ad alcune regole matematiche di base:

def sum(a: Option[Int], b: Option[Int]) = (a,b) match {
  case (None,None) => None
  case _ => Some(a.getOrElse(0)+b.getOrElse(0))
}

scala> sum(Some(5), Some(3))  // result = Some(8)
res6: Option[Int] = Some(8)

scala> sum(Some(5), None)     // result = Some(5)
res7: Option[Int] = Some(5)

scala> sum(None, Some(3))     // result = Some(3)
res8: Option[Int] = Some(3)

scala> sum(None, None)        // result = None
res9: Option[Int] = None
Autorizzato sotto: CC-BY-SA insieme a attribuzione
Non affiliato a StackOverflow
scroll top