Export const downloadXml = async (req:Request,res:Response):Promise<Response<string,Record<string, string>>> => {

Question

Export const downloadXml = async (req:Request,res:Response):Promise<Response<string,Record<string, string>>> => {

try {
await axios.get(LINK).then(async({data}):Promise<void | object>=>{
await writeFile("./products.xml",data,(error) => {
if(error){
console.log(error, " err")
return res.status(500).json({message:CANT_DOWNLOAD_FILE})
}
});
});
return res.status(200).json({message:"File Downloaded Complete"})
} catch(err){
return res.status(500).json({message:CANT_DOWNLOAD_FILE})
}
}

народ всем привет. подскажите есть что улучшить в этой функции или это нормально ?

#backend #javascript #node.js #programming #russian

0

25.03.2022

4 ответов

24 просмотра

🌏 ࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣩࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧࣧ ࣩࣩࣩࣩࣩࣩࣩࣩࣧࣧࣧࣧࣧࣧࣧ

Lёsha
множественные вложенные типы нечитаемы, лучше обья...

Жс хуйня и это факт, я перехожу на паскаль

0

25.03.2022

M F Автор вопроса

Lёsha
множественные вложенные типы нечитаемы, лучше обья...

object это ошибка. я не нахожу как типизировать ошибки

0

25.03.2022

Lёsha

M F
object это ошибка. я не нахожу как типизировать ош...

обработать ошибку и свести ее к какому то обьекту

0

25.03.2022

Lёsha · Accepted Answer

множественные вложенные типы нечитаемы, лучше обьявить в каком то другом месте и обьявить здесь одним словом. Как по мне void | object вообще лишает смысла использование тапйскрипта. Уж либо возвращает либо нет. Ну и тип object прописывать это все равно что any, лишено смысла.

170 похожих чатов

Export const downloadXml = async (req:Request,res:Response):Promise<Response<string,Record<string, string>>> => {

4 ответов

Похожие вопросы