Using streams for a null-safe conversion from an array to list












8















I'm looking for a succinct way to rewrite a piece of Java code so that it uses streams to convert an array to a list in a null-safe fashion. Here's the original code:



public MailObject toMailObject(final String ccAddresses) {

    final MailObject mailObject = new MailObject();



    // line of code to be altered

    mailObject.setCcAddresses(ccAddresses == null 

        ? Collections.emptyList() : Arrays.asList(ccAddresses));



    // other necessary code



    return mailObject;

}


I've thought of doing something like this:



// psuedocode that obviously doesn't compile

Optional.ofNullable(ccAddresses).SOMETHING.orElse(Collections.emptyList());


where SOMETHING would be along the lines of:



Arrays.stream(ints).collect(Collectors.toList());


but I can't seem to get the syntax quite right.



This question was helpful but didn't exactly address my issue. Could anyone point me in the right direction? I feel like I'm close...



Thank you very much for your time.






java java-stream







share|improve this question




















  • 3





    Your original code seems succinct enough

    – GBlodgett
    5 hours ago











  • Fair enough, but I'm curious as to the best way it can be done using streams.

    – risingTide
    5 hours ago






  • 1





    .map(Arrays::asList)

    – shmosel
    4 hours ago
















8















I'm looking for a succinct way to rewrite a piece of Java code so that it uses streams to convert an array to a list in a null-safe fashion. Here's the original code:



public MailObject toMailObject(final String ccAddresses) {

    final MailObject mailObject = new MailObject();



    // line of code to be altered

    mailObject.setCcAddresses(ccAddresses == null 

        ? Collections.emptyList() : Arrays.asList(ccAddresses));



    // other necessary code



    return mailObject;

}


I've thought of doing something like this:



// psuedocode that obviously doesn't compile

Optional.ofNullable(ccAddresses).SOMETHING.orElse(Collections.emptyList());


where SOMETHING would be along the lines of:



Arrays.stream(ints).collect(Collectors.toList());


but I can't seem to get the syntax quite right.



This question was helpful but didn't exactly address my issue. Could anyone point me in the right direction? I feel like I'm close...



Thank you very much for your time.






java java-stream







share|improve this question




















  • 3





    Your original code seems succinct enough

    – GBlodgett
    5 hours ago











  • Fair enough, but I'm curious as to the best way it can be done using streams.

    – risingTide
    5 hours ago






  • 1





    .map(Arrays::asList)

    – shmosel
    4 hours ago














8












8








8


1






I'm looking for a succinct way to rewrite a piece of Java code so that it uses streams to convert an array to a list in a null-safe fashion. Here's the original code:



public MailObject toMailObject(final String ccAddresses) {

    final MailObject mailObject = new MailObject();



    // line of code to be altered

    mailObject.setCcAddresses(ccAddresses == null 

        ? Collections.emptyList() : Arrays.asList(ccAddresses));



    // other necessary code



    return mailObject;

}


I've thought of doing something like this:



// psuedocode that obviously doesn't compile

Optional.ofNullable(ccAddresses).SOMETHING.orElse(Collections.emptyList());


where SOMETHING would be along the lines of:



Arrays.stream(ints).collect(Collectors.toList());


but I can't seem to get the syntax quite right.



This question was helpful but didn't exactly address my issue. Could anyone point me in the right direction? I feel like I'm close...



Thank you very much for your time.






java java-stream







share|improve this question
















I'm looking for a succinct way to rewrite a piece of Java code so that it uses streams to convert an array to a list in a null-safe fashion. Here's the original code:



public MailObject toMailObject(final String ccAddresses) {

    final MailObject mailObject = new MailObject();



    // line of code to be altered

    mailObject.setCcAddresses(ccAddresses == null 

        ? Collections.emptyList() : Arrays.asList(ccAddresses));



    // other necessary code



    return mailObject;

}


I've thought of doing something like this:



// psuedocode that obviously doesn't compile

Optional.ofNullable(ccAddresses).SOMETHING.orElse(Collections.emptyList());


where SOMETHING would be along the lines of:



Arrays.stream(ints).collect(Collectors.toList());


but I can't seem to get the syntax quite right.



