Как зарефакторить данный код?

Question

[Alexander]

Как можно предотвратить дублирования кода с ошибкой?
Буду благодарен за помощь

async login(email: string, password: string) {
    const user = await User.findOne({ where: { email } })

    if (!user) {
      throw ApiError.BadRequest('Your email or password was entered incorrectly.')
    }
    const isPassEquals = await bcrypt.compare(password, user.password)
    if (!isPassEquals) {
      throw ApiError.BadRequest('Your email or password was entered incorrectly.')
    }
    const userDto = new UserDto(user)
    const tokens = tokenService.generateTokens(userDto.id, userDto.email)
    await tokenService.saveToken(userDto.id, tokens.refreshToken)
    return { ...tokens, user: userDto }
  }

Comments

[WbICHA]

const errorMessage = 'Your email or password was entered incorrectly.';

Answer 1

[Василий Банников]

Например можно вынести код по валидации в отдельную функцию:

async findUserByEmailAndPassword(email: string, password: string): User {
    const user = await User.findOne({ where: {email} });
    if(user) {
        const passwordValid = await bcrypt.compare(password, user.password);
        if(passwordValid)
            return user;
    }
    return null;
}

И потом в месте использования:

const user = await findUserByEmailAndPassword(email, password);
if(user === null)
  throw ApiError.BadRequest('Your email or password was entered incorrectly.');

Ну или можно await bcrypt.compare(password, user.password) затащить в if, но мне такой вариант не очень нравится.

Comments

[Alexander]

Спасибо большое

[Василий Банников]

Alexander, обновил ответ. Мне больше нравится вариант с выносом функции.
С || выглядит слишком громоздко

Answer 2

[Tony]

async login(email: string, password: string) {
    const user = await User.findOne({ where: { email } })

    var isPassEquals = false;
    if (user) {
    	isPassEquals = await bcrypt.compare(password, user.password)
    }

    if (!user || !isPassEquals) {
      throw ApiError.BadRequest('Your email or password was entered incorrectly.')
    }
    const userDto = new UserDto(user)
    const tokens = tokenService.generateTokens(userDto.id, userDto.email)
    await tokenService.saveToken(userDto.id, tokens.refreshToken)
    return { ...tokens, user: userDto }
  }