【Scala】Play 2.0 Scala で Option のネストを綺麗にしたい【初心者】

gistfile1.scala

def gabu(id: String, bid: String) = Action {
  Project.findOneByID(id) match {
    case Some(project: Project) => Build.findOneByID(new ObjectId(bid)) match {
      case Some(build: Build) =>
        // etc...
        Ok(views.html.gabu(build)
      case None => NotFound
    }
    case None => NotFound
  }
}

これはまだ2個だけど、3個、4個になることもしばしば。

https://gist.github.com/2382341
こちらが参考になると思います。

ありがとうございます＞＜

def gabu(id: String, bid: String) = Action {
  (for {
    project <- Project.findOneByID(id).toRight(NotFound).right
    build <- Build.findOneByID(new ObjectId(bid)).toRight(NotFound).right
  } yield {
    // etc...
    Ok(views.xml.gabu(build))
  }).merge
}

こんな感じになりました。
for内包表記って便利ですね！

EitherActionを使うと読みやすくなりますよ。

https://gist.github.com/2626167

上記の例だと、こんな感じです。
projectを引き継ぐように少し変えてみました。

def gabu(id: String, bid: String) = EitherAction {req =>
  id >>= {
    Project.findOneByID(_) match {
      case Some(project: Project) => project  継続
      case None => NotFound                   終了
    }
  } >>= { project => 
    Build.findOneByID(new ObjectId(bid)) match {
      case Some(build: Build) => 
        Ok(views.html.gabu(project, build)) 終了
      case None => NotFound                終了
    }
  }
}

うおお、こんな書き方もできるんですね。

そうなんです。サンプルだとprojectを使ってないコードになってしまっていますが、
実際のコードはprojectを引き継いで使いたいのでツッコミ非常にありがたいです。

正常系、異常系が読みやすくなっていいですね！

アドバイスありがとうございます＞＜