This question was helpful but didn't exactly address my issue. Could anyone point me in the right direction? I feel like I'm close...



Thank you very much for your time.






java java-stream




java java-stream






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited 3 hours ago







risingTide

















asked 5 hours ago









risingTiderisingTide

60841238




60841238








  • 3





    Your original code seems succinct enough

    – GBlodgett
    5 hours ago











  • Fair enough, but I'm curious as to the best way it can be done using streams.

    – risingTide
    5 hours ago






  • 1





    .map(Arrays::asList)

    – shmosel
    4 hours ago














  • 3





    Your original code seems succinct enough

    – GBlodgett
    5 hours ago











  • Fair enough, but I'm curious as to the best way it can be done using streams.

    – risingTide
    5 hours ago






  • 1





    .map(Arrays::asList)

    – shmosel
    4 hours ago








3




3





Your original code seems succinct enough

– GBlodgett
5 hours ago





Your original code seems succinct enough

– GBlodgett
5 hours ago













Fair enough, but I'm curious as to the best way it can be done using streams.

– risingTide
5 hours ago





Fair enough, but I'm curious as to the best way it can be done using streams.

– risingTide
5 hours ago




1




1





.map(Arrays::asList)

– shmosel
4 hours ago





.map(Arrays::asList)

– shmosel
4 hours ago












2 Answers
2






active

oldest

votes


















6














You might use the map :



List<String> ccAddrs = Optional.ofNullable(ccAddress)

                               .map(Arrays::asList)

                               .orElse(Collections.emptyList())





