有没有更好的方法来编写这段代码？

Question

我主要考虑测试可选值并提取这些值.这是代码:

val optEmail: Option[String] = getOptionalEmailParam
val optPassword: Option[String] = getOptionalPasswordParam

if (optEmail.nonEmpty && optPassword.nonEmpty) {
  doSomething(optEmail.get, optPassword.get)
} else {
  doSomethingElse
}

我喜欢这样的东西

for (email <- optEmail; password <- optPassword) {
  doSomething(email, password)
} else {
  // Didn't get inside the for loop
  doSomethingElse
}

Answer 1

我认为匹配声明在这里看起来很简洁:

val optEmail: Option[String] = getOptionalEmailParam
val optPassword: Option[String] = getOptionalPasswordParam

(optEmail, optPassword) match {
  case (Some(email),Some(password)) => doSomething(email, password)
  case _ => doSomethingElse
}