Solucionado (ver solução)
Solucionado
(ver solução)
2
respostas

Simplificando ainda mais o Código

Pra não ter que declarar 3 métodos com o código praticamente idêntico, eu preferi passar a url por parâmetro em um método único na classe NegociacaoService. Acredito que código ficou muito menor e até mais fácil de ler. Segue meu repositório no github: https://github.com/PereiraFelipeSilva/alura-ES6/blob/master/ES6/client/js/app/services/NegociacaoService.js A classe NegociacaoController ficou assim: https://github.com/PereiraFelipeSilva/alura-ES6/blob/master/ES6/client/js/app/controllers/NegociacaoController.js Espero ter ajudado!

2 respostas
solução!

Boa tarde, Felipe! Como vai?

Obrigado pela sua colaboração e por compartilhar com a comunidade da Alura a sua solução! É com participações como essa sua que fazemos da Alura uma plataforma cada vez melhor! Continue assim!

Comentando diretamente sobre seu código, ficou bem bacana! Contudo, veja que agora vc precisa passar uma string para o método obterNegociacoes(), como em service.obterNegociacoes('semana'). Isso não é o ideal, pois se um dia o valor dessa string mudar, vc terá que passar por todos os pontos do código onde ela era utilizada e fazer a alteração que for necessária!

Para solucionar isso vc poderia criar uma classe que encapsulasse os valores das strings em forma de constantes, algo como um ENUM.

Pegou a ideia? Qualquer coisa é só falar!

Grande abraço e bons estudos, meu aluno!

Tem razão. Obrigado pelo toque!