Análise

[hladf]

Fala Luiz!

Segue abaixo uma análise do código com pontos positivos e negativos, mas o processo seletivo não foi encerrado ainda e quem vai entrar em contato falando das proximas etapas é o nosso RH, ok?

Se quiser explicar algo ou perguntar, pode usar os comentários aqui

pros:

• soube usar e seguir o exemplo de aplicação do reduxjs/toolkit
• soube centralizar os exports dos módulos pra organizar imports
• usou todos os hooks
• fez teste de render (apenas um)
• separou tipos e chamadas á api (mas poderia estar um pouco mais próximo do solid)

contras:

• (src\components\AddItem.tsx - btnClick) Nomenclatura de função abreviada e não condiz com o evento. Poderia ser: handleAddItem ou handleClick (Object Calisthenics - 6. Don’t Abbreviate)
• Não usou styled components
• tem umas condições erradas...
  • todo.duck.ts:12 (getTodoList) -> "&&" do if deveria ser "||"
• BaseUrl de 'client/config' não pega valor do ENV, sendo que são iguais
• algumas funções usam 2 arrow functions sem necessidade, não entendi o porque disso...
  • const deleteButtonClick = () => () => deleteAction();
  • ai no onClick precisou chamar a função com () por conta disso... onClick={deleteButtonClick()}
• A instancia do axios ta só com nome de 'Instance' e não ta em pasta http nem nada do tipo, por isso não tem um nome muito claro
• o db.json foi renomeado, por isso a api não funcionou aqui, tive que voltar para o nome padrão

[darknessxk]

Excelentes pontos, em contra parte o macro do twin.macro serve como Styled-components pela library usar ele como base, entao todos esse componentes chamando tw.div, span etc etc sao macros do styled-components

Mas quanto aos outros pontos estou totalmente de acordo com o que foi dito nos contras

Faltou da minha parte documentar apropriadamente o renaming do db.json explicando o motivo e etc pra ficar mais claro as minha razoes

E bom parece que sai melhor que o esperado do Object Calisthenics

Anotei todos os pontos no qual me sai mal pra corrigir em um proximo futuro, muito obrigado por dar esse feedback

[darknessxk]

Segue um bonus de algo que fiz no tempo que fiquei esperando o feedback de voces, fixado pra utilizar caso utilizem o antigo hotel agora conhecido agora como server-hub

https://github.com/katsuhirogg/server-hub/

Vou montar um private namespace em breve pra hospedar no git por hora estou em contato com o dono da package server-hub pra conseguir o nome e subir essa versao fixada do antigo projeto que nao funcionou!

[hladf]

Eu notei a semelhança grande do twin.macro com o styled-c, e isso foi um pequeno contra apenas, deu pra ver que conhece sobre. O que eu queria ter visto mais, por causa do styled, seria a reutilização de styles e o css utilizado nele, sem classes prontas

[darknessxk]

Compreendi, eu tentei optar pelo tailwind por conta de conveniencia de tempo, conhecimento ao redor do postcss / scss eu tenho, tanto que desenvolvi usando o tailwind por nao ser tao boilerplated como um Material UI, Bootstrap ou ate mesmo o Bulma.

Mas entendo o ponto em questao, sigo anotando os pontos pra conseguir fazer coisas mais simples no futuro hahaha