3+ Ways to Write Clean Code in JavaScript

[@dcode-software]

OK, I did mess up the "if statement" regarding the hours and light/dark mode. Sorry about that. It probably should be an OR instead.

[@mrTiberiuDubau]

DOM IS BACK!!!! 🙂

[@eip408]

Amazing stuff Dom! We need more of this sheesh 🔥 Thanks man for giving out valuable content as always🥂

[@saschamajewsky7990]

I would have written something like: function getThemeColors() { const currentHour = new Date.getHours(); const darkTheme = { textColor: "#fff", backgroundColor: "#222222"}; const lightTheme = { textColor: "#222222", backgroundColor: "#fff"}; return (currentHour > 20 && currentHour < 7) ? darkTheme : lightTheme; }

[@christian-schubert]

Also numbers.forEach(number => 12 % number === 0 && console.log(`${number} is a factor of twelve!`)); 😎 ...if it's really the shortening of our codebase that we're after. Might not be as performant though

[@hassansyed6087]

I like it. It's clean

[@marcinzale]

Very good advices. Thank you.

[@EnzoAuditore]

Thank you for this.

[@danielfrimu1996]

what theme are you using??

[@beinyourguard]

really cool!

[@castletown999]

Thought provoking as usual - thank you. But I do disagree with one of your solutions. I was taught that functions should have just one entry point and one exit point. Why? Imagine say you have a function that looks in a file for something. On entry it opens the file, and on exit it closes it again. If you exit in the middle you could forget to close it and cause a resource leak. In general a function on entry has some setup to do, and on exit knocks it down again. Returning form the middle is just asking for trouble. A maintainer may add something towards the end of the function, not realizing that it can exit earlier and not run the new code.

[@magicorpse]

Sorry for the question, but why at 13:15, there is not need to write names.map(name => capitalize(name)); is it a javascript shortcut? thanks.

[@pgd724]

I was thinking the same thing!

[@dcode-software]

Using the parenthesis means that you're calling function. If you pass in "capitalize" without the parenthesis, it's passing a reference to the function (which is what we want)

[@magicorpse]

@@dcode-software Thanks for the reply and the video, dcode.

[@Pareshbpatel]

Thanks for the Clean-Code tips, Dom! {2023-10-21}, {2024-03-25}

[@mofmse]

I try to avoid concatenate. const capitalize = function(name, locale = 'en') { return name.replace(/^./, name.at(0).toLocaleUpperCase(locale)); }

[@DroidNexus]

Dcode tenho uma dúvida. No appgyver Com a api da xano eu criei um aplicativo e adicionei dois métodos de login, um para cliente e outro para empresas, com isso eu criei duas tabelas de base de dados na database, uma para a conta pessoal e outra para conta da empresa. Até aí tudo bem. Tudo da parte de cadastro funcionou. Mas eu queria criar uma data base média para fazer upload de imagem do aplicativo, Da foto de perfil de usuário. MAS ISSO PARA CADA USUÁRIO. para as duas contas tanto de empresa quanto de usuário eu queria que eles fizessem upload de imagem na mesma database, de novo até aí tudo bem. O PROBLEMA É que quando eu quero que ele busque a media_id do id da imagem na database ele precisa do token de autenticação, o problema é que toda a vez quando tento, dá acesso não autorizado no aplicativo. Ele não pega o token de autorização da conta empresa, Isso quando eu faço pela conta da empresa! Só pega a do cliente que foi a que eu fiz primeiro. Queria criar um aplicativo duas databases diferentes uma para empresa e uma para o cliente. Mas quando eu quero que ele busque id pra colocar na tabela da média ele da erro de autenticação access unaltorizhed.

[@maxdevjs3457]

Am I wrong or should be "7 < currentHour)" (eventually "currentHour > 7)") ?

[@Fatali.Fataliyev]

currentHour < 7 explanation: Current hour less than 7.

[@maxdevjs3457]

@@Fatali.Fataliyev ​ I guess that that && in reality is a || 🤔

[@Fatali.Fataliyev]

@@maxdevjs3457 In this case?

[@maxdevjs3457]

@@Fatali.Fataliyev yes. If I am not missing something, currentHour can not be less than 7 and, simultaneously, to be greater than 20. Right...?

[@Fatali.Fataliyev]

@@maxdevjs3457 Yes.

[@andrew_ortega89]

“currentHour > 20 && currentHour < 7” is how we say it. In a program, however, this will always be false. But I see the point anyway, thank you!

[@dcode-software]

Yep I messed this up. I've added a pinned comment :) thanks.

[@suelingsusu1339]

👌👌🙏🙏👍👍🌹🌹🖖🖖🖖🖖