История изменений

Исправление foror, 14.12.17 07:57 (текущая версия) :

Я не понял про что тут, но как я и говорил назначение Optional в оборачивании null значений при возврате из метода.

Это нормальный код (для этого и придуман Optional):

String keyId = jws.getKeyIdHeaderValue()
    .orElseThrow(() -> new TokenCorruptedException("no kid header"));

return store.get(keyId)
    .orElseThrow(() -> new TokenUnsignedException("unknown or expired kid: " + keyId));

Вот это ненормальный:

String keyId = Optional
    .ofNullable(jws.getKeyIdHeaderValue())
    .orElseThrow(() -> new TokenCorruptedException("no kid header"));

return Optional
    .ofNullable(store.get(keyId))
    .orElseThrow(() -> new TokenUnsignedException("unknown or expired kid: " + keyId));

Исходная версия foror, 14.12.17 07:56:

Я не понял про что тут, но как я и говорил назначение Optional в оборачивании null значений при возврате из метода.

Это нормальный код:

String keyId = jws.getKeyIdHeaderValue()
    .orElseThrow(() -> new TokenCorruptedException("no kid header"));

return store.get(keyId)
    .orElseThrow(() -> new TokenUnsignedException("unknown or expired kid: " + keyId));

Вот это ненормальный:

String keyId = Optional
    .ofNullable(jws.getKeyIdHeaderValue())
    .orElseThrow(() -> new TokenCorruptedException("no kid header"));

return Optional
    .ofNullable(store.get(keyId))
    .orElseThrow(() -> new TokenUnsignedException("unknown or expired kid: " + keyId));