Java 8 opsiyonel: ifPresent dönüş nesnesi orElseThrow istisna

Question

Böyle bir şey yapmaya çalışıyorum: ifPresent hükümsüz kabul sahiptir parametre olarak Tüketici fonksiyonel bir arayüz alır, çünkü

private String getStringIfObjectIsPresent(Optional<Object> object){ 
     object.ifPresent(() ->{ 
      String result = "result"; 
      //some logic with result and return it 
      return result; 
     }).orElseThrow(MyCustomException::new); 
    } 

Bu işe yaramaz (T t). Herhangi bir değeri iade edemez. Bunu yapmanın başka yolu var mı?

Answer 1

Ben değerdir emin olduktan sonra eşleme tercih ediyorum mevcut

private String getStringIfObjectIsPresent(Optional<Object> object) { 
    Object ob = object.orElseThrow(MyCustomException::new); 
    // do your mapping with ob 
    String result = your-map-function(ob); 
    return result; 
} 

ya da bir astar

private String getStringIfObjectIsPresent(Optional<Object> object) { 
    return your-map-function(object.orElseThrow(MyCustomException::new)); 
} 

arasında

Answer 2

Bunun yerine map işlevini kullanın. İsteğe bağlı olarak değeri dönüştürür. Bunun gibi

:

private String getStringIfObjectIsPresent(Optional<Object> object) { 
    return object.map(() -> { 
     String result = "result"; 
     //some logic with result and return it 
     return result; 
    }).orElseThrow(MyCustomException::new); 
} 

Answer 3

Aslında ne arıyorsunuz geçerli: Optional.map. Eğer yapabilirsen doğrusu Optional geçen ihmal ediyorum

object.map(o -> "result" /* or your function */) 
     .orElseThrow(MyCustomException::new); 

: gibi Kodunuz sonra görünecektir. Sonunda burada Optional'u kullanarak hiçbir şey kazanamazsınız. Biraz başka varyantı: Zaten nedeniyle başka bir çağrıya Optional -Nesne varsa

public String getString(Object yourObject) { 
    if (Objects.isNull(yourObject)) { // or use requireNonNull instead if NullPointerException suffices 
    throw new MyCustomException(); 
    } 
    String result = ... 
    // your string mapping function 
    return result; 
} 

, hala tek bir nedenle yerine isPresent vs., sen map kullanılan yöntem kullanmak öneriyoruz, ben Daha okunabilir (açıkça öznel bir karar ;-)) bulun. Burada

Answer 4

İki seçenek:

map ile ifPresent değiştirin ve kullanımı Function yerine Consumer ait

private String getStringIfObjectIsPresent(Optional<Object> object) { 
    return object 
      .map(obj -> { 
       String result = "result"; 
       //some logic with result and return it 
       return result; 
      }) 
      .orElseThrow(MyCustomException::new); 
} 

Kullanım isPresent:

private String getStringIfObjectIsPresent(Optional<Object> object) { 
    if (object.isPresent()) { 
     String result = "result"; 
     //some logic with result and return it 
     return result; 
    } else { 
     throw new MyCustomException(); 
    } 
}