alterando o try catch do getForecastPointsFromApi #78
Conversation
|
Oi @leoander01 desculpa a demora! Vou revisar logo 🚀 |
| /** | ||
| * This is handling the Axios errors specifically | ||
| */ | ||
| if (HTTPUtil.Request.isRequestError(err)) { | ||
| const axiosError = err as AxiosError; |
There was a problem hiding this comment.
Cara tua lógica está certa, mas como sei que tu também esta buscando entender a forma que eu faria eu abri outro pull request mostrando como eu fiz #79.
A principal diferença é que aqui no teu tu ta importando o AxiosError na classe StormGlass e isso faz com que o Axios seja exposto para a aplicação, no curso a gente encapsula toda a lógica de requisição dentro da classe Request, dessa maneira se mudarmos do Axios para outra biblioteca não precisaremos alterar o resto da aplicação.
Quanto menos o resto da aplicação souber das bibliotecas externas melhor :D
There was a problem hiding this comment.
Ah beleza, consegui entender melhor agora o porquê de utilizar a encapsulação e também a reutilização de código caso seja necessário alterar o Axios para outra lib.
Tranquilo então @waldemarnt, obrigado pela explicação ;D
There was a problem hiding this comment.
@leoander01 Valeu pela contribuição! Ja atualizei todo o código e todos os passos 💯
|
@leoander01 valeu pela contribuição! Da uma olhada no meu comentário e me diz o que tu acha |
|
Fechando esse pull request porque a mudança ja foi feita no código |
Alterei o try catch tipando a mensagem de erro, não sei se é uma boa prática.
Pois, após a versão 4.0 do TS ele espera que o objeto de erro dentro catch seja em 'unknown' ao invés de 'any'.