If...else 有问题

Question

我有这个user.model

// user.model.ts

interface User {
  name: string
  email: string
  password: string
  is_active: boolean
}

我会在使用电子邮件注册之前检查电子邮件是否被占用并且用户是否处于活动状态。

    // If email is taken and is_active is true
    if (found && found.is_active) {
      return next(Boom.badRequest('Email is taken'))
    } else if (found && !found.is_active) {
      // If email is taken and is_active is false, ask to signin
      return next(Boom.unauthorized('Please signin to activate'))
    } else {
      const created = await service.create(data)
      res.send('OK')
    }

但问题是，当我尝试使用已注册并处于活动状态的电子邮件 (some@one.com) 时，它显示 Email is taken 是正确的。然后，当我尝试使用另一封 已注册但未激活的电子邮件 (any@one.com) 时，它尝试插入数据并破坏服务器，因为 email 列是 unique 是数据库（Postgres）。

我又试了一次，但这次是 (any@one.com) 第一和 (some@one.com) 第二。它在第二种情况下再次破裂。

我想问题是它只适用于一种情况。

我也试过switch...case，也不起作用。

    switch (true) {
      case found && found.is_active:
        // If email is taken and is_active is true
        return next(Boom.badRequest('Email is taken'))
      case found && !found.is_active:
        // If email is taken and is_active is false, ask to signin
        return next(Boom.unauthorized('Please signin to activate'))
      default: {
        const created = await service.create(data)
        res.send('OK')
      }
    }

请帮忙，我该如何解决这个问题？

Answer 1

你为什么要这样使用if 语句？

if (found) {
  if (found.is_active) {
    // already in use by active user
    return next(Boom.badRequest('Email is taken'))
  } else {
   // in use, but inactive
   return next(Boom.unauthorized('Please signin to activate'))
  }
else {
// not found, must create
  const created = await service.create(data)
  res.send('OK')
}