Должен ли я использовать делегат для объявления анонимного асинхронного метода?

Question

Я ищу идеи об использовании delegate с async и await. Хорошо это или нет? Я искал в Google, но ни один из них не похож на мой.

я определяю способ, чтобы изменить статус сообщения (isRead = true) с SignalR:

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; 
    } 
} 

мой код странно? Должен ли я использовать его?

Answer 1

Зачем использовать делегат? Что случилось с:

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; 
    } 
} 

Вы могли бы хотеть, чтобы сломать эту верхнюю часть из в частный метод, но я не уверен, почему вы хотите, чтобы сделать его делегатом ...