PR de correção #3
Conversation
Branch one
Arquitetura da aplicacao
There was a problem hiding this comment.
Olá, Pedro! Este é o review do seu código. Deixei alguns comentários ao longo do projeto. Você fez um projeto bem sólido, apesar das coisinhas apontadas mais pontualmente. Os principais pontos de atenção são sobre a consistência da arquitetura. Em alguns pontos você usou a camada de business, em outros não, algumas classes estão em arquivos separados, por exemplo. Outro pontinho é o excesso de espaços em branco, que acabam dificultando um pouco a leitura. De resto, muito bom, parabéns!
| public signup = async (name: string, email: string, password: string)=> { | ||
| if(!name || !email || !password) { | ||
| throw new Error('Insira todas as informações necessárias para o cadastro'); | ||
| }//regra de negocio |
There was a problem hiding this comment.
Lembre de remover os comentários!
| import { HashManager } from "../services/HashManager"; | ||
| import { IdGenerator } from "../services/IdGenerator"; | ||
|
|
||
| export class UserloginBusiness { |
There was a problem hiding this comment.
Talvez essa classe possa ser absorvida pela UserBusiness
| export const followUser = async (req: Request, res: Response) => { | ||
| try { | ||
| const token = req.headers.authorization as string; | ||
| const amizadeId = req.body.amizadeId; |
There was a problem hiding this comment.
Cuidado ao misturar idiomas nos nomes de variável.
| if(!amizadeId) { | ||
| throw new Error('Insira um id válido') | ||
| } |
| const user = await userDataBase.getUserByEmail(amizadeId); | ||
|
|
||
| if(!user) { | ||
| throw new Error('Usuário não existe') | ||
| } |
There was a problem hiding this comment.
Uma outra boa é checar se a amizade já existe antes de criar.
| const token = req.headers.authorization as string; | ||
| const authenticator = new Authenticator(); | ||
| const authenticationData = authenticator.verify(token); | ||
| const userId = authenticationData.id; | ||
|
|
||
| const idGenerator = new IdGenerator(); |
There was a problem hiding this comment.
Aqui seria interessante manter o padrão das classes de user e jogar as dependências para a classe de business.
|
|
||
|
|
||
|
|
||
|
|
| app.post('/signup', signup); | ||
| app.post('/login', login); | ||
| app.post('/post', newPost); | ||
|
|
There was a problem hiding this comment.
O feed acabou ficando sem endpoint.
Corrigindo o projeto.