Ярлык для использования Guava с использованием исключений?

Question

Я продолжаю писать специальные исключения в случае, если опция отсутствует.

Например:

Optional<?> optional = ...; 
if (!optional.isPresent()) { 
    throw new MyException(); 
} 
Object result = optional.get(); 

Я нахожу этот код не очень свободно, особенно использование челки (!). Я предпочел бы, как писать что-то вроде:

Optional<?> optional = ...; 
Object result = optional.orThrow(MyException.class); 

Есть ли такой ярлык в гуавы, что я еще не нашел?

Answer 1

Говоря как разработчик Guava, позвольте мне попытаться распаковать здесь логику. Отвечая как на исходный вопрос, так и на комментарий, прямо на вопрос:

Это абсолютно тот случай, когда мы пытаемся заставить пользователей Guava уважать наши стандарты хороших привычек программирования. (Наши стандарты: сильно под влиянием, например, эффективной Java.)

Это говорит о том, что в этом конкретном вопросе есть совершенно хорошие варианты использования: «если отсутствует, выведите исключение «. Возможно, вы реализуете класс, к которому можно получить доступ в обоих направлениях - один метод с необязательным возвращаемым значением и один метод, который предполагает, что значение всегда будет присутствовать, иначе исключая исключение. Интерфейс Deque, например, предоставляет специальные и исключающие исключения версии peek, poll и offer.

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

if (value.isPresent()) { 
    return value.get(); 
} else { 
    throw new MyException(); 
} 

Метод «orThrow» вы предлагаете требует отражения (!!), не делает позволяют настраивать исключение с полезным сообщением и т. д. «Обычный способ» отлично читается и более эффективен.

Иногда Guava не обеспечивает явной поддержки для вещей, потому что для этих случаев использования мы считаем, что лучше всего делать только «обычный путь». Я думаю, что это так.

Answer 2

Я не думаю, что это будет принадлежать библиотеке. Я нахожу очень редким найти библиотеку, которая получает экземпляр исключения, которое нужно бросить, если что-то не так, как ожидалось, особенно потому, что во многих случаях в исключении должно быть сообщение о том, что пошло не так.

При этом вы можете создать свой собственный дополнительный класс, который сделает все, что вам нужно. Или вы можете создать свой собственный класс OptionalHelper, где у вас есть метод, который делает то, что вы хотите:

public class OptionalHelper { 
    public <T> T valueOrThrow(final Optional<T> optional) throws MyException { 
     if (optional.isPresent()) { 
      return optional.get(); 
     } else { 
      throw new MyException(); 
     } 
    } 
} 

EDIT:

Предположив у вас есть пользовательский класс, который принимает имя параметра/поля, которое нужно проверить , вы могли бы иметь лучший подход, аналогичный тому, что делает Предпосылки:

public class OptionalHelper { 
    public <T> T valueOrFail(final Optional<T> optional, final String fieldName) throws OptionalNotPresentError { 
     if (optional.isPresent()) { 
      return optional.get(); 
     } else { 
      throw new OptionalNotPresentError(fieldName); 
     } 
    } 
} 

Answer 3

Вот еще один способ сделать это без дополнений гуавы:

Object result = optional.or(new Supplier() { 
    public Object get() { 
     throw new MyException(); 
    } 
}); 

MyException должно быть снято, но это позволяет передавать аргументы его конструктору. И, конечно, если вы делаете это много, вы можете хранить Поставщика где-нибудь и использовать его в каждом месте, в котором оно вам нужно.

Object result = optional.or(SomethingIsMissing.INSTANCE); 

Answer 4

Это работает для меня (без отражения, просто введите умозаключение):

public class ExceptionSupplier<T, E extends RuntimeException> implements Supplier<T> { 

    private final E exception; 

    private ExceptionSupplier(E exception) { 
     this.exception = exception; 
    } 

    public static <T, E extends RuntimeException> ExceptionSupplier<T, E> throwA(E exception) { 
     return new ExceptionSupplier<T, E>(exception); 
    }  

    public static <T, E extends RuntimeException> ExceptionSupplier<T, E> throwA(@SuppressWarnings("UnusedParameters") Class<T> class_, E exception) { 
     return new ExceptionSupplier<T, E>(exception); 
    } 

    @Override 
    public T get() { 
     throw exception; 
    } 
} 

Использование:

Something something = optionalSomething.or(throwA(Something.class, new SomeException())); 

Это, вероятно, может быть продлен еще дальше, но для моих случаев использования это достаточно и легко понимать.

Answer 5

Не имеет значения, что Java 8's Optional имеет метод orElseThrow, который позволяет запрашивать поведение.

Answer 6

См official issue here

Решение: НЕТ - слишком дорого, а не общий шаблон, можно просто использовать isPresent(), бросить

Поскольку optional.orThrow(new Exception()) не хорошо для работы, я предпочитаю статический импорт, который аналогичен @timk 's answer.

Result result = optional.or(throwException()); 

Равно

Result result = optional.or(SomeSupplier.throwException()); 

статический метод

public static Supplier<Result> throwException() { 
    return new Supplier<Result>() { 
     @Override 
     public Result get() { 
      throw new RuntimeException(); 
     } 

    }; 
} 

===========

Трассировка стека выглядит

Exception ... RuntimeException 
at SomeSupplier$1.get(SomeSupplier.java:line where throw RuntimeException) 
at SomeSupplier$1.get(SomeSupplier.java:1) 
at com.google.common.base.Absent.or(Absent.java:60) 
at line where call optional.or(throwException());