Как избежать дублирования кода внутри двух методов?

Question

У меня есть два идентичных методов, но один из них оператор возврата внутри try catch

public void A(Guid agentId) 
{ 
    var agent = _agentsProvider.GetAgentById(agentId); 
    var updateCompletionSource = C(agentId); 
    try 
    { 
     var cacheEntry = UpdateAgentMetadataCacheEntry(agent, true, false); 
     updateCompletionSource.SetResult(cacheEntry); 
    } 
    catch (Exception e) 
    { 
     updateCompletionSource.SetException(e); 
    } 
} 

private Entry B(IAgent agent) 
{ 
    var updateCompletionSource = C(agent.Id); 
    try 
    { 
      var cacheEntry = UpdateAgentMetadataCacheEntry(agent, false, false); 
      updateCompletionSource.SetResult(cacheEntry); 
      return cacheEntry; 
     } 
     catch (Exception e) 
     { 
      updateCompletionSource.SetException(e); 
      return GetPreviousCacheEntry(); 
     } 
} 

Как собрать идентичную часть и создать новый метод с этой частью?

Answer 1

Если GetPreviousCacheEntry может иметь проблемные побочные эффекты, мне кажется, что вам не нужен метод A.

Просто вызовите метод B и проигнорируйте возвращаемое значение, если оно вас не интересует.

Как было отмечено в комментариях, методы не идентичны, кроме заявлений возвратных, хотя - потому что они используют другой второй аргумент для UpdateAgentMetadataCacheEntry, и они имеют разные параметры тоже (один имеет Guid и один имеет Agent). Вы можете реорганизовать это в:

private Entry B(IAgent agent, bool foo) 
{ 
    var updateCompletionSource = C(agent.Id); 
    try 
    { 
      var cacheEntry = UpdateAgentMetadataCacheEntry(agent, foo, false); 
      updateCompletionSource.SetResult(cacheEntry); 
      return cacheEntry; 
     } 
     catch (Exception e) 
     { 
      updateCompletionSource.SetException(e); 
      return GetPreviousCacheEntry(); 
     } 
} 

... с многозначительным названием для foo, очевидно. Я предполагаю, что разница в типе параметра в действительности не является проблемой.

Answer 2

Как сказал Джон, вам не нужен метод A. Просто добавьте еще один параметр для логического значения.

public void A(Guid agentId) 
{ 
    var agent = _agentsProvider.GetAgentById(agentId); 
    AnotherA(agent, true); 
} 

private Entry B(IAgent agent) 
{ 
    return AnotherA(agent, false); 
} 

private Entry AnotherA(IAgent agent, bool a) 
{ 
    try 
    { 
     var cacheEntry = UpdateAgentMetadataCacheEntry(agent, a, false); 
     updateCompletionSource.SetResult(cacheEntry); 
     return cacheEntry; 
    } 
    catch (Exception e) 
    { 
     updateCompletionSource.SetException(e); 
     return GetPreviousCacheEntry(); 
    } 
}