uma frase pra resumir e lembrar fácil: se precisa explicar, então da pra refatorar 15:14 em vez de "!!", acho que ficaria mais claro colocando dentro de um "Boolean()"
9:34 Essa forma de verificar se um arquivo termina com a extensão permitida pode possibilitar algo indesejado, como um arquivo "testepng.txt" já que a validação que está sendo feita é se a extensão está incluída no nome do arquivo. Uma validação melhor seria utilizar o método "endsWith()" da seguinte forma: const isImageMayk = filename => { let allowedImages = [".jpg", ".png", ".svg"] return !!allowedImages.find(ext => filename.endsWith(ext)) }
Eu fiquei com uma dúvida. Pq usar: const isImageMayk = filename => { let allowedImages = [".jpg", ".png", ".svg"] return !!allowedImages.find(ext => filename.includes(ext)) } Sendo que posso usar simplesmente: const isImageMayk = filename => { let allowedImages = [".jpg", ".png", ".svg"] return !!allowedImages.includes(filename) } Qual é a brisa de usar esse find nesse caso? o que influenciaria?
o includes é para ver se a extensão está presente no filename, e o find para que sejam avaliadas todas as extensões do array. No seu segundo exemplo só retornaria true se o filename inteiro estivesse no array de allowedImages. O includes pode ser usado com arrays, mas, no exemplo do Mayk, está sendo usado para encontrar uma substring dentro de uma string maior
Por volta de 21:16, nas linhas 12 e 13 const date = new Date() Log.add(date, "signed up", email) Não seria interessante já chamar a função new Date dentro do Log.add? Assim: Log.add(new Date(), "signed up", email) E até um caso seria mais interessante que o outro?
e ai maykao excelente video outra forma de fazer seria assim sem o uso do find. const isImageRefactored_ = (filename) => { let allowedImages = ['jpg', 'png', 'svg']; const extFile = filename.slice(-3); return allowedImages.includes(extFile); }
Poxa apagaram meu comentário que eu escrevi com tanto carinho sobre o erro na funcao isImage. Parabens pela prática de apagar os comentarios construtivos.
Cara, to trampando numa empresa meio grande, e por mais que eu conheça formas mais limpas de escrever a mesma coisa que todo mundo ja sabe fazer, as vezes nao é tao interessante aplicar, pois a cultura da empresa ainda não se alinhou com a forma mais atual de escrever
O que acaba ocasionando em um passo a mais do usual, onde eu tenho quase a obrigação de difundir o que eu tenho de experiência com os meus colegas de trabalho, na minha opinião a nossa profissão teve uma grande evolução no que diz respeito as soft skills
Conteúdo excelente! Formar os novos programadores já com a consciência da excelência de código. Que venha muito mais desse conteúdo.
uma frase pra resumir e lembrar fácil: se precisa explicar, então da pra refatorar
15:14 em vez de "!!", acho que ficaria mais claro colocando dentro de um "Boolean()"
Total! Baita refatoração que traria muito mais clareza pro código 💛
Faria sentido usar o some(), ele retorna um boolean se encontrar algum dentro do array.
Gostei, entendi muito! 😄 Continue com o ótimo trabalho👍
Obrigado 😃
Show vc é 10! Sou teu fã! 👍
Vídeo super relevante. Valeu!
9:34
Essa forma de verificar se um arquivo termina com a extensão permitida pode possibilitar algo indesejado, como um arquivo "testepng.txt" já que a validação que está sendo feita é se a extensão está incluída no nome do arquivo.
Uma validação melhor seria utilizar o método "endsWith()" da seguinte forma:
const isImageMayk = filename => {
let allowedImages = [".jpg", ".png", ".svg"]
return !!allowedImages.find(ext => filename.endsWith(ext))
}
const isImageMayk = filename => {
let allowedImages = [".jpg", ".png", ".svg"]
return allowedImages.some(ext => filename.endsWith(ext))
}
usando some, seria melhor
tira do titulo java script. Isso é aplicavel a qualquer linguagem. Top!
Obrigado amigo pelo conteúdo!!
Nós que agradecemos!
Eu fiquei com uma dúvida.
Pq usar:
const isImageMayk = filename => {
let allowedImages = [".jpg", ".png", ".svg"]
return !!allowedImages.find(ext => filename.includes(ext))
}
Sendo que posso usar simplesmente:
const isImageMayk = filename => {
let allowedImages = [".jpg", ".png", ".svg"]
return !!allowedImages.includes(filename)
}
Qual é a brisa de usar esse find nesse caso? o que influenciaria?
const isImageMayk = filename => {
let allowedImages = [".jpg", ".png", ".svg"]
return allowedImages.some(ext => filename.endsWith(ext))
}
usando some, seria melhor
o includes é para ver se a extensão está presente no filename, e o find para que sejam avaliadas todas as extensões do array. No seu segundo exemplo só retornaria true se o filename inteiro estivesse no array de allowedImages.
O includes pode ser usado com arrays, mas, no exemplo do Mayk, está sendo usado para encontrar uma substring dentro de uma string maior
Conteúdo incrível!
Podia analisar qualquer trecho do kernel do Linux sob a ótica do "Clean Code"
🤣🤣🤣🤣
Maykão é brabo!
Por volta de 21:16, nas linhas 12 e 13
const date = new Date()
Log.add(date, "signed up", email)
Não seria interessante já chamar a função new Date dentro do Log.add? Assim:
Log.add(new Date(), "signed up", email)
E até um caso seria mais interessante que o outro?
Valeu Mayk.
e ai maykao excelente video outra forma de fazer seria assim sem o uso do find.
const isImageRefactored_ = (filename) => {
let allowedImages = ['jpg', 'png', 'svg'];
const extFile = filename.slice(-3);
return allowedImages.includes(extFile);
}
Precisava ver esse vídeo kkk
Faz um vídeo sobre o Vlang
Em 7:00, oque quer dizer com trazer uma documentação para a linha de código ?
Poxa apagaram meu comentário que eu escrevi com tanto carinho sobre o erro na funcao isImage. Parabens pela prática de apagar os comentarios construtivos.
Top ❤️
meus códigos eram tão poluídos que eu me perdia conforme o código ia crescendo, Quando ia ver no outro dia então... esquece KKKKKKKK.
A gente só vai evoluindo com o tempo!
Mantenha isso estúpido, seu limpo! hahah ótimo video. Valeuuuu
Cara, to trampando numa empresa meio grande, e por mais que eu conheça formas mais limpas de escrever a mesma coisa que todo mundo ja sabe fazer, as vezes nao é tao interessante aplicar, pois a cultura da empresa ainda não se alinhou com a forma mais atual de escrever
O que acaba ocasionando em um passo a mais do usual, onde eu tenho quase a obrigação de difundir o que eu tenho de experiência com os meus colegas de trabalho, na minha opinião a nossa profissão teve uma grande evolução no que diz respeito as soft skills
fala maykao
Pode dar o aumento pro Maikao 👍🏻
desnecessário o inglês, não agregou nada à sua explicação, e ainda se complicando pra escrever, então pra que?
Obrigado pelo feedback, Everton!