1

我正在尝试编写一个接收我的 UserModel 的函数，并执行一些检查以查看用户是否

  1. 被锁住了
  2. 在一段时间内没有尝试过多的登录。

然后返回结果的布尔指示。

查找在我的身份验证过程中工作。但是我想分解确定是否允许用户（尝试）登录的代码，这样我就可以在多个地方使用它而无需重复代码。

但是（对于 Vapor/Swift 来说是新手）我遇到了一个错误，我无法弄清楚我做错了什么： 无法将“EventLoopF​​uture”类型的返回表达式转换为“Bool”类型

错误位于}.all().map {行（单独成一行，因此更容易找到）。

数据库结构方面，我涉及 2 个表：

  • UserAccess 保存我的用户配置文件详细信息（此用户可以进行多少次错误尝试，以及我们在日志中查找登录尝试的时间有多长）
  • UserLog，其中记录了每个用户的登录尝试以及他们何时进行尝试

到目前为止，这是我的代码片段：

func CanUserLogin(user: UserModel, req: Request) -> EventLoopFuture<Bool> {
  if(!(user.locked ?? false)) {
    let userProfileId = user.userprofile

    return Usertype.query(on: req.db)
      .filter(\.$profilenum == userProfileId)
      .first().map { useraccess in
        let badloginperiod = Double((useraccess!.badloginperiod ?? 0) * -1 * 60) // convert minutes to seconds (we need a negative number)
        let lookUpDate = Date().addingTimeInterval(badloginperiod)

        return Userlog.query(on: req.db).group(.and) {
          and in
          and.filter(\.$username == user.username)
          and.filter(\.$datecreated >= lookUpDate)
        }.all().map {
          UserLogs -> Bool in
          let value = userLogs.count

          // the account is locked or the max attempts for the time peroid
          if(value >= (useraccess.maxloginattempts ?? 3)) {
            return false
          } else {
            return true
          }
        }
    }
  }
}

任何方向将不胜感激。

4

1 回答 1

2

您试图EventLoopFuture从map块中返回，但您只能从中返回non-future值。因此，map您不必使用flatMap查询Usertype。

签出此代码

func canUserLogin(user: UserModel, req: Request) -> EventLoopFuture<Bool> {
    guard user.locked != true else {
        return req.eventLoop.makeFailedFuture(Abort(.badRequest, reason: "User is locked"))
    }
    let userProfileId = user.userprofile
    return Usertype.query(on: req.db)
        .filter(\.$profilenum == userProfileId)
        .first()
        .unwrap(or: Abort(.forbidden, reason: "No access"))
        .flatMap { userAccess in
            let badloginperiod = Double((useraccess.badloginperiod ?? 0) * -1 * 60) // convert minutes to seconds (we need a negative number)
            let lookUpDate = Date().addingTimeInterval(badloginperiod)
            return Userlog.query(on: req.db).group(.and) {
              $0.filter(\.$username == user.username)
              $0.filter(\.$datecreated >= lookUpDate)
            }.all().map { attempts -> Bool in
                // the account is locked or the max attempts for the time peroid
                if attempts.count >= (userAccess.maxloginattempts ?? 3) {
                    return false
                } else {
                    return true
                }
            }
    }
}
于 2020-06-04T06:25:40.567 回答