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()"
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); }
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)) }
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?
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
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.
Muito bom!!! Obrigado e parabéns!!!
tira do titulo java script. Isso é aplicavel a qualquer linguagem. Top!
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 😃
Obrigado amigo pelo conteúdo!!
Nós que agradecemos!
Vídeo super relevante. Valeu!
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);
}
Valeu Mayk.
Show vc é 10! Sou teu fã! 👍
Podia analisar qualquer trecho do kernel do Linux sob a ótica do "Clean Code"
🤣🤣🤣🤣
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
Maykão é brabo!
Conteúdo incrível!
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?
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
Em 7:00, oque quer dizer com trazer uma documentação para a linha de código ?
Faz um vídeo sobre o Vlang
Precisava ver esse vídeo kkk
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.
Mantenha isso estúpido, seu limpo! hahah ótimo video. Valeuuuu
Pode dar o aumento pro Maikao 👍🏻
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!
fala maykao
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
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!