Transform a match to avoid duplicate

Question

Is there a construct which allows to drop the repetition of the 0.0 routine below

val price: Option[Double] = ???

price match {
  case Some(d) =>  
    if (isPriceFair(d))  
      d  
    else  
      0.0  
  case _ =>  
    0.0  
}

You can move the if to the case, you could also use filter + getOrElse. — Luis Miguel Mejía Suárez
– Luis Miguel Mejía Suárez, Commented Jun 24 at 17:14
have just edited the question - it does not allow to move the if into the case — IUnknown
– IUnknown, Commented Jun 24 at 17:16

Dmytro Mitin · Accepted Answer · 2025-06-24 17:57:09Z

4

Try matching with a pattern guard:

price match {
  case Some(d) if d > 0 => d 
  case _                => 0.0  
}

price match {
  case Some(d) if isPriceFair(d) => d 
  case _                         => 0.0  
}

Please see https://docs.scala-lang.org/tour/pattern-matching.html#pattern-guards

Or

price.filter(_ > 0).getOrElse(0.0)

price.filter(isPriceFair).getOrElse(0.0)

https://scastie.scala-lang.org/DmytroMitin/C5LtgVbuSg6BsQKp2BqFqQ/1

edited Jun 24 at 17:57

answered Jun 24 at 17:17

Dmytro Mitin

53.1k3 gold badges34 silver badges76 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Transform a match to avoid duplicate

1 Answer 1

Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Related