Refatorar as controllers

[PauloGoncalvesBH]

No description provided.

[eliasreis54]

Olá @PauloGoncalvesBH tudo bem?
Algum ponto em específico necessita ser analisado ou seria algo mais genérico mesmo? Talvez eu possa ajudar nessa issue.

[PauloGoncalvesBH]

Oi @eliasreis54, o que acontece é que as controllers estão totalmente confusas e nunca foram alteradas desde a primeira implementação.

Exemplo da controller POST /carrinhos:
https://github.com/ServeRest/ServeRest/blob/trunk/src/controllers/carrinhos-controller.js#L23

Se analisar esse método vai perceber que o código está seguindo nenhuma boa prática e algumas implementações poderiam ir para outras camadas, como service, para tornar esse código mais limpo e claro.

O escopo dessa issue seria:

1. Identificar quais controllers precisam de refatoração para o código ficar mais limpo.
2. Abrir PR para cada controller refatorada.

O código é 100% coberto de testes e com bom índice de teste de mutação, então qualquer mudança de comportamento será facilmente detectada, tornando a refatoração mais fácil de ser feita.

[eliasreis54]

Entendido perfeitamente!
Irei atuar nessa issue.

[PauloGoncalvesBH]

perfeito @eliasreis54, se tiver qualquer dúvida pode me chamar aonde preferir (github, linkedin, etc).
Te coloquei como assignee da issue.

[edumaxsantos]

Alguma atualização de status quanto a refatoração?

[leoGalani]

Alguma atualização de status quanto a refatoração?

Isso está afetando o seu uso? Se sim... PR sao bem vindas ;)

[PauloGoncalvesBH]

oi @edumaxsantos, não teve PR para essa refatoração e acabo não tendo muito tempo para dedicar com esse assunto específico já que é feito nas horas vagas.
Estou procurando novos colaboradores, caso queira colaborar essa é uma boa task para pessoas que querem começar no mundo do open source.
O material como contribuir possui um guia de como pode iniciar com essa issue.
Se tiver dúvidas sobre pode me acionar que auxílio a prosseguir com o PR dessa issue.
Se interessar me deixe sabendo através de comentário na issue para te colocar como assignee.

[edumaxsantos]

Olá! Estou dando uma olhada no que posso fazer. Já fiz algumas refatorações iniciais, mas ainda não está boa. Vou continuar mexendo nisso no meu tempo livre.

[PauloGoncalvesBH]

oi Edu, nesse caso o que acha de abrir um PR para eu ir dando uma olhada enquanto implementa?

[edumaxsantos]

Boa noite! Criei um draft #303 de PR para o carrinhos-controller, específico para o método POST. Como é a primeira vez que envio um PR para projeto open source, peço que me deem um feedback do que poderia melhorar

[PauloGoncalvesBH]

A controller de POST de /carrinhos foi refatorada e entregue, obrigado @edumaxsantos

[edumaxsantos]

Boa noite! Criei um draft com mais refatoração para o controller de carrinhos. A maior parte da refatoração foi mover qualquer coisa relacionada com outros serviços para carrinhosService.

[edumaxsantos]

Acredito que já seria interessante continuar as refatorações aplicando também a tradução para inglês do que for alterado nos controllers com base na issue #50.

[PauloGoncalvesBH]

concordo contigo @edumaxsantos, o que acha de entrar na discussão da issue de como deve ser feita essa tradução procurando o mínimo impacto para os atuais usuários do ServeRest em PT-BR?

[edumaxsantos]

Vou entrar, sim.