Zrjydtj

Do people actually use the word "kaputt" in conversation?

Can I run 125kHz RF circuit on a breadboard?

Why does a 97 / 92 key piano exist by Bösendorfer?

Why can't the Brexit deadlock in the UK parliament be solved with a plurality vote?

Is there a reason to prefer HFS+ over APFS for disk images in High Sierra and/or Mojave?

Difference between shutdown options

Sigmoid with a slope but no asymptotes?

Sound waves in different octaves

How would you translate "more" for use as an interface button?

Why do Radio Buttons not fill the entire outer circle?

Why does the Persian emissary display a string of crowned skulls?

What the heck is gets(stdin) on site coderbyte?

How do I tell my boss that I'm quitting in 15 days (a colleague left this week)

If the only attacker is removed from combat, is a creature still counted as having attacked this turn?

Does Doodling or Improvising on the Piano Have Any Benefits?

Has the laser at Magurele, Romania reached a tenth of the Sun's power?

Deciphering cause of death?

How to make money from a browser who sees 5 seconds into the future of any web page?

Do I have to know the General Relativity theory to understand the concept of inertial frame?

What does "Scientists rise up against statistical significance" mean? (Comment in Nature)

Why is participating in the European Parliamentary elections used as a threat?

How to preserve electronics (computers, iPads and phones) for hundreds of years

Should I assume I have passed probation?

How do I prevent inappropriate ads from appearing in my game?

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

Converting Array to ListCreate ArrayList from arrayHow to simplify a null-safe compareTo() implementation?Converting array to list in JavaConvert list to array in JavaRetrieving a List from a java.util.stream.Stream in Java 8How to Convert a Java 8 Stream to an Array?Java 8 Stream and operation on arraysHow can I create a stream from an array?How to sum a list of integers with java streams?Java 8 stream - how to properly make NPE-safe stream

7

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

edited 3 hours ago

risingTide

asked 4 hours ago

risingTiderisingTide

60341238

• 
  
  
  

  
  3
  

  
  
  
  
  
  

  
  
  Your original code seems succinct enough
  
  – GBlodgett
  4 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

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

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  .map(Arrays::asList)
  
  – shmosel
  4 hours ago
  

  
  
  
  

add a comment | 

7

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

edited 3 hours ago

risingTide

asked 4 hours ago

risingTiderisingTide

60341238

• 
  
  
  

  
  3
  

  
  
  
  
  
  

  
  
  Your original code seems succinct enough
  
  – GBlodgett
  4 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

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

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  .map(Arrays::asList)
  
  – shmosel
  4 hours ago
  

  
  
  
  

add a comment | 

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

edited 3 hours ago

risingTide

asked 4 hours ago

risingTiderisingTide

60341238

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;

// psuedocode that obviously doesn't compile
Optional.ofNullable(ccAddresses).SOMETHING.orElse(Collections.emptyList());

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

share|improve this question

edited 3 hours ago

risingTide

asked 4 hours ago

risingTiderisingTide

60341238

share|improve this question

edited 3 hours ago

risingTide

asked 4 hours ago

risingTiderisingTide

60341238

asked 4 hours ago

risingTiderisingTide

60341238

asked 4 hours ago

risingTiderisingTide

60341238

2 Answers
2

active

oldest

votes

5

You might use the map :

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

share|improve this answer

answered 4 hours ago

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

11k1930

• 
  
  
  

  
  2
  

  
  
  
  
  
  

  
  
  @downvoter: Should let me know the reason :)
  
  – Mạnh Quyết Nguyễn
  4 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  They don't . Because they are random clickers
  
  – soorapadman
  2 hours ago
  

  
  
  
  

add a comment | 

3

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

edited 4 hours ago

answered 4 hours ago

Jacob G.Jacob G.

16.3k52363

add a comment | 

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

Sign up or log in

StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);

Sign up using Google

Sign up using Facebook

Sign up using Email and Password

Post as a guest

Name

Email

Required, but never shown

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

Name

Email

Required, but never shown

5

You might use the map :

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

share|improve this answer

answered 4 hours ago

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

11k1930

• 
  
  
  

  
  2
  

  
  
  
  
  
  

  
  
  @downvoter: Should let me know the reason :)
  
  – Mạnh Quyết Nguyễn
  4 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  They don't . Because they are random clickers
  
  – soorapadman
  2 hours ago
  

  
  
  
  

add a comment | 

5

You might use the map :

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

share|improve this answer

answered 4 hours ago

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

11k1930

• 
  
  
  

  
  2
  

  
  
  
  
  
  

  
  
  @downvoter: Should let me know the reason :)
  
  – Mạnh Quyết Nguyễn
  4 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  They don't . Because they are random clickers
  
  – soorapadman
  2 hours ago
  

  
  
  
  

add a comment | 

You might use the map :

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

share|improve this answer

answered 4 hours ago

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

11k1930

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

share|improve this answer

answered 4 hours ago

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

11k1930

answered 4 hours ago

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

11k1930

answered 4 hours ago

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

11k1930

3

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

edited 4 hours ago

answered 4 hours ago

Jacob G.Jacob G.

16.3k52363

add a comment | 

3

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

edited 4 hours ago

answered 4 hours ago

Jacob G.Jacob G.

16.3k52363

add a comment | 

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

edited 4 hours ago

answered 4 hours ago

Jacob G.Jacob G.

16.3k52363

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

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

share|improve this answer

edited 4 hours ago

answered 4 hours ago

Jacob G.Jacob G.

16.3k52363

answered 4 hours ago

Jacob G.Jacob G.

16.3k52363

answered 4 hours ago

Jacob G.Jacob G.

16.3k52363