share|improve this answer































    4














    In my opinion, the code that you have so far is perfectly readable. I think using a Stream for something like this will just complicate things. If you still want to use a Stream, then something like the following would work:



    mailObject.setCcAddresses(Stream.ofNullable(ccAddresses)
    
          .flatMap(Arrays::stream)
    
          .collect(Collectors.toUnmodifiableList()));


    As you can see, this is a more unreadable, and I wouldn't recommend it over your simple ternary expression.




    Your Optional solution is slightly more readable and would look like the following:



    mailObject.setCcAddresses(Optional.ofNullable(ccAddresses)
    
          .map(Arrays::asList)
    
          .orElse(Collections.emptyList()));





    share|improve this answer

























      Your Answer






      StackExchange.ifUsing("editor", function () {
      StackExchange.using("externalEditor", function () {
      StackExchange.using("snippets", function () {
      StackExchange.snippets.init();
      });
      });
      }, "code-snippets");

      StackExchange.ready(function() {
      var channelOptions = {
      tags: "".split(" "),
      id: "1"
      };
      initTagRenderer("".split(" "), "".split(" "), channelOptions);

      StackExchange.using("externalEditor", function() {
      // Have to fire editor after snippets, if snippets enabled
      if (StackExchange.settings.snippets.snippetsEnabled) {
      StackExchange.using("snippets", function() {
      createEditor();
      });
      }
      else {
      createEditor();
      }
      });

      function createEditor() {
      StackExchange.prepareEditor({
      heartbeatType: 'answer',
      autoActivateHeartbeat: false,
      convertImagesToLinks: true,
      noModals: true,
      showLowRepImageUploadWarning: true,
      reputationToPostImages: 10,
      bindNavPrevention: true,
      postfix: "",
      imageUploader: {
      brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
      contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
      allowUrls: true
      },
      onDemand: true,
      discardSelector: ".discard-answer"
      ,immediatelyShowMarkdownHelp:true
      });


      }
      });














      draft saved

      draft discarded


















      StackExchange.ready(
      function () {
      StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f55272522%2fusing-streams-for-a-null-safe-conversion-from-an-array-to-list%23new-answer', 'question_page');
      }
      );

      Post as a guest















      Required, but never shown

























      2 Answers
      2






      active

      oldest

      votes








      2 Answers
      2






      active

      oldest

      votes









      active

      oldest

      votes






      active

      oldest

      votes









      6














      You might use the map :



      List<String> ccAddrs = Optional.ofNullable(ccAddress)
      
                               .map(Arrays::asList)
      
                               .orElse(Collections.emptyList())





      share|improve this answer




























        6














        You might use the map :



        List<String> ccAddrs = Optional.ofNullable(ccAddress)
        
                               .map(Arrays::asList)
        
                               .orElse(Collections.emptyList())





        share|improve this answer


























          6












          6








          6







          You might use the map :



          List<String> ccAddrs = Optional.ofNullable(ccAddress)
          
                               .map(Arrays::asList)
          
                               .orElse(Collections.emptyList())





          share|improve this answer













          You might use the map :



          List<String> ccAddrs = Optional.ofNullable(ccAddress)
          
                               .map(Arrays::asList)
          
                               .orElse(Collections.emptyList())






          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered 4 hours ago









          Mạnh Quyết NguyễnMạnh Quyết Nguyễn

          11k1930




          11k1930

























              4














              In my opinion, the code that you have so far is perfectly readable. I think using a Stream for something like this will just complicate things. If you still want to use a Stream, then something like the following would work:



              mailObject.setCcAddresses(Stream.ofNullable(ccAddresses)
              
          .flatMap(Arrays::stream)
              
          .collect(Collectors.toUnmodifiableList()));


              As you can see, this is a more unreadable, and I wouldn't recommend it over your simple ternary expression.




              Your Optional solution is slightly more readable and would look like the following:



              mailObject.setCcAddresses(Optional.ofNullable(ccAddresses)
              
          .map(Arrays::asList)
              
          .orElse(Collections.emptyList()));





              share|improve this answer






























                4














                In my opinion, the code that you have so far is perfectly readable. I think using a Stream for something like this will just complicate things. If you still want to use a Stream, then something like the following would work:



                mailObject.setCcAddresses(Stream.ofNullable(ccAddresses)
                
          .flatMap(Arrays::stream)
                
          .collect(Collectors.toUnmodifiableList()));


                As you can see, this is a more unreadable, and I wouldn't recommend it over your simple ternary expression.




                Your Optional solution is slightly more readable and would look like the following:



                mailObject.setCcAddresses(Optional.ofNullable(ccAddresses)
                
          .map(Arrays::asList)
                
          .orElse(Collections.emptyList()));





                share|improve this answer




























                  4












                  4








                  4







                  In my opinion, the code that you have so far is perfectly readable. I think using a Stream for something like this will just complicate things. If you still want to use a Stream, then something like the following would work:



                  mailObject.setCcAddresses(Stream.ofNullable(ccAddresses)
                  
          .flatMap(Arrays::stream)
                  
          .collect(Collectors.toUnmodifiableList()));


                  As you can see, this is a more unreadable, and I wouldn't recommend it over your simple ternary expression.




                  Your Optional solution is slightly more readable and would look like the following:



                  mailObject.setCcAddresses(Optional.ofNullable(ccAddresses)
                  
          .map(Arrays::asList)
                  
          .orElse(Collections.emptyList()));





                  share|improve this answer















                  In my opinion, the code that you have so far is perfectly readable. I think using a Stream for something like this will just complicate things. If you still want to use a Stream, then something like the following would work:



                  mailObject.setCcAddresses(Stream.ofNullable(ccAddresses)
                  
          .flatMap(Arrays::stream)
                  
          .collect(Collectors.toUnmodifiableList()));


                  As you can see, this is a more unreadable, and I wouldn't recommend it over your simple ternary expression.




                  Your Optional solution is slightly more readable and would look like the following:



                  mailObject.setCcAddresses(Optional.ofNullable(ccAddresses)
                  
          .map(Arrays::asList)
                  
          .orElse(Collections.emptyList()));






                  share|improve this answer














                  share|improve this answer



                  share|improve this answer








                  edited 4 hours ago

























                  answered 4 hours ago









                  Jacob G.Jacob G.

                  16.3k52363




                  16.3k52363






























                      draft saved

                      draft discarded




















































                      Thanks for contributing an answer to Stack Overflow!


                      • Please be sure to answer the question. Provide details and share your research!

                      But avoid …



                      • Asking for help, clarification, or responding to other answers.

                      • Making statements based on opinion; back them up with references or personal experience.


                      To learn more, see our tips on writing great answers.




                      draft saved


                      draft discarded














                      StackExchange.ready(
                      function () {
                      StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f55272522%2fusing-streams-for-a-null-safe-conversion-from-an-array-to-list%23new-answer', 'question_page');
                      }
                      );

                      Post as a guest















                      Required, but never shown





















































                      Required, but never shown














                      Required, but never shown












                      Required, but never shown







                      Required, but never shown

































                      Required, but never shown














                      Required, but never shown












                      Required, but never shown







                      Required, but never shown







                      -

                      Popular posts from this blog

                      Ponta tanko

                      Image
                      Usona ponta tanko M60A1 Ponta tanko estas, simile kiel kirasita savveturilo, inter specialaj kirasitaj transportiloj. Kiel pontaj tankoj eblas konsideri ĉiujn memmovigaj rimedoj ebligantaj superi obstaklojn ĝis larĝeco 24 metroj rekte en spaco de bataloperacoj. Ties ĉefa tasko estas ebligi al la propraj taĉmentoj daŭrigi en rapida antaŭenirado sen senbezona reteniĝo. El tio pli-malpli rezultas taktikaj-teknikaj postuloj, kiujn ili devas plenumi. Precipe temas pri kapablo operacii komune kun unue-sinsekvaj taĉmentoj kaj transpontigi obstaklojn dum kelke da minutoj. This page is only for reference, If you need detailed information, please check here
                      Read more

                      Tantalo (mitologio)

                      Image
                      Ĉi tiu artikolo temas pri mitologio. Por aliaj signifoj vidu la paĝon Tantalo (apartigilo) . Tantalo , laŭ Gioacchino Assereto. Laŭ la helena mitologio, Tantalo (el la greka Τάνταλος , ŝancelanta aŭ tre malfeliĉa ) estis filo de Zeŭso kaj la oceanidino Pluto, laŭ la plej divastigata versio. Li estis reĝo de Frigio [1] aŭ de la monto Sipilo, en Lidio (Malgrandazio) [2] kaj post sia morto li suferis eternan torturadon ĉe la Tartaro, la plej profunda parto de Hadeso, rezervita por la plej perversaj homoj. Tamen, kelkaj rakontoj asertas ke li estis reĝo de Argo aŭ de Korinto [3] , aŭ ke li migris norden el la monto Sipilo por akiri la tronon de Paflagonio, el kie Ilo forpelis lin pro tio, ke Tantalo forkaptis kaj seksperfortis lian fraton Ganimedon [4] , kaj tiel ofendis la diojn. Verŝajne li ankaŭ regis urbon en okcidenta Anatolio, kiun oni nomis Tantalis [5] pro li, aŭ Sipilo , pro la monto sur kies flanko ankoraŭ staras ĝiaj ruinaĵoj [6] . Laŭ la geografiisto Strabo,...
                      Read more

                      Erzsébet Schaár

                      Image
                      Erzsébet Schaár Memportreto de Erzsébet Schaár en Pécs Personaj informoj Naskiĝo 27-an de julio 1908 ( 1908-07-27 ) en Budafok Morto 29-an de aŭgusto 1975 ( 1975-08-29 ) (67-jara) en Budapeŝto Ŝtataneco Hungario Profesio skulptisto Familianoj Edz(in)o Tibor Vilt v   •   d   •   r Erzsébet Schaár [erĵEbet ŝAr], laŭ hungarlingve kutima sinsekvo Schaár Erzsébet estis hungara skulptisto. Ŝia edzo estis Tibor Vilt. Déryné Széppataki Róza en Miskolc fare de Erzsébet Schaár Erzsébet Schaár [1] naskiĝis la 27-an de julio 1908 en Budafok, ŝi mortis la 29-an de aŭgusto 1975 en Budapeŝto. Enhavo 1 Biografio 2 Elektitaj solaj ekspozicioj 3 Elektitaj kolektivaj ekspozicioj 4 Elektitaj skulptaĵoj 5 Elektitaj disĉiploj 6 Fontoj 7 Referencoj Biografio | Erzsébet Schaár studis en Industriarta Lernejo, poste en Reĝa Belarta Altlernejo (1924-1926). Ŝia ĉefinstruisto estis Zsig...
                      Read more