我想编写更少的代码,因此我将创建和更新代码组合到一个方法中,看起来它不起作用。
@Post('user')
@Put('user')
async createOrUpdateUser(@Res() res, @Req() req) {
if (req.method == 'POST') {
//do user creating work
} else {
//do user updating work
}
}
我试过了,但只有@Post
装饰工作在这里。
它似乎不是那样工作,但你可以这样做,以便你可以避免重复。
@Post('user')
async createUser(@Res() res, @Req() req) {
if(req.method=='POST')
{
//do user creating work
}else {
//do user updating work
}
}
@Put('user')
async updateUser(@Res() res, @Req() req) {
this.createUser(res, req);
}
您可以使用@All
装饰器在一个方法中处理所有请求方法:
@All('user')
async createOrUpdateUser(@Req() req) {
switch (req.method) {
case 'POST':
return 'new user';
case 'PUT':
return 'updated user';
default:
return throw new NotFoundException();
}
}
如果你在@Get('user')
处理程序之前定义一个@All('user')
处理程序,它将处理get请求。 (我建议不要这样做,见下文。)
1)通过使用@Res
注入框架特定的响应对象,您将失去使嵌套如此强大的大多数功能,如拦截器或序列化。只有这样,如果你真的需要。
2)我总是喜欢将常用逻辑提取到方法或类中,而不是在方法中创建条件分支。在我看来,这是一种更具可读性和可维护性的方法。它还使其他嵌套集成更容易,例如使用swagger轻松记录您的API。
@Put('user')
async update(@Body() body) {
const user = await this.service.prepare(body);
return this.service.updateUser(user);
}
@Post('user')
async update(@Body() body) {
const user = await this.service.prepare(body);
return this.service.createUser(user);
}
在这个例子中,公共部分被提取到UserService#prepare
。