Muito maneiro, outra dica é criar um teste unitário antes de iniciar uma refatoração, assim garante o mesmo comportamento ou resultado esperado de antes da refatoração, isso ajuda em components mais complexos ou até em simples, sempre tem algum detalhe q podemos esquecer!
Muito boas as ideias passadas! Amei o video! Não si se to viajando, mas em 11:53 não entendi a necessidade de confirmar o `confirmPass` (linha 28) pra fazer a renderização do componente, já que a passagem pra essa function em si já é consequência de existir um `confirmPass` (linha 11). Mesma coisa na linha 47, acho que tiraria o `confirmPass` e deixaria só a checagem do `passwordMatchRegex`. Com isso acho que faria mais uma function para determinar essa frase de senha aí. Uma dúvida que me surgiu é se teria problema declarar essas functions dentro do próprio componente, em vez de por fora. Tem diferença de performance, é um padrão mais comum, tá mais acostumado a fazer assim, ou tem algum outro motivo por ter declarado como um outro componente separado?
Olha, eu tô mais acostumado a colocar fora, principalmente pq ajuda pra vc ter testes isolados dessa função, tu só importa é testa, mas vai de gosto /o n tem certo ou errado
Muito bom! Parabéns!! :D Sobre a parte que voce fala sobre o webstorm (~1:30) tem um plugin para vscode que simula essa funcionalidade, chamado (Inline Parameters for VSCode) não é tão boa quanto o nativo do webstorm, mas quebra um bom galho. :D
Excelente conteúdo cara!, se possível mais videos de refatoração. Mas tenho que dizer que a refatoração do cara do twitter ficou coisa linda kkkk Uma opinião é diminuir mais a fonte/zoom facilita mais a visualização do exemplo como um todo
Me liguei nisso também. O código inicial já vem com uma válidação na :10 pra entrar quando `confirmPass` for undefined. Dessa forma ali dentro ele vai mostrar somente a string vazia pois nunca passaria pela :12. Mas votlando e vendo o código no Twitter vi que seria !== de undefined. Foi um erro na hora de copiar o código pro VSCode. O Mario acaba trocando na refatoração, mas em relação ao código original, é o que seria correto. No final deu certo 😂
Muito maneiro, outra dica é criar um teste unitário antes de iniciar uma refatoração, assim garante o mesmo comportamento ou resultado esperado de antes da refatoração, isso ajuda em components mais complexos ou até em simples, sempre tem algum detalhe q podemos esquecer!
NOSSA, TOTAL! Incllusive ta um ótimo video essa dica
Show demais!
Muito bom
Vídeo excelente de refatoração, vlw sr. Souto por trazer conteúdos incríveis!!
Muuuito obrigado pelo feedback Sr Cristiano 🌟
Seus vídeos são incriveis!!! Seria uma boa pra 2023 trazer videos sobre testes no frontend ;)
Já tá no radar! Inclusive fica o convite pra entrar no discord do canal
Muito boas as ideias passadas! Amei o video!
Não si se to viajando, mas em 11:53 não entendi a necessidade de confirmar o `confirmPass` (linha 28) pra fazer a renderização do componente, já que a passagem pra essa function em si já é consequência de existir um `confirmPass` (linha 11). Mesma coisa na linha 47, acho que tiraria o `confirmPass` e deixaria só a checagem do `passwordMatchRegex`.
Com isso acho que faria mais uma function para determinar essa frase de senha aí.
Uma dúvida que me surgiu é se teria problema declarar essas functions dentro do próprio componente, em vez de por fora. Tem diferença de performance, é um padrão mais comum, tá mais acostumado a fazer assim, ou tem algum outro motivo por ter declarado como um outro componente separado?
Tem um ponto! Eu fui tentando só seguir o código mesmo nem fui olhando tanto o que tava rolando
Olha, eu tô mais acostumado a colocar fora, principalmente pq ajuda pra vc ter testes isolados dessa função, tu só importa é testa, mas vai de gosto /o n tem certo ou errado
Muito bom! Parabéns!! :D
Sobre a parte que voce fala sobre o webstorm (~1:30) tem um plugin para vscode que simula essa funcionalidade, chamado (Inline Parameters for VSCode) não é tão boa quanto o nativo do webstorm, mas quebra um bom galho. :D
Massa!!! Não sabia desse plugin no vscode
@@DevSoutinho O VsCode tem a configuração inlayHint para quase todas as linguagens. É uma configuração nativa.
Muitooo bom o vídeo Mario, gostei demais dessa pegado de boas práticas no react, poderia ter mais vídeos de boas práticas, exelente coteúdo 😄🎉
Olha que eu faço em ahahha
Conteúdo de qualidade ;)!!!
Vaaaaleu \ooo/
Excelente conteúdo cara!, se possível mais videos de refatoração. Mas tenho que dizer que a refatoração do cara do twitter ficou coisa linda kkkk
Uma opinião é diminuir mais a fonte/zoom facilita mais a visualização do exemplo como um todo
Valeu pela dica! E o will manda bem demais!!! nao a toa trabalha na netflix hahaha
esse pedido de like do nalu ai eim hahah
parabéns pelo vídeo
nalu rei do tibia asuhsahuas e dos pedido de like
Parece que o pessoal só conhece strategy.😅
O If tá errado não? Antes era cpwrd == undef, com o If ficou o contrário?
Qual if? Tem uns 70 mil Ifs nesse código 🥲
Mas tá passivo de erro 😂 ainda mais pq o correto seria ter escrito um teste
Me liguei nisso também. O código inicial já vem com uma válidação na :10 pra entrar quando `confirmPass` for undefined. Dessa forma ali dentro ele vai mostrar somente a string vazia pois nunca passaria pela :12. Mas votlando e vendo o código no Twitter vi que seria !== de undefined. Foi um erro na hora de copiar o código pro VSCode.
O Mario acaba trocando na refatoração, mas em relação ao código original, é o que seria correto. No final deu certo 😂