Correção do Desafio!

[WellsSA]

Implementação

• Completou com êxito o desafio principal!

src/entity

Pontos fortes

• Soube implementar o toString para possibilitar a coerção em User.js
  

  jsexpert-challenge04-datatypes/src/entity/User.js

  Lines 6 to 8 in 01ede15

  	toString() {
  	return `[id=${this.id}]`
  	}

• Soube criar um Symbol para a propriedade privada kUsers
  

  jsexpert-challenge04-datatypes/src/entity/Users.js

  Line 4 in 01ede15

  const kUsers = Symbol('kUsers');

• Soube escolher corretamente a estrutura esperada para essa parte do desafio: o Set
  

  jsexpert-challenge04-datatypes/src/entity/Users.js

  Line 9 in 01ede15

  this[kUsers] = new Set();

• Soube adicionar valores na estrutura escolhida
  

  jsexpert-challenge04-datatypes/src/entity/Users.js

  Line 15 in 01ede15

  this[kUsers].add(user)

• Soube verificar se há informações na estrutura escolhida
  

  jsexpert-challenge04-datatypes/src/entity/Users.js

  Line 20 in 01ede15

  return this[kUsers].size;

• Usou corretamente o Symbol.iterator para tornar o objeto gerado iterável
  

  jsexpert-challenge04-datatypes/src/entity/Users.js

  Lines 24 to 29 in 01ede15

  	*[Symbol.iterator]() {
  	const users = Array.from(this[kUsers])
  	for (const user of users) {
  	yield user
  	}
  	}

Possíveis pontos de melhoria

• Aqui em vez de usar o toString(), também poderia ter sido usado o Symbol.toPrimitive, já gerenciando qualquer tipo de conversão :)
  

  jsexpert-challenge04-datatypes/src/entity/User.js

  Lines 6 to 8 in 01ede15

  	toString() {
  	return `[id=${this.id}]`
  	}

• Aqui tem linhas com e sem ponto e vírgula no final. Usar alguma extenção na IDE (tipo o Prettier no VSCode) pra fazer o linting e padronizar o projeto pode ser uma boa! - em desafios de código em processos seletivos essa acaba sendo uma das coisas que a galera mais repara
  

  jsexpert-challenge04-datatypes/src/entity/Users.js

  Line 15 in 01ede15

  this[kUsers].add(user)

src/service

Pontos fortes

• Soube implementar o generator que chama a repository fazendo a paginação
  

  jsexpert-challenge04-datatypes/src/service/CryptoService.js

  Lines 9 to 23 in 01ede15

  	async *list() {
  	// TODO: implementar generator que chama a repository fazendo a paginação
  	let page = 1
  	let hasData = true
  	while (hasData) {
  	const resultObject = await this.repository.list(page)
  	if (resultObject.data.length > 0 ) {
  	hasData = true
  	const cryptos = resultObject.data
  	const cryptosObjects = cryptos.map(c => new Crypto({...c}))
  	page += 1
  	yield cryptosObjects
  	}
  	}
  	}

Possíveis pontos de melhoria

• Aqui vale lembrar que o spread faz um loop internamente, então usar spread dentro de um map é um loop dentro de um loop (complexidade quadrática) - Como nesse caso não precisaria quebrar a referência e criar um novo objeto, é bom evitar :)
  

  jsexpert-challenge04-datatypes/src/service/CryptoService.js

  Line 18 in 01ede15

  const cryptosObjects = cryptos.map(c => new Crypto({...c}))

• Forma bem bacana de fazer a verificação se tem dados ou não antes de continuar o loop no generator! Mas como o hasData nunca é setado pra false, ainda continua um loop infinito 😆 - o que nesse caso não tem problema pq a nossa repository já gerencia isso e não vai quebrar, mas invalida a implementação do hasData já que nesse caso ele tá fazendo o papel de um while(true) -
  

  jsexpert-challenge04-datatypes/src/service/CryptoService.js

  Lines 12 to 13 in 01ede15

  	let hasData = true
  	while (hasData) {

src/util

Pontos fortes

• Soube criar um Symbol para a propriedade privada kData
  

  jsexpert-challenge04-datatypes/src/service/CryptoService.js

  Lines 12 to 13 in 01ede15

  	let hasData = true
  	while (hasData) {

• Escolheu a estrutura correta a usar: Map
  

  jsexpert-challenge04-datatypes/src/util/CustomTerminal.js

  Line 19 in 01ede15

  this[kData] = new Map();

• Soube usar um array gerado a partir dos valores de um Map
  

  jsexpert-challenge04-datatypes/src/util/CustomTerminal.js

  Line 33 in 01ede15

  const data = [...this[kData].values()]

• Soube verificar se há informações no Map
  

  jsexpert-challenge04-datatypes/src/util/CustomTerminal.js

  Line 40 in 01ede15

  return this[kData].size;

• Soube adicionar valores no Map levando a chave em consideração
  

  jsexpert-challenge04-datatypes/src/util/CustomTerminal.js

  Line 48 in 01ede15

  data.map(crypto => this[kData].set(crypto.id, crypto ) )

• Soube pegar valores de um Map
  

  jsexpert-challenge04-datatypes/src/util/CustomTerminal.js

  Line 53 in 01ede15

  return this[kData].get(id)

• Soube remover valores de um Map
  

  jsexpert-challenge04-datatypes/src/util/CustomTerminal.js

  Line 58 in 01ede15

  return this[kData].delete(id);

Considerações finais

Como sempre, mandou muito bem! Conseguiu aprender e aplicar corretamente as estruturas vistas no curso no melhor cenário possível pra elas dentro do desafio, e ainda por cima a implementação ficou praticamente idêntica à solução oficial! Como sempre, mandou muito bem, e espero poder ver mais códigos seus aqui pelos desafios. Parabéns, Claudio! 🚀