0

私の Playframework プロジェクトには、次の DAO、サービス オブジェクト、およびコントローラー パターンがあります。

CredentialsDAO.scala:

def find(authType: AuthType.Value, authAccountId: String) =
    collection.find(
      Json.obj(Fields.AuthType → authType,
        Fields.AuthAccountId → authAccountId)).one[Credentials].recover(wrapLastError)

CredentialsService.scala:

def checkEmailPassword(email: String, password: String) = credentialsDAO.find(AuthType.EmailPassword, email).map(_.get) //Needs review

私のコントローラーで:

def auth = Action.async(parse.json) { request =>
    {

      val authRequest = request.body.validate[AuthRequest]
      authRequest.fold(
        errors => Future(BadRequest),
        auth => {
          //First verify username and password
          val authRequestResult = for {
            validCredential <- credentialsManager.checkEmailPassword(auth.email, auth.password)
            validPassword <- BCrypt.checkFuture(auth.password, validCredential.passwordHash)
            validAccount <- accountManager.getAccount(validCredential)
            session <- sessionManager.createSession(validAccount.id, validAccount.roles)

            touchedSession <- sessionManager.touchSession(session.id)

          } yield AuthResponse(session.id, session.token, validAccount.id, validAccount.roles)

          authRequestResult map {
            case res: AuthResponse => Ok(Json.toJson(res))
            case _ => NotFound

          }
        })
    }
  }

質問: この「パターン」は、関数型プログラミングの観点から見て問題ありませんか? 特に、次の行が気になりCredentialsService.scalaます。

def checkEmailPassword(email: String, password: String) = credentialsDAO.find(AuthType.EmailPassword, email).map(_.get) //Needs review

私は Scala を初めて使用しますが_.get、上記の行を処理するためのより良い方法があると思いますか? 任意の提案/アイデアをいただければ幸いです。

前もって感謝します。

4

0 に答える 0