ArrayList循环返回对象成员变量值异常问题排查求助

嘿，我帮你梳理下实现这个需求时最容易踩的几个坑，以及对应的解决办法：

最常见的错误：目标对象被重复累加 #

很多人会先单独给目标对象加10，再遍历整个列表给所有对象加3，结果导致目标对象总共被加了13（10+3），完全不符合预期。比如下面这段错误代码：

// 错误示例：重复累加目标对象
public void updateRelationships(Person target, ArrayList<Person> personList) {
    // 先给目标对象加10
    target.setRelationship(target.getRelationship() + 10);
    // 遍历列表给所有对象加3，包括目标对象
    for (Person p : personList) {
        p.setRelationship(p.getRelationship() + 3);
    }
}

问题根源 #

目标对象本身就在personList里，遍历的时候会被再次累加3，最终得到的是+13而不是预期的+10。

另一个常见坑：对象引用判断错误 #

如果你传入的target和列表里的对象不是同一个引用（比如你新建了一个属性相同但内存地址不同的对象），而且你的Person类没有重写equals()方法，那么用p.equals(target)判断时会一直返回false，导致所有对象都只加3，目标对象根本没被特殊处理。比如：

// 错误示例：equals判断失效
public void updateRelationships(Person target, ArrayList<Person> personList) {
    for (Person p : personList) {
        if (p.equals(target)) { // Person未重写equals，仅比较引用地址
            p.setRelationship(p.getRelationship() + 10);
        } else {
            p.setRelationship(p.getRelationship() + 3);
        }
    }
}

// 测试时的错误用法：新建了一个和列表中对象属性相同的新对象
Person target = new Person("Alice", 50); // 列表里的Alice是另一个引用
updateRelationships(target, personList);

问题根源 #

默认的equals()方法比较的是对象的内存地址，而不是属性值。如果你的target不是列表里的那个引用，判断就会失败。

正确的实现方式 #

针对上面的问题，我们可以在遍历列表时逐个判断当前对象是否是目标对象，然后分别处理：

方案1：基于对象引用判断（推荐，效率高） #

如果你的target就是列表里的那个对象引用，直接用==判断即可：

public void updateRelationships(Person target, ArrayList<Person> personList) {
    for (Person p : personList) {
        if (p == target) {
            // 目标对象加10
            p.setRelationship(p.getRelationship() + 10);
        } else {
            // 其他对象加3
            p.setRelationship(p.getRelationship() + 3);
        }
    }
}

方案2：基于属性判断（如果无法保证引用相同） #

如果你的target是一个属性相同但引用不同的对象，需要先给Person类重写equals()和hashCode()方法，然后用equals()判断：

// 先给Person类重写equals和hashCode
public class Person {
    private String name;
    private int relationship;

    // 构造方法、getter、setter省略

    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;
        Person person = (Person) o;
        return relationship == person.relationship && Objects.equals(name, person.name);
    }

    @Override
    public int hashCode() {
        return Objects.hash(name, relationship);
    }
}

// 修正后的函数
public void updateRelationships(Person target, ArrayList<Person> personList) {
    for (Person p : personList) {
        if (p.equals(target)) {
            p.setRelationship(p.getRelationship() + 10);
        } else {
            p.setRelationship(p.getRelationship() + 3);
        }
    }
}

验证测试方法 #

可以写一个简单的测试来验证：

public static void main(String[] args) {
    ArrayList<Person> personList = new ArrayList<>();
    Person alice = new Person("Alice", 50);
    Person bob = new Person("Bob", 40);
    Person charlie = new Person("Charlie", 30);
    personList.add(alice);
    personList.add(bob);
    personList.add(charlie);

    // 调用函数，目标是Alice
    updateRelationships(alice, personList);

    // 预期输出：Alice:60, Bob:43, Charlie:33
    for (Person p : personList) {
        System.out.println(p.getName() + ": " + p.getRelationship());
    }
}

如果你的代码不是以上这些问题，可以把你的具体代码和输出结果贴出来，我再帮你精准定位！

内容的提问来源于stack exchange，提问作者SwampCrawford