search

roger-melo-treinamentos / curso-de-js-roger-melo

Repositório de informações do CJRM
491 stars 170 forks source link

Aplicação: [Weather-App] #3970

Closed MivlaM closed 2 years ago

MivlaM commented 2 years ago

As cores da sua versão da aplicação são diferentes das cores da aplicação mostrada na aula?

Sim

A aplicação contém funcionalidades que não foram mostradas nas aulas?

Não

A aplicação contém funcionalidades da linguagem que não foram mostradas nas aulas?

Não

Link do repositório da aplicação (ou pasta pública no Google Drive)

https://github.com/MivlaM/Meu-WeatherApp

Maiores dificuldades durante a implementação

-> Enquanto eu testava a aplicação, tive um problema com a API, acho que cheguei no limite de usos que era permitido diariamente, mas achei a solução por meio de umas issues passadas aqui no repositório.

-> Consegui fazer umas refatorações nas funções, mas acredito que ainda teria espaço para fazer mais (com o if da const IsDayTime, por exemplo).

Menores dificuldades durante a implementação

-> Nomenclatura das variáveis

-> Manipulação do DOM

Roger-Melo commented 2 years ago

Olá @MivlaM.

Passando para dizer que visualizei a issue e em até 10 dias úteis a análise será feita =)

Roger-Melo commented 2 years ago

Olá @MivlaM!

Você pode baixar a análise de sua Weather Application neste link.

Se houver qualquer dúvida, é só dar o toque =)

MivlaM commented 2 years ago

Acabei de assistir a análise aqui, durante o vídeo eu ia pausando para anotar num papel as melhorias que eu posso implementar mais tarde no código.

Aquela const Chave não tem uso nenhum na aplicação mesmo, eu tinha deixado ela lá para guardar a Url do site da API porque toda hora ele me deslogava por inatividade, aí aquele link me voltava direto para a parte que eu estava lendo na documentação. Enfim, eu devia ter removido essa const quando a aplicação estava pronta, mas por a const não influenciar em nada, passou-me despercebido.

Outra coisa, eu estava tão focado em refatorar por meio de funções de responsabilidade única que me esqueci, em determinado momento, que eu podia ter usado os ternários para refatorar o if/else da const IsDayTime. Erro meu, na próxima vou me certificar de analisar mais possibilidades.

No mais, acho que é isso mesmo, excelente análise e valoráveis agradecimentos por ter analisado a minha aplicação =)

Roger-Melo commented 2 years ago

No que precisar, é só abrir uma nova issue =)