desenvolvedor-io / dev-store

A microservices e-commerce reference application built with ASP.NET 6
https://devstore.academy
MIT License
1.05k stars 199 forks source link

fix: removendo o dapper do repositório de produto #8

Closed Lucasbk123 closed 2 years ago

Lucasbk123 commented 2 years ago

Feito apenas na API de catalog

EduardoPires commented 2 years ago

Olá @Lucasbk123,

Por algum motivo não consegui ver as alterações. Tive que fazer um clone do projeto.

Algumas considerações essenciais:

  1. Utilizar o x.Name.Contains(query) ao invés de EF.Functions
    catalogQuery = catalogQuery.Where(x => EF.Functions.Like(x.Name, $"%{query}%"));
  2. Apenas uma variavel com AsQueryable(). E a partir dessa vc pode utilizar tanto pra query do Catalog quanto do total.

Considerações opcionais

  1. Utilizar o Where junto da query principal, sem ter a necessidade de fazer um if para verificar se a query é null.
  2. Utilizar o Conventional Commits

O que acha?

Lucasbk123 commented 2 years ago

Boa tarde @EduardoPires

obrigado pelo o feedback, vou começa a utilizar o Conventional commits como padrão. Em questão de dois Asqueryable realmente não precisava, vou munda isso.

1.Sobre utilizar o Contains não utilizei por que ele monta a query com dois condicional, porem não tem nenhum problema e coisa minha mesmo..

2.Sobre tira o IF, caso eu utilize o Contains é a query for null o resultado vai vim Vazio..Select gerado pelo o Entity framework:

image

Ja utilizado EF.Functions.Like ele vai consegui trazer o resultado igual quando estava com dapper sem o IF image

brunobritodev commented 2 years ago

Ok, tira o if e mantem o EF.Functions!

brunobritodev commented 2 years ago

:tada: This PR is included in version 1.1.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: