Sfrgttk

How do I fix the group tension caused by my character stealing and possibly killing without provocation?

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

Alignment of six matrices

Is there a distance limit for minecart tracks?

Giving feedback to someone without sounding prejudiced

How to leave product feedback on macOS?

How can I safely use "Thalidomide" in my novel while respecting the trademark?

How much do grades matter for a future academia position?

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

Is there anyway, I can have two passwords for my wi-fi

Usage of an old photo with expired copyright

If Captain Marvel (MCU) were to have a child with a human male, would the child be human or Kree?

Isometric embedding of a genus g surface

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

Mimic lecturing on blackboard, facing audience

Is it feasible to let a newcomer play the "Gandalf"-like figure I created for my campaign?

What is this high flying aircraft over Pennsylvania?

What is the meaning of "You've never met a graph you didn't like?"

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

How do you justify more code being written by following clean code practices?

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

Why would five hundred and five be same as one?

In One Punch Man, is King actually weak?

Quoting Keynes in a lecture

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

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

edited 3 hours ago

risingTide

asked 4 hours ago

risingTiderisingTide

60841238

• 
  
  
  

  
  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 | 

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

edited 3 hours ago

risingTide

asked 4 hours ago

risingTiderisingTide

60841238

• 
  
  
  

  
  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

60841238

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

60841238

share|improve this question

edited 3 hours ago

risingTide

asked 4 hours ago

risingTiderisingTide

60841238

asked 4 hours ago

risingTiderisingTide

60841238

asked 4 hours ago

risingTiderisingTide

60841238

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

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 | 

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

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

6

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 | 

6

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

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

edited 4 hours ago

answered 4 hours ago

Jacob G.Jacob G.

16.3k52363

add a comment | 

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

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