웹 서비스의 반환이 Optional을 사용하여 null인지 확인하고 noSuchElementException을 피하는 방법을 고차 함수를 사용하는 방법

Question

컨텍스트 : 나는 menbers가있는 패밀리를 반환하는 웹 서비스가 있습니다. 가족에게는 항상 아버지와 어머니, 자녀가 없거나 여러 개의 자녀가 있습니다. 서비스는 wsdl에 의해 아래에서 설명됩니다.

목적 : Java 8에서 선택 사항을 효과적으로 사용하고 null을 확인하는 고전적인 방법을 사용하지 않겠습니다.

@Test 
public void test1() { 
    Family f = helloWorldClientImplBean.allFamily(); 

    f.getChildren().stream().filter(x -> x.getFirstName().equalsIgnoreCase("John")).findFirst() 
      .ifPresent(y -> System.out.println(y.getLastName())); 
} 

내가 테스트하고 나는 것을 볼 수 있습니다 : 고전, 나는 우리가 내가 웹 서비스는 항상 가족을 반환한다고 가정하면 자바 7

까지 달성하는 데 사용 된 방법을 의미하는이 충분하다 , 나는 가족이 봉사함으로써 응답을받는 한, 나는 자녀가 있는지 없는지 상관없이 완벽하게 작동 할 것입니다. 나는 아래의 서비스 구현에서 oldSon 및 youngSon 코드에 null 예외가 전혀 없다고 주석을 달았습니다.

서비스가 null을 반환하면 문제가 발생합니다.

블로그를 읽고 그것에 대한 토론을 마친 후 서비스 반환이 null인지 제대로 확인하는이 코드에 도달했습니다.

