{"id":21015297,"url":"https://github.com/laisfrigerio/who-wrote-this-shitty-code","last_synced_at":"2025-05-15T05:31:58.221Z","repository":{"id":45658579,"uuid":"408231803","full_name":"laisfrigerio/who-wrote-this-shitty-code","owner":"laisfrigerio","description":":hankey: Antes e depois de trechos de códigos refatorados.","archived":false,"fork":false,"pushed_at":"2022-03-10T15:14:52.000Z","size":1793,"stargazers_count":18,"open_issues_count":0,"forks_count":0,"subscribers_count":2,"default_branch":"main","last_synced_at":"2025-04-03T04:33:14.907Z","etag":null,"topics":["clean-code","code-smells","good-code","javascript","shitty-code"],"latest_commit_sha":null,"homepage":"","language":"JavaScript","has_issues":true,"has_wiki":null,"has_pages":null,"mirror_url":null,"source_name":null,"license":"mit","status":null,"scm":"git","pull_requests_enabled":true,"icon_url":"https://github.com/laisfrigerio.png","metadata":{"files":{"readme":"README.md","changelog":null,"contributing":null,"funding":null,"license":"LICENSE","code_of_conduct":null,"threat_model":null,"audit":null,"citation":null,"codeowners":null,"security":null,"support":null}},"created_at":"2021-09-19T20:39:09.000Z","updated_at":"2022-07-17T01:24:04.000Z","dependencies_parsed_at":"2022-08-04T18:00:40.509Z","dependency_job_id":null,"html_url":"https://github.com/laisfrigerio/who-wrote-this-shitty-code","commit_stats":null,"previous_names":[],"tags_count":0,"template":false,"template_full_name":null,"repository_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories/laisfrigerio%2Fwho-wrote-this-shitty-code","tags_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories/laisfrigerio%2Fwho-wrote-this-shitty-code/tags","releases_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories/laisfrigerio%2Fwho-wrote-this-shitty-code/releases","manifests_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories/laisfrigerio%2Fwho-wrote-this-shitty-code/manifests","owner_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/owners/laisfrigerio","download_url":"https://codeload.github.com/laisfrigerio/who-wrote-this-shitty-code/tar.gz/refs/heads/main","host":{"name":"GitHub","url":"https://github.com","kind":"github","repositories_count":254282173,"owners_count":22045110,"icon_url":"https://github.com/github.png","version":null,"created_at":"2022-05-30T11:31:42.601Z","updated_at":"2022-07-04T15:15:14.044Z","host_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub","repositories_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories","repository_names_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repository_names","owners_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/owners"}},"keywords":["clean-code","code-smells","good-code","javascript","shitty-code"],"created_at":"2024-11-19T10:09:23.675Z","updated_at":"2025-05-15T05:31:53.205Z","avatar_url":"https://github.com/laisfrigerio.png","language":"JavaScript","funding_links":[],"categories":[],"sub_categories":[],"readme":"# 💩 Who wrote this shitty code?\n\nAo encontrar um código antigo me fiz a seguinte pergunta \"Quem escreveu este código, meu deus?\"\n\nAdivinha quem era a autora? Yes, Baby. Eu mesma.\n\nEntão surgiu a ideia de compartilhar trechos de códigos que eu já desenvolvi e mostrar como eu reescreveria atualmente. Demonstrar que com estudos, leituras e, claro, muita prática a evolução vem.\n\n## Example 1\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-1/who-wrote-this-shitty-code.png\" alt=\"#1 Who wrote this shitty code?\" title=\"#1 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\n**Contexto deste código**: era um formulário de cadastro com nome, e-mail, cpf, telefone e senha. Todos estes campos eram obrigatórios. Desenvolvi a validação do formulário utilizando javascript com jQuery. A validação ocorria toda vez que o usuário pressionava o botão de submit. O botão só ficava desabilitado quando estava ocorrendo a ação de salvar o registro na base de dados.\n\nPontos de evolução do código que podemos perceber após a refatoração:\n\n- Evitei a estrutura if/else e utilizei o padrão de retorno;\n- Evitei a estrutura de if alinhados (um if dentro do outro);\n- Evitei usar o operador de negação \"!\" nas condições (if). Por isso alterei os nomes das funções de validações. Exemplo: `isValidEmail` renomeei para `isInvalidEmail`;\n- Utilizei nomes de funções mais descritivas;\n- No código da esquerda é possível verificar que o trecho de código para exibir a mensagem de erro está duplicado. A ação de habilitar o botão para clique também está duplicada. Na reescrita optei por criar dois métodos para cada ação: `showErrorMessage` e `enableButton` respectivamente.\n\n## Example 2\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-2/who-wrote-this-shitty-code.png\" alt=\"#2 Who wrote this shitty code?\" title=\"#2 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\nPodemos dizer que \"Clean Code\" é a arte de dar significado ao seu código.\n\nUma das inúmeras maneiras é nomear as funções e variáveis adequadamente para dar mais clareza para o propósito do seu código.\n\nAlguns outros pontos de melhorias que é possível observar:\n\n- Uso de objetos como parâmetro de função ao invés de N argumentos;\n- Não uso de if aninhados (um if dentro de outro)\n- Não uso da estrutura else-if, else\n- Padrão de retorno inicial\n\nAlém disso, sempre tirar o que de melhor que a linguagem de programação usada fornece. Ex: Desestruturação, interface\n\nAcredito que a Lais do futuro vai conseguir olhar esse trecho e torná-lo ainda melhor. Mas por ora é essa evolução que consigo evidenciar.\n\n## Example 3\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-3/who-wrote-this-shitty-code.png\" alt=\"#3 Who wrote this shitty code?\" title=\"#3 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\n🤔 A 4 meses atrás eu comecei a publicar conteúdo no Linkedin.\n\n💡Meu foco desde de então é trazer ideias de como tornar nosso código mais legível, de fácil entendimento, utilizando boas práticas da linguagem.\n\nAcontece que essa evolução é constante. Por exemplo, este código \"ruim\", eu escrevi a mais ou menos 5 meses atrás, ou seja, bem recente.\n\nE ontem (25/10/2021) eu estava revendo este trecho e identifiquei que era possível melhorá-lo.\n\n✔️ Veja que no primeiro trecho de código a minha condição (if) está relativamente grande. Ao utilizar a função includes, eu diminuí a quantidade de condições OR (que poderiam evoluir e tornar essa condição ainda maior).\n\n🚀 Assim, é sempre importante estudar a linguagem de programação e usar o melhor que ela pode nos oferecer.\n\n❗ Para mais informações sobre o método includes: [Documentação](https://developer.mozilla.org/pt-BR/docs/Web/JavaScript/Reference/Global_Objects/Array/includes)\n\n## Example 4\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-4/who-wrote-this-shitty-code.png\" alt=\"#4 Who wrote this shitty code?\" title=\"#4 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\n💡 Você provavelmente já escreveu um código e depois de um tempo ao revê-lo percebeu o qual ruim estava, certo?\n\nSe isso aconteceu e/ou ainda acontece é um bom sinal: você está evoluindo sua capacidade de programação.\n\nE é justamente esta a intenção do post: mostrar como eu desenvolvi e como reescreveria o mesmo trecho de código.\n\nO conteúdo de hoje faz parte do \"quarto episódio\" da série \"Who wrote this shitty code\".\n\nE, a partir da imagem, conseguimos fazer as seguintes pontuações:\n\n👉  Evitar o uso de ifs alinhados diminui a complexidade do código;\n\n👉  Evitar escrever um trecho de código com mais de 80 caracteres em uma linha. O ideal é sempre o trecho de código estar visível sem a necessidade de scroll lateral;\n\n👉  Estude/acompanhe as features da linguagem de programação e use o que de melhor ela pode nos oferecer. Neste exemplo, eu usei o recurso Operador de encadeamento opcional. O seu uso faz com que não seja necessário verificar cada referência, resultando em expressões mais curtas e simples.\n\n❗ Leia mais sobre o operador de encadeamento opcional [aqui](https://lnkd.in/d843vMyj)\n\nSe gostou, deixe seu starzinho ⭐ aqui repositório!!\n\n📷 Para mais conteúdo, segue meu insta: https://lnkd.in/d6Bxw4HV\n\n## Example 5\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-5/1.png\" alt=\"#5 Who wrote this shitty code?\" title=\"#5 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-5/2.png\" alt=\"#5 Who wrote this shitty code?\" title=\"#5 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-5/3.png\" alt=\"#5 Who wrote this shitty code?\" title=\"#5 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-5/4.png\" alt=\"#5 Who wrote this shitty code?\" title=\"#5 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\n\u003cp align=\"center\"\u003e\n  \u003ca\u003e\u003cimg src=\"./example-5/5.png\" alt=\"#5 Who wrote this shitty code?\" title=\"#5 Who wrote this shitty code?\"\u003e\u003c/a\u003e\n\u003c/p\u003e\n\n\n## :woman: Author\n\n| [\u003cimg src=\"https://avatars.githubusercontent.com/u/20709086?v=4\" width=\"100px;\" alt=\"Lais Frigério\"/\u003e\u003cbr /\u003e\u003csub\u003e\u003cb\u003e@laisfrigerio\u003c/b\u003e\u003c/sub\u003e](https://github.com/laisfrigerio)\u003cbr /\u003e |\n| :---: |\n\n## 📄 License\n\nThis project is licensed under the MIT License - see the LICENSE.md file for details","project_url":"https://awesome.ecosyste.ms/api/v1/projects/github.com%2Flaisfrigerio%2Fwho-wrote-this-shitty-code","html_url":"https://awesome.ecosyste.ms/projects/github.com%2Flaisfrigerio%2Fwho-wrote-this-shitty-code","lists_url":"https://awesome.ecosyste.ms/api/v1/projects/github.com%2Flaisfrigerio%2Fwho-wrote-this-shitty-code/lists"}