我应该使用委托来声明匿名异步方法吗?

时间:2015-11-30 14:48:57

标签: c# asynchronous delegates

我正在寻找有关在delegateasync中使用await的想法。它好还是不好?我在Google上搜索过,但没有一个与我的相似。

我正在定义一种使用SignalR更改消息状态(isRead = true)的方法:

enum MessageStatus
{
  Failure,
  Success
}

delegate Task<MessageStatus> MsgDelegate(string id);

public async Task ChangeMessageStatus(string id)
{
   string error = string.Empty;

   MsgDelegate msg = async (x) =>
   {
      try
      {
         using (var db = new VinaChanelDbContext())
         {
            var message = db.Messages.SingleOrDefault(m => m.Id == x);
            if (message != null)
            {
               message.IsRead = true;
            }
            return await db.SaveChangesAsync() > 0 ? 
                MessageStatus.Success : MessageStatus.Failure;
         }
      }
      catch (Exception e)
      {
         error = e.Message;
         return MessageStatus.Failure;
      }
   };
   switch (await msg(id))
   {
      case MessageStatus.Success:
         Clients.Caller.updateStatus(true);
         break;
      case MessageStatus.Failure:
         Clients.Caller.errorMessage(error);
         Clients.Caller.updateStatus(false);
      break;
   }
}

我的代码怪异吗?我应该用吗?

1 个答案:

答案 0 :(得分:0)

为什么要使用代表?出了什么问题:

enum MessageStatus
{
    Failure,
    Success
}

public async Task ChangeMessageStatus(string id)
{
    string error = string.Empty;
    var status = MessageStatus.Failure;

    try
    {
        using (var db = new VinaChanelDbContext())
        {
            var message = db.Messages.SingleOrDefault(m => m.Id == id);
            if (message != null)
            {
               message.IsRead = true;
               if (await db.SaveChangesAsync() > 0)
               {
                 status = MessageStatus.Success;
               }
            }
        }
    }
    catch (Exception e)
    {
        error = e.Message;
    }

    switch (status)
    {
        case MessageStatus.Success:
            Clients.Caller.updateStatus(true);
            break;
        case MessageStatus.Failure:
            Clients.Caller.errorMessage(error);
            Clients.Caller.updateStatus(false);
            break;
    }
}

您可能希望将该顶部部分分解为私有方法,但我不确定您为什么要将其作为委托...