Как можно улучшить код?

Question

[Speakermen]

Вижу дубликаты

this.user = await this.usersService.findOne({ email: createAuthDto.email });

но как можно улучшить подобный код использовать Фабрику, Стратегию? Есть ещё идея разнести ещё на одни отдельные модули

AuthModule
--LoginModule REST
--RegisterModule REST

import {
  BadRequestException,
  Injectable,
  UnauthorizedException,
} from '@nestjs/common';
import { User } from '@prisma/client';
import * as bcrypt from 'bcrypt';
import { UsersService } from '../users/users.service';
import { CreateLoginDto } from './dto/create-login.dto';
import { CreateAuthDto } from './index';

@Injectable()
export class AuthService {
  private user: User;

  public constructor(private readonly usersService: UsersService) {}

  private async validateEmail(createAuthDto): Promise<void> {
    this.user = await this.usersService.findOne({ email: createAuthDto.email });

    if (this.user && this.user.email === createAuthDto.email) {
      throw new BadRequestException('Choose another email');
    }

    createAuthDto.password = await bcrypt.hash(createAuthDto.password, 12);
  }

  private async validatePassport(createLoginDto): Promise<void> {
    this.user = await this.usersService.findOne({
      email: createLoginDto.email,
    });

    if (
      !this.user ||
      !(await bcrypt.compare(createLoginDto.password, this.user.password))
    ) {
      throw new UnauthorizedException('Email or password is incorrect');
    }
  }

  public async register(createAuthDto: CreateAuthDto): Promise<User> {
    await this.validateEmail(createAuthDto);

    return await this.usersService.create(createAuthDto);
  }

  public async login(createLoginDto: CreateLoginDto): Promise<User> {
    await this.validatePassport(createLoginDto);

    return this.user;
  }
}

Comments

[Victor Bomberow]

Укажите язык в теге, пожалуйста.

Применение паттерна не обязательно улучшает код, а может усложнить, но зато организовывает дальнейшее внесение изменений. Если паттерн применение по делу

У вас есть идея размножить сущности и разделить на другие модули - вы прежде должны найти доказательства в существующей кодовой базе, что это нужно сделать. Если вам просто хочется так сделать, потому что так написано в книге или статье, то это однозначно ухудшение кода.

[Speakermen]

Victor Bomberow, TypeScript Спасибо я видел подобное в laravel на php. Хочется добиться чтобы код читался как книга)

[Victor Bomberow]

Speakermen, извиняюсь за оффтоп, мой опыт говорит, что подобной чистоты можно добиться только в инструментальном коде - велосипедный, но маленький модуль, библиотека или фреймворк с жёстко ограниченной моделью данных / множеством сущностей. Если код про прикладное решение, то он всегда испорчен отложенным рефакторингом вызывающего контекста, и как правило, оказывается, что нельзя просто так взять и отрефакторить бяку за часок.

Answer 1

[Aetae]

Про паттерны-фигаторны ничего не скажу, а вот то что метод начинающийся на validate не должен ничего никуда писать - это факт.)

Answer 2

[Сергей Песков]

private async validateEmail(createAuthDto): Promise

private async validatePassport(createLoginDto): Promise

Указать типы для createAuthDto и createLoginDto

if (this.user && this.user.email === createAuthDto.email) {
throw new BadRequestException('Choose another email');
}

Ну и думаю данную проверку(this.user.email === createAuthDto.email) можно убрать, вы же делаете выборку из usersService по email, поэтому если был получен не null, то пользователь с таким email уже существует.

Comments

[Speakermen]

Спасибо Я вот думаю вместо Auth module. Разбить на Login, Register Module и использовать createLoginDto
createRegisterDto