How to add a set of values from an ArrayList to another ArrayList using Lambda expression

Question

I have to create a result list by adding objects from one array list to another. Here is my code that works.

    private List<MyObject> getObjectList(List<OtherObject> objects) {
        List<MyObject> resultList = new ArrayList<>();
        for (int i = 0; i < objects.size(); i++) {
        MyObject object = new MyObject()
            object.setId(objects.get(i).getId());
            object.setText(objects.get(i).getTextValue());
            object.setUserId(objects.get(i).getUserName());
            object.setCreatedDate(objects.get(i).getCreateDateTime());

            resultList.add(object);
        }
        
        return resultList;
    }

How can I achieve this by using lambda expression?

Your loop solution can be significantly improved by using a for-each loop — Hulk
– Hulk, Commented Oct 5, 2020 at 15:37
With "using lambda expression", I assume you mean using Stream? — Hulk
– Hulk, Commented Oct 5, 2020 at 15:38
Does this answer your question? How to clone ArrayList and also clone its contents? — Hulk
– Hulk, Commented Oct 5, 2020 at 15:44
In your case it's not a straight-forward copy, as you are mapping to a different element type, but the basics are the same. I'd write either a constructor for MyObject that accepts a OtherObject. — Hulk
– Hulk, Commented Oct 5, 2020 at 15:48

George · Accepted Answer · 2020-10-05 16:19:31Z

3

Here's how you can do it with a method reference:

private List<MyObject> getObjectList(List<OtherObject> objects) {
    return objects.stream()
        .map(this::map)
        .collect(Collectors.toList());
}

private MyObject map(OtherObject otherObject) {
    MyObject object = new MyObject();
    object.setId(otherObject.getId());
    object.setText(otherObject.getTextValue());
    object.setUserId(otherObject.getUserName());
    object.setCreatedDate(otherObject.getCreateDateTime());
    return object;
}

Here's how you can do it using streams and lambda expression:

private List<MyObject> getObjectList(List<OtherObject> objects) {
    return objects.stream()
        .map(obj -> {
        MyObject object = new MyObject();
        object.setId(obj.getId());
        object.setText(obj.getTextValue());
        object.setUserId(obj.getUserName());
        object.setCreatedDate(obj.getCreateDateTime());
        return object;
    }).collect(Collectors.toList());
}

edited Oct 5, 2020 at 16:19

answered Oct 5, 2020 at 15:50

George

3,0606 gold badges35 silver badges69 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

erickson Over a year ago

The method reference approach is much more readable; I'd encourage its use by showing it first.

iranichai Over a year ago

Thank you, very much. This is the perfect solution for my situation.

ollitietavainen · Accepted Answer · 2020-10-05 16:00:24Z

1

Here is an example:

    private List<MyObject> getObjectList(List<OtherObject> objects) {
        List<MyObject> resultList = new ArrayList<>();
        objects.forEach(obj -> {
            MyObject object = new MyObject();
            object.setId(obj.getId());
            object.setText(obj.getTextValue());
            object.setUserId(obj.getUserName());
            object.setCreatedDate(obj.getCreateDateTime());
            resultList.add(object);
        });
        return resultList;
    }

edited Oct 5, 2020 at 16:00

answered Oct 5, 2020 at 15:41

ollitietavainen

4,34016 silver badges36 bronze badges

3 Comments

George Over a year ago

.stream() is redundant if you're going to use the terminating operator forEach() anyways. objects.forEach( would suffice.

ollitietavainen Over a year ago

Edited to objects.forEach()

erickson Over a year ago

But still, mutating external state in a stream is considered an anti-pattern; that is better done via reduction, or in this case, the special form of reduction called collection. Here is some more information: developer.ibm.com/articles/j-java-streams-2-brian-goetz

Abra · Accepted Answer · 2020-10-05 16:26:07Z

Using Record

Equivalent of your OtherObject

import java.time.LocalDateTime;

public record OtherRec(int            id,
                       String         textValue,
                       String         userName,
                       LocalDateTime  createDateTime) {
}

Equivalent of your MyObject

import java.time.LocalDateTime;

public record MyRecord(int id,
                       String text,
                       String userId,
                       LocalDateTime createDate) {
}

Copying list of OtherRec to MyRecord

import java.time.LocalDateTime;
import java.util.List;
import java.util.stream.Collectors;

public class CopyList {

    public static void main(String[] args) {
        List<OtherRec> source = List.of(new OtherRec(1, "First", "One", LocalDateTime.of(2020, 10, 5, 12, 0)),
                                        new OtherRec(2, "Second", "Two", LocalDateTime.of(2020, 10, 5, 13, 0)));
        List<MyRecord> target = source.stream()
                                      .map(or -> new MyRecord(or.id(), or.textValue(), or.userName(), or.createDateTime()))
                                      .collect(Collectors.toList());
        System.out.println(target);
    }
}

Collectives™ on Stack Overflow

How to add a set of values from an ArrayList to another ArrayList using Lambda expression

3 Answers 3

2 Comments

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related