En utilisant PostSharp pour réessayer la méthode sur Exception

Question

Pour un de mes modules DAL J'ai beaucoup de plomberie dupliquée en forme de:

while (retry) 
{ 
... 
try 
{ 
    ...do something 
    retry = false; 
} 
catch (SqlException sqlEx) 
{ 
    // Retry only if -2 = Connection Time Out or 1205 = Deadlock 
    if (sqlEx.Number == -2 || sqlEx.Number == 1205) 
    { 
     ..retry if attempt < max 
    } 
     ..log and rethrow exception 
} 
} 

et avoir découvert PostSharp récemment, je suis tenté de remplacer ces codes de plomberie avec un attribut.

Mon plan initial était de: - étendre OnMethodInvocationAspect et rappelez-vous l'événement d'appel de méthode args lors de l'appel de la méthode - mettre en œuvre IOnExceptionAspect et mettre en œuvre OnException pour vérifier le type d'exception et si nouvelle tentative est nécessaire d'utiliser objet l'événement d'appel de méthode args de l'original appel, à savoir:

[Serializable] 
public sealed class RetryAttribute : OnMethodInvocationAspect, IOnExceptionAspect 
{ 
    [NonSerialized] 
    private MethodInvocationEventArgs m_initialInvocationEventArgs = null; 

    public override void OnInvocation(MethodInvocationEventArgs eventArgs) 
    { 
     if (m_initialInvocationEventArgs == null) 
      m_initialInvocationEventArgs = eventArgs; 

     base.OnInvocation(eventArgs); 
    } 

    public void OnException(MethodExecutionEventArgs eventArgs) 
    { 
     // check if retry is necessary 
     m_initialInvocationEventArgs.Proceed(); 
    } 
} 

mais la méthode de OnInvocation ne se déclenche plus une fois que je l'ai ajouté IOnExceptionAspect ..

est-ce que quelqu'un sait ce que je dois faire ici? Ou peut-être qu'il y a un aspect plus approprié que je devrais utiliser?

Merci,

Answer 1

Vous ne pouvez pas avoir un aspect qui mettent en œuvre deux interfaces (aspect IOnMethodInvocation et IOnExceptionAspect dans votre cas). Le tisserand prendra une interface arbitraire et implémentera l'aspect.

Je pense que tout ce dont vous avez besoin pour atteindre votre objectif est OnMethodInvocationAspect. Pourquoi ne mettez-vous pas la boucle for et le try-catch dans le gestionnaire OnInvocation?

-gael

Answer 2

est ici une solution assez simple qui ne nécessite pas PostSharp. Créez la méthode d'utilité suivante.

public static void Try(Func<bool> task, int retryCount) 
{ 
    int attemptNum = 1; 
    while (attemptNum++ <= retryCount && task()) ; 
} 

Ensuite, créez la tâche à réessayer. La valeur de retour doit indiquer si une nouvelle tentative doit être tentée.

public bool UnreliableTask() 
{ 
    try 
    { 
     // Do something 
    } 
    catch (SqlException ex) 
    { 
     return (ex.Number == -2 || ex.Number == 1205); 
    } 

    return false; 
} 

Ensuite, il suffit d'appeler la tâche comme si une nouvelle tentative de cinq fois:

Try(UnreliableTask, 5); 

Answer 3

Une question assez vieux, mais je voudrais partager une approche qui pourrait aider. Nous utilisons également ce paradigme avec succès. Jetez un oeil sur le code ci-dessous. Ce que vous faites est en fait hériter de MethodLevelAspect et d'utiliser "Conseils".

OnException n'aide pas args.Proceed() à lancer une erreur. Donc, nous avons utilisé un bloc try-catch supplémentaire directement dans OnInvoke.

[Serializable] 
public class MyAspectAttribute : MethodLevelAspect 
{ 
    object exceptionReturn = null; 

    public MyAspectAttribute(object ExceptionReturn) : base() 
    { 
    } 

    [OnMethodInvokeAdvice, SelfPointcut] 
    [AdviceDependency(AspectDependencyAction.Order, AspectDependencyPosition.Before, "OnEntry")] 
    public void OnInvoke(MethodInterceptionArgs args) 
    { 
     try 
     { 
      args.Proceed(); 
     } 
     catch (Exception exc) 
     { 
      // do logging here 
      args.ReturnValue = exceptionReturn; 
     } 
    } 

    [OnMethodExceptionAdvice, SelfPointcut] 
    public void OnException(MethodExecutionArgs args) 
    { 
    } 

    [OnMethodEntryAdvice, SelfPointcut] 
    public void OnEntry(MethodExecutionArgs args) 
    { 
    } 

    [OnMethodExitAdvice, SelfPointcut] 
    [AdviceDependency(AspectDependencyAction.Order, AspectDependencyPosition.After, "OnInvoke")] 
    [AdviceDependency(AspectDependencyAction.Order, AspectDependencyPosition.After, "OnEntry")] 
    public void OnExit(MethodExecutionArgs args) 
    { 
     // your exit statements, such as committing transaction etc. 
    } 
}