2011-12-27 13 views
18

mam refaktoryzacji kodu użyć guava Cache.Guava cache i konserwowanie sprawdzane wyjątki

kod początkowa:

public Post getPost(Integer key) throws SQLException, IOException { 
    return PostsDB.findPostByID(key); 
} 

Aby nie złamać coś, czego potrzebujesz, aby zachować jakąkolwiek rzucony wyjątek jak jest bez zawijania.

Aktualny rozwiązanie wydaje się nieco brzydki:

public Post getPost(final Integer key) throws SQLException, IOException { 
    try { 
     return cache.get(key, new Callable<Post>() { 
      @Override 
      public Post call() throws Exception { 
       return PostsDB.findPostByID(key); 
      } 
     }); 
    } catch (ExecutionException e) { 
     Throwable cause = e.getCause(); 
     if (cause instanceof SQLException) { 
      throw (SQLException) cause; 
     } else if (cause instanceof IOException) { 
      throw (IOException) cause; 
     } else if (cause instanceof RuntimeException) { 
      throw (RuntimeException) cause; 
     } else if (cause instanceof Error) { 
      throw (Error) cause; 
     } else { 
      throw new IllegalStateException(e); 
     } 
    } 
} 

Czy istnieje jakikolwiek sposób, aby to ładniejszy?

Odpowiedz

31

Tuż po napisaniu pytanie zacząłem myśleć o sposobie użytkowego zasilanego z rodzajowych. Następnie przypomniałem sobie coś o Throwables. I tak, już tam jest!)

Może być również konieczne do obsługi UncheckedExecutionException or even ExecutionError.

Więc rozwiązanie jest:

public Post getPost(final Integer key) throws SQLException, IOException { 
    try { 
     return cache.get(key, new Callable<Post>() { 
      @Override 
      public Post call() throws Exception { 
       return PostsDB.findPostByID(key); 
      } 
     }); 
    } catch (ExecutionException e) { 
     Throwables.propagateIfPossible(
      e.getCause(), SQLException.class, IOException.class); 
     throw new IllegalStateException(e); 
    } catch (UncheckedExecutionException e) { 
     Throwables.throwIfUnchecked(e.getCause()); 
     throw new IllegalStateException(e); 
    } 
} 

Very nice!

Zobacz także ThrowablesExplained.

+0

wahał jeśli self-odpowiedział na pytanie powinna być wysłana w ogóle. Ale to jasno: http://meta.stackexchange.com/questions/2706/posting-and-answering-questions-you-have-already-found-the-answer-to – Vadzim

+1

I dziękuję, guawa chłopaki! – Vadzim

+0

Oznacz to więc jako ** poprawna odpowiedź;) – Xaerxess