Comment passer une liste de classe en tant que paramètre?

Question

J'essayais d'avoir une méthode handleException, qui peut prendre un objet d'exception et une liste de classes d'exceptions acceptables pour vérifier si l'exception est acceptable et peut être réessayée.

void handleException(Exception e, String... acceptableExceptionNames) 
     throws MyException { 

    boolean isRetryable = false; 

    for(String acceptableExceptionName: acceptableExceptionNames) { 
     try { 
     if (Class.forName(acceptableExceptionName).isInstance(e)) { 
      isRetryable = true; 
      break; 
     } 
     } catch (ClassNotFoundException e1) { 
     continue; 
     } 
    } 

    if (isRetryable) { 
     // log retryable 
    } else { 
     // log error 
    } 

    throw new MyException(isRetryable, "Failed"); 
    } 

Le paramètre je passe un String... classNames au lieu de Class<? extends Exception> classes, parce que si je fais quelque chose comme ceci:

void handleException(
    Exception e, 
    Class<? extends Exception>... acceptableExceptions) 
     throws MyException { 
    for (Class acceptableException : acceptableExceptions) { 
     if (e instanceOf acceptableException) {} 
    } 
} 

L'IDE se plaindront unknown class acceptableException

Quelqu'un sait s'il y a un moyen pour passer Class<?>? Ou une meilleure façon d'éviter d'utiliser String classNames et Class.forName()?

Answer 1

acceptableExceptions est pas un Class, c'est un Class[]. Vous pouvez garder votre design original, cependant, et il suffit d'utiliser la place directement les objets Class de les créer à partir de chaînes:

void handleException 
     (Exception e, Class<? extends Exception>... acceptableExceptionNames) 
     throws MyException { 

    boolean isRetryable = false; 

    for(Class<?> acceptableExceptionName: acceptableExceptionNames) { 
     if (acceptableExceptionName.isInstance(e)) { 
      isRetryable = true; 
      break; 
     } 
    } 

    if (isRetryable) { 
     // log retryable 
    } else { 
     // log error 
    } 

    throw new MyException(isRetryable, "Failed"); 
} 

EDIT:
Comme une note de côté, ce code peut être fait beaucoup plus court en utilisant les flux de Java 8 :

void handleException 
     (Exception e, Class<? extends Exception>... acceptableExceptionNames) 
     throws MyException { 

    boolean isRetryable = 
     Arrays.stream(acceptableExceptionNames).anyMatch(x -> x.isInstance(e)); 

    if (isRetryable) { 
     // log retryable 
    } else { 
     // log error 
    } 

    throw new MyException(isRetryable, "Failed"); 
} 

Answer 2

Utiliser le contrôle suivant

for (Class<? extends Exception> exceptionClass : acceptableExceptions) { 
    if (exceptionClass.isInstance(e)) { 
     // it is your exception 
    } 
} 

Answer 3

à mon avis, il est plus facile de le faire si en effectuant les simples comparaisons de chaînes de caractères comme ceci:

private void handleException(Exception ex, String... acceptableException) { 
    for (int x = 0; x < acceptableException.length; x++) { 
     String[] exceptionClass = ex.getClass().toString().split("."); 
     if (!acceptableException[x] 
       .equals(exceptionClass[exceptionClass.length - 1])) { 
      /* Exception Not Acceptable */ 
     } 
    } 

    /* Exception Acceptable */ 
} 

Answer 4

Vous ne voulez pas vérifier si e est une instance d'un Class[], qui est ce que acceptableExceptions est, mais s'il s'agit d'une instance de l'une des les classes référencées par le tableau acceptableExceptions.

Pour ce faire, vous devez les itérer, et vous devez utiliser la méthode de réflexion Class.isInstance(Object obj). Comme le dit javadoc:

Cette méthode est l'équivalent dynamique de l'opérateur Java instanceof.

Pour éviter les avertissements du compilateur, vous devez également ajouter @SafeVarargs vous méthode si elle est static ou final. Sinon, vous devez ajouter @SuppressWarnings("unchecked") à la fois à la méthode et à l'appelant (s).

@SuppressWarnings("unchecked") 
void handleException(Exception e, Class<? extends Exception>... acceptableExceptions) throws MyException { 
    boolean acceptable = false; 
    for (Class<? extends Exception> acceptableException : acceptableExceptions) 
     if (acceptableException.isInstance(e)) { 
      acceptable = true; 
      break; 
     } 
    if (acceptable) { 
     // code here 
    } 
}