我在WebAPI2控制器中有一个post方法。我只想确保我所做的一切都是正确的。我希望创建一个新任务,异步运行该方法并返回一条消息。
public async Task<IHttpActionResult> Post([FromBody]Menu m)
{
using (MySqlConnection con = new MySqlConnection(""))
using (MySqlCommand cmd = new MySqlCommand("Insert into Menu (Description,LanguageId,IsActive) values (@Description,@LanguageId,@IsActive) ", con))
{
try
{
if (con.State == ConnectionState.Closed)
{
await con.OpenAsync();
cmd.Parameters.AddWithValue("@Description", m.Description);
cmd.Parameters.AddWithValue("@LanguageId", m.LanguageId);
cmd.Parameters.AddWithValue("@IsActive", m.IsActive);
await cmd.ExecuteNonQueryAsync();
}
}
catch (MySqlException ex)
{
return Content(HttpStatusCode.NotFound,ex);
}
finally
{
await con.CloseAsync();
}
return Ok("Inserted Succesfully");
}
}
我应该创建一个新的异步方法来执行查询,然后将消息返回到控制器的post方法中,还是这是正确的方法?
发布于 2017-09-08 08:13:41
你需要在这里处理单一责任原则。Post方法应该只处理HTTP请求,然后将其传递给另一个处理程序进行进一步操作,并在收到结果后,返回给用户。
您可以在代码中遵循SRP和DRY原则。
https://codereview.stackexchange.com/questions/174669
复制相似问题