@Test 
public void testWorkingButSeemsOdd() { 

    //Family f = helloWorldClientImplBean.allFamily(); 
    Family f = null; //to make simple the explanation 

    Optional<Family> optFamily = Optional.ofNullable(f); 

    if (optFamily.isPresent()) { 

     optFamily.filter(Objects::nonNull).map(Family::getChildren).get().stream().filter(Objects::nonNull) 
       .filter(x -> x.getFirstName().equalsIgnoreCase("John")).findFirst() 
       .ifPresent(y -> System.out.println("Optional: " + y.getLastName())); 

    } 

나 이러한 방법 중 하나 (모두가 failling하고 있지만 나는 그들이 내가 할 노력하고 무엇을 보여줄 수 있다고 생각) 것 더 깨끗한 것이 무엇 : 여기에 내가 시도

// f는 매핑

@Test 
public void testFilterNonNull() { 
    Family f = null; 
    Optional.ofNullable(f).filter(Objects::nonNull).map(Family::getChildren).get().stream().filter(Objects::nonNull) 
      .filter(x -> x.getFirstName().equalsIgnoreCase("John")).findFirst() 
      .ifPresent(y -> System.out.println(y.getLastName())); 

} 

내가 컴파일되지 않습니다 다음을 알고 있지만 나는 비슷한에 도달 할 수 있습니다 생각하기 전에 null가 아닌 경우, 필터링하는

@Test 
@Ignore 
public void testOptionalNullable() { 
    Family f = helloWorldClientImplBean.allFamily(); 

    Optional.ofNullable(f).orElse(System.out.println("Family is null")).map(Family::getChildren).get().stream().filter(Objects::nonNull) 
      .filter(x -> x.getFirstName().equalsIgnoreCase("John")).findFirst() 
      .ifPresent(y -> System.out.println(y.getLastName())); 

} 

01 23,516,

WSDL

<?xml version="1.0" encoding="UTF-8" standalone="no"?> 
<wsdl:definitions targetNamespace="http://codenotfound.com/services/helloworld" 
    xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:tns="http://codenotfound.com/services/helloworld" 
    xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:xsd="http://www.w3.org/2001/XMLSchema" 
    name="HelloWorld"> 

    <wsdl:types> 
     <schema targetNamespace="http://codenotfound.com/services/helloworld" 
      xmlns="http://www.w3.org/2001/XMLSchema" xmlns:tns="http://codenotfound.com/services/helloworld" 
      elementFormDefault="qualified" attributeFormDefault="unqualified" 
      version="1.0"> 

      <element name="family"> 
       <complexType> 
        <sequence> 
         <element name="father" type="tns:persontype" minOccurs="1" 
          maxOccurs="1" /> 
         <element name="mother" type="tns:persontype" minOccurs="1" 
          maxOccurs="1" /> 
         <element name="children" type="tns:persontype" minOccurs="0" 
          maxOccurs="unbounded" /> 
        </sequence> 
       </complexType> 
      </element> 

      <complexType name="persontype"> 
       <sequence> 
        <element name="firstName" type="xsd:string" /> 
        <element name="lastName" type="xsd:string" /> 
       </sequence> 
      </complexType> 

      <element name="EmptyParameter" type="tns:voidType" /> 

      <complexType name="voidType"> 
       <sequence /> 
      </complexType> 
     </schema> 
    </wsdl:types> 

    <!-- Message --> 

    <wsdl:message name="emptyRequest"> 
     <wsdl:part name="emptyParameter" element="tns:EmptyParameter" /> 
    </wsdl:message> 

    <wsdl:message name="allFamiliesResponse"> 
     <wsdl:part name="allFamiliesResponse" element="tns:family" /> 
    </wsdl:message> 

    <!-- PortType --> 

    <wsdl:operation name="allFamilies"> 
      <wsdl:input message="tns:emptyRequest" /> 
      <wsdl:output message="tns:allFamiliesResponse"></wsdl:output> 
     </wsdl:operation> 
    </wsdl:portType> 

    <!-- Binding --> 

    <wsdl:binding name="HelloWorld_Binding" type="tns:HelloWorld_PortType"> 
     <soap:binding style="document" 
      transport="http://schemas.xmlsoap.org/soap/http" /> 

     <wsdl:operation name="allFamilies"> 
      <wsdl:input> 
       <soap:body use="literal" /> 
      </wsdl:input> 
      <wsdl:output> 
       <soap:body use="literal" /> 
      </wsdl:output> 
     </wsdl:operation> 
    </wsdl:binding> 

    <wsdl:service name="HelloWorld_Service"> 
     <wsdl:port name="HelloWorld_Port" binding="tns:HelloWorld_Binding"> 
      <soap:address location="http://localhost:9090/cnf/services/helloworld" /> 
     </wsdl:port> 
    </wsdl:service> 
</wsdl:definitions> 

서비스 구현의

관련 부분 :

그래서, 내 바로 질문입니다

@Override 
public Family allFamilies(VoidType emptyParameter) { 
    ObjectFactory factory = new ObjectFactory(); 
    Family result = factory.createFamily(); 
    Persontype father = new Persontype(); 
    father.setFirstName("Jose"); 
    father.setLastName("Pereira"); 

    Persontype mother = new Persontype(); 
    mother.setFirstName("Maria"); 
    mother.setLastName("Pereira"); 

    result.setFather(father); 
    result.setMother(mother); 


    Persontype olderSon = new Persontype(); 
    olderSon.setFirstName("John"); 
    olderSon.setLastName("Pereira"); 

    Persontype youngerSon = new Persontype(); 
    youngerSon.setFirstName("Ana"); 
    youngerSon.setLastName("Pereira"); 
    result.getChildren().add(olderSon); 
    result.getChildren().add(youngerSon); 

    return result; 
} 

: WSDL 및 구현 전술 내 시나리오에 따라이 있는지 확인하기 위해 정말 유일한 방법입니다 우리가 고전적인 null 검사 (if (f! = null) {...)와 매우 유사한 방식으로 isPresent()를 사용하여 웹 서비스로부터의 반환 값은 null이됩니까?

Answer 1

오해의 주된 오해는 선택적으로 .filter(Objects::nonNull)과 같은 작업을 수행해야한다고 가정하는 것입니다. 빈 optional에 필터링이 필요한 경우, 옵션의 전체 목적을 무효화합니다. 특히, 술어가 false으로 평가 될 때 필터링의 결과가 다시 빈 선택 사항이되므로, 다시 정사각형 1로 되돌립니다.

사실, 은 비어 있지 않은 옵션의 경우 항상 true이고, 어쨌든 평가되지는 않는 빈 옵션의 경우 .filter(x -> true)과 동일한 효과가 있습니다.

또한 ifPresent에 대해 이미 알고 있음에도 불구하고 if 문으로 다시 전환됩니다.그래서 원래의 코드에서 파생 된 하나 똑바로 앞으로 솔루션은

Optional.ofNullable(helloWorldClientImplBean.allFamily()) 
     .ifPresent(f -> f.getChildren().stream() 
      .filter(x -> x.getFirstName().equalsIgnoreCase("John")) 
      .findFirst() 
      .ifPresent(y -> System.out.println(y.getLastName())); 

당신은이 주소 만에게

Optional.ofNullable(helloWorldClientImplBean.allFamily()) 
     .flatMap(f -> f.getChildren().stream() 
      .filter(x -> x.getFirstName().equalsIgnoreCase("John")) 
      .findFirst()) 
     .ifPresent(y -> System.out.println(y.getLastName())); 

에 동작을 변경하여 귀하의 질문, 즉에서 설명하는 문제를 중첩 된 부분을 줄일 수있을 것입니다 서비스 allFamily()은 null을 반환 할 수 있습니다. 또한 하위 인스턴스가 null 인 경우를 처리 할 스트림 작업에 새로운 null 체크를 포함 시켰습니다. 그 정말로 필요한 경우

는 가장 좋은 방법은 서비스 구현에 대한 책임이 누구의 엉덩이에 킥 겠지만, 어쨌든, 두 번째 가장 좋은 해결책은 간단하다

Optional.ofNullable(helloWorldClientImplBean.allFamily()) 
     .flatMap(f -> f.getChildren().stream() 
      .filter(x -> x!=null && x.getFirstName().equalsIgnoreCase("John")) 
      .findFirst()) 
     .ifPresent(y -> System.out.println(y.getLastName())); 

을하는 것입니다 추가 스트림 .filter(Objects::nonNull)을 스트림에 삽입하는 것보다

Answer 2

Optional으로 결과를 처리하려면 Holger's answer을 참조하십시오. 여기서 저는 다른 접근 방식을 취하고 싶습니다.

자신에게 하나의 질문을하십시오. 왜이 경우를 처리하려면 Optional이 필요할까요? 반환 된 Family 값에 if 블록이있는 Null 검사를 사용하지 않는 것이 맞습니까?

이 코드를 고려

Family f = helloWorldClientImplBean.allFamily(); 

if (f != null) { 
    f.getChildren().stream() 
     .filter(x -> x.getFirstName().equalsIgnoreCase("John")) 
     .findFirst() 
     .ifPresent(y -> System.out.println(y.getLastName())); 
} 

그것은 선명한, 읽기 쉽고 유지 보수가 용이하다.

지금 코드 이런 식으로, 즉 고려 :

Optional.ofNullable(helloWorldClientImplBean.allFamily()) 
    .map(Family::getChildren) 
    .map(Collection::stream) 
    .map(stream -> stream.filter(x -> "John".equalsIgnoreCase(x.getFirstName()))) 
    .flatMap(Stream::findFirst) 
    .map(Persontype::getLastName) 
    .ifPresent(System.out::println); 

이 코드는 기능적인 스타일이다. 모든 작업은 Optional.flatMap을 통해 수행되는 스트림의 첫 번째 요소를 반환하는 작업을 제외한 Optional을 통해 Optional.map 작업을 통해 수행됩니다. 스트림은 단계별로 처리되며 1 회선으로 처리되지 않습니다. Optional에 대해 수행 된 모든 작업은 null 안전합니다. 즉, 반환 된 초기 Family 인스턴스가 null인지 여부뿐만 아니라 f.getChildren()이 null을 반환하는지 여부도 확인한다는 의미입니다.

어떤 버전이 더 짧습니까? 어느 것이 더 우아한가? 어느 것이 더 명확하고 이해하기 쉬우십니까? 어느 것이 가장 좋은 방법으로 프로그래머의 의도를 표현합니까?

내가 답을 알고 ...