deesouza/challenge-02-js-expert

This is a code challenge where you are supposed to implement the given functions taking into consideration the Javascript Lifecycle and other particularities of the language.

Fala Devs! 👨‍💻

Eu sou o Diego. Atualmente trabalho como Desenvolvedor Frontend e Backend.

Posts and Articles

DevTo

• How to intercept a request with Axios
• Mocking com o MSW - Mock Service Worker
• Container & Presentational Pattern - React

Medium

• Como integrar Laravel, NodeJS + Express + Socket.IO
• Criando um BOT no Telegram com AdonisJS
• 1. Criando Site com AdonisJS — Estrutura
• 2. Criando Site com AdonisJS — Controllers e Rotas

Social

• DevTo
• Medium
• Stackoverflow
• Linkedin
• Youtube

Skills

---

challenge-02-js-expert's Issues

Correção do desafio

Implementação

• Completou com êxito o desafio principal e o sub-desafio!
• Excelente padrão de nomenclatura de funções e variáveis!
• Ótimo uso de clean code e organização criando inclusive estruturas novas para complementar ainda mais o desafio!

Possíveis pontos de melhoria

• Vale a pena dar uma olhada no padrão de commits, que é como você documenta o processo de desenvolvimento. 😉 Como você já apresenta ótimos pontos positivos em termos de desenvolvimento, polir ainda mais os detalhes que complementam isso sem dúvidas dá um "up" ainda maior na sua imagem profissional. o/

ObjectMethodsService

Pontos fortes

• Implementação impecável
• Ótimo domínio de Coerção de tipos e Objects Lifecycle implementando soluções usando valueOf, toString e [Symbol.toPrimitive]
• Boa decisão em usar a criação de um objeto types como visto em aula
  https://github.com/DeeSouza/jsexpert-challenge02-lifecycle/blob/87353086265006357377e37fdea9e772b91a5549/src/service/ObjectMethodsService.js#L36
• Excelente uso de clean code e reaproveitamento de código criando a função formattedAndMergeValues para fazer a conversão do objeto. Inclusive, bem interessante sua decisão em fazer dessa uma função mais genérica (que serviria para qualquer objeto)! Isso sem dúvidas seria extremamente útil para padronização de projeto, levando o desenvolvimento para um outro nível num cenário real!
  https://github.com/DeeSouza/jsexpert-challenge02-lifecycle/blob/87353086265006357377e37fdea9e772b91a5549/src/service/ObjectMethodsService.js#L48

Possíveis pontos de melhoria

• Como você demonstrou um super interesse em nomenclatura de variáveis/funções e já está num outro nível de polimento das próprias habilidades, acho interessante levantar como um ponto de melhoria o padrão no uso de verbos na nomenclatura, que funciona mais ou menos assim: O método "faz algo" e a variável onde você armazena esse resultado "sofre a ação", então, pegando como exemplo esse cenário do formattedAndMergeValues:

  • Implementação atual:

  Aqui você declara a função como formattedAndMergeValues
  https://github.com/DeeSouza/jsexpert-challenge02-lifecycle/blob/87353086265006357377e37fdea9e772b91a5549/src/service/ObjectMethodsService.js#L48

  E logo em seguida você chama ela, atribuindo ela ao stringifyObject
  https://github.com/DeeSouza/jsexpert-challenge02-lifecycle/blob/87353086265006357377e37fdea9e772b91a5549/src/service/ObjectMethodsService.js#L31

  • Implementação aplicando o padrão de nomenclatura atualizado:

  Pensando na ação que a função faz, somente formatAndMergeValues (ou mesmo stringifyObject) seria um bom nome pra declarar a função

  function formatAndMergeValues() { ... }

  E aí na hora de chamar a função, pensando que o objeto sofre a ação, esse sim seria o stringifiedObject (ou formattedObject)

  E, inclusive, copyRawObject seria um nome de função, e o nome da variável ficaria algo tipo copiedRawObject ou rawObjectCopy

  const stringifiedObject = this.formatAndMergeValues(rawObjectCopy);

  Percebe como a leitura fica mais voltada ao que de fato a linha representa, e isso impacta na legibilidade, ficando algo como const objetoModificado = modificaObjeto(objetoCópia)? São detalhes bem simples mas que impactam bastante. :D

ReferencesService

Pontos fortes

• Implementação impecável
• Ótimo domínio de tipos de referência e Mutability vs Immutability no Javascript
• Ênfase novamente na excelente nomenclatura de variáveis aqui,
  https://github.com/DeeSouza/jsexpert-challenge02-lifecycle/blob/87353086265006357377e37fdea9e772b91a5549/src/service/ReferencesService.js#L15
  usando inclusive o padrão comentado um pouco acima nessa issue com maestria
  https://github.com/DeeSouza/jsexpert-challenge02-lifecycle/blob/87353086265006357377e37fdea9e772b91a5549/src/service/ReferencesService.js#L3

StatementsService

Pontos fortes

• Implementação impecável
• Ótimo domínio do uso de conditional statements no Javascript e seus respectivos retornos em diferentes cenários.

Possíveis pontos de melhoria

• Muito cuidado com dupla negação em statements, geralmente acaba que se você precisa de uma dupla negação na verificação, remover todas as negações (que seria o caso nessa função, inclusive) ou colocar apenas uma negação externa na sentença (!(condition1 && condition2)) já resolve.
  https://github.com/DeeSouza/jsexpert-challenge02-lifecycle/blob/87353086265006357377e37fdea9e772b91a5549/src/service/StatementsService.js#L15

WeirdMathService

Pontos fortes

• Implementação impecável
• Ótimo domínio do fluxo de vida do Javascript e coerções implícitas que ocorrem em expressões matemáticas.

Considerações finais

Concluiu com proficiência o Desafio principal e o sub-desafio, demonstrando uma preocupação excelente com nomenclatura de funções/variáveis e clean code. Mostrando assim possuir um ótimo domínio de princípios essenciais do Javascript e um padrão de polimento bem elevado das próprias habilidades. Parabéns, Diego! 🚀 Mandando ver mais uma vez aqui nos desafios, e espero contar contigo nos próximos também! 😄