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


















7














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









      7














      You might use the map :



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





      share|improve this answer




























        7














        You might use the map :



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





        share|improve this answer


























          7












          7








          7







          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

                      Liste der Baudenkmale in Friedland (Mecklenburg)

                      Image
                      In der Liste der Baudenkmale in Friedland sind alle denkmalgeschützten Bauten der Stadt Friedland (Mecklenburg-Vorpommern) und ihrer Ortsteile aufgelistet. Grundlage ist die Veröffentlichung der Denkmalliste des Landkreises Mecklenburgische Seenplatte (Auszug) vom 28. September 2016. Inhaltsverzeichnis 1 Baudenkmale nach Ortsteilen 1.1 Friedland 1.2 Bresewitz 1.3 Brohm 1.4 Cosa 1.5 Dishley 1.6 Dishley-Ausbau 1.7 Eichhorst 1.8 Glienke 1.9 Heinrichshöh 1.10 Heinrichswalde 1.11 Jatzke 1.12 Jatzke-Meierei 1.13 Liepen 1.14 Ramelow 1.15 Schwanbeck 2 Vorgesehener Denkmalbereich 3 Quelle 4 Weblinks Baudenkmale nach Ortsteilen | Friedland | ID Lage Bezeichnung Beschreibung Bild 281 An der Kleinbahn 2 Windmühle mit Umgang 282 Anklamer Straße alter Friedhof mit Portalwand 282 Anklamer Straße drei Erbbegräbnissen und Gräbern u. a. H.A. Riemann 284
                      Read more

                      Single-Malt-Whisky

                      Image
                      Fünf verschiedene schottische Single-Malt-Whiskys Als Single-Malt-Whisky oder kürzer Single Malt werden Whiskys bezeichnet, die zwei besondere Bedingungen erfüllen: Sie stammen zum ersten aus einer einzigen Brennerei, sind also kein Verschnitt aus mehreren Whiskysorten (daher Single , deutsch: einzeln). Zum zweiten wird als Getreide ausschließlich gemälzte Gerste verwendet (deswegen Malt , deutsch: Malz). Der Zusatz Single Malt wird als Prädikat verstanden, die so ausgezeichneten Whiskys sind in aller Regel besonders hochwertig. Hinzu kommt ihre besonders lange Lagerung, die Reifungszeit währt oft ein Jahrzehnt und länger. Inhaltsverzeichnis 1 Allgemeines 2 Herkunftsländer 2.1 Schottland 2.2 Irland 2.3 Japan 2.4 Kontinentaleuropa 3 Produktionsschritte 3.1 Mälzen 3.2 Brauen 3.3 Brennen 3.4 Reifung und Finish 3.5 Abfüllung 4 Altersangaben 5 Lagerung von Whiskyflaschen 6 Geschmacksklassifikation 7 Sonstig
                      Read more

                      Czorneboh

                      Image
                      Dieser Artikel erläutert den Berg Czorneboh, für die Gottheit siehe Czorneboh (Gottheit). Czorneboh Čornobóh Berggasthof mit Turm Höhe 555,7  m ü. NHN [1] Lage Freistaat Sachsen, Deutschland Gebirge Lausitzer Bergland Dominanz 15,4 km →  Kottmar Koordinaten 51° 7′ 11″  N , 14° 31′ 31″  O 51.119722222222 14.525194444444 555.7 Koordinaten: 51° 7′ 11″  N , 14° 31′ 31″  O Gestein Granodiorit Besonderheiten Aussichtsturm Czorneboh Der Czorneboh , obersorbisch Čornobóh , ist ein Berg zwischen Hochkirch und Cunewalde in der Oberlausitz. Er gehört zu der südöstlich von Bautzen gelegenen Czorneboh-Bergkette und ist mit einer Höhe von 555,7  m ü. NHN [1] der höchste Punkt dieses Ausläufers des Lausitzer Berglandes. Er leitet als Landschaftselement die wellige Granitlandschaft der Lausitzer Vorbergzone ein. Der Gipfel des Czorneboh liegt in der Gemarkung Meschwitz (Gemeinde Hochkirch), die Czornebohbaude jedoch
                      Read more