如何将方法分离到路由

Question

我有这条路线，它工作得很好

router.get('/api/User/:id',async(req,res)=>{
try {
const{id}=req.params;
let result =await pool1.request()
.input('Iduser', sql.Int, id)
.execute('GetUser')
res.json(result);
} 
catch (err) {
res.json({ error: 'Does Not exist' })
}
});

但是我想分离函数并尽可能保持路线干净，尝试按如下方式分离它，但我收到以下错误：类型错误：一个不是函数

路线

router.get('/api/User/:id', async(req,res)=>{
try {
res.json((await one(req.params.id))[0]);
} catch (err) {
console.log(err);
res.sendStatus(500);
}
})

功能

const one = async(id)=>{
return new Promise((resolve,reject)=>{
pool.request()
.input('Iduser', sql.Int, id)
.execute('User')((err,results) =>{
if(err){
return reject(err);
}
resolve(results);
});
});
}

我的错误是什么，我调用函数是错误的吗？

Answer 1

为了使您的代码更简洁，您可以这样做：

const getUserById =async(req,res)=>{
try {
const{id}=req.params;
let result =await pool1.request()
.input('Iduser', sql.Int, id)
.execute('GetUser')
res.json(result);
} 
catch (err) {
res.json({ error: 'Does Not exist' })
}
}
router.get('/api/User/:id',getUserById);

也为了让它更干净，你可以这样做

export const asyncHandler = (fn) => async (request, response, next) => {
try {
return await fn(request, response, next);
} catch (error) {
return next(error); // or response.json({ error: 'Does Not exist' })
}
};
const getUserById =async(req,res)=>{
const { params: { id } }=req;
const result =await pool1.request()
.input('Iduser', sql.Int, id).execute('GetUser');
return  res.json(result);
}

router.get('/api/User/:id',asyncHandler(getUserById));

Answer 2

谢谢伙计，我解决了为什么他告诉我这不是一个函数，我叫得不好，但你解释它的方式路线要干净得多

功能

const one = async(id)=>{
return new Promise((resolve,reject)=>{
pool.request()
.input('IdUser', sql.Int, id)
.execute('GetUser',(err,results)=>{
if(err){
return reject(err);
}
resolve(results);
}
)    
});
}

路线

router.get('api/user/:id', async(req,res)=>{
try {
let result=await m.one(req.params.id);
res.json(result);
} catch (error) {
console.log(error);
res.sendStatus(500);
}
})