java - 从服务中获取错误的输出

Question

我正在研究逻辑，其中我必须使用 PersonDto 列表，并根据中存在的规则值(私有(private)字符串规则；)应用规则人 Dto.

我有一个名为Rules的接口(interface)，它将有多个实现。但对于这个问题，我只添加了 1 个名为 Rule1 的实现。

我还有一个 RuleMapper 类，负责按需提供 Bean。

问题:根据 PersonDto 中存在的规则值，我正在应用规则。在 Rule1 类中，我有一个修改 id 字段的逻辑。我正在设置新的id，我将其作为参数获取。
最后，我将结果存储到 ArrayList 中。
但在 ArrayList 中，所有 PersonDto 的值都作为我在应用规则时传递的最后一个 id 值。

例如:

 List<String> ids = Arrays.asList("10001", "100002");
    List<PersonDto> result = new ArrayList<PersonDto>();

    persons.stream().forEach(person -> {
        ids.stream().forEach(id -> {
            System.out.println(ruleMapper.getRule(person.getRule()).applyRule(person, id));

            result.add(ruleMapper.getRule(person.getRule()).applyRule(person, id));

        });
    });

正如您在上面的代码片段中看到的，有两个 id 10001 和 10002，但是当我存储然后打印结果时，我得到所有元素中 id 值为 10002 的 PersonDto。但是当我在循环内执行 System.out.println() 时，我会看到正确的结果。

如需了解更多信息或重现问题，请引用下面的完整代码:

PersonDto

package com.example.dto;

public class PersonDto {
protected int id;
protected String name;
private String rule;

public int getId() {
    return id;
}

public void setId(int id) {
    this.id = id;
}

public String getName() {
    return name;
}

public void setName(String name) {
    this.name = name;
}



public String getRule() {
    return rule;
}

public void setRule(String rule) {
    this.rule = rule;
}

@Override
public String toString() {
    return "PersonDto [id=" + id + ", name=" + name + ", rule=" + rule + "]";
}

}

规则界面

package com.example.service;

import com.example.dto.PersonDto;

public interface Rules {

public PersonDto applyRule(PersonDto input, String newId);

}

规则 1 实现

package com.example.service;

import org.springframework.stereotype.Service;

import com.example.dto.PersonDto;

@Service
public class Rule1 implements Rules {

@Override
public PersonDto applyRule(PersonDto input, String newIdt) {
    input.setId(Integer.parseInt(newIdt));
    return input;
}

}

规则映射器

package com.example.service;

import java.util.Map;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;

@Service

公共(public)类规则映射器{

@Autowired
private Map<String, Rules> beans;

public Rules getRule(String ruleName) {
    return beans.get(ruleName);
}

public Map<String, Rules> gelAllBeans() {
    return beans;
  }
}

Controller

package com.example.demo;

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.Random;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.RestController;

import com.example.dto.PersonDto;
import com.example.service.RuleMapper;

@RestController
public class StudentContoller {

@Autowired
private RuleMapper ruleMapper;

@GetMapping(value = "/test")
public void saveStudent() throws Exception {
    List<String> orders = Arrays.asList("order 1", "order 2");

    List<PersonDto> persons = new ArrayList<PersonDto>();

    for (int i = 0; i < 10; i++) {
        PersonDto per = new PersonDto();
        per.setId(i);
        per.setName("John Doe_ ".concat(String.valueOf(i)));
        per.setRule("rule" + getRandomRule());
        persons.add(per);
    }


    List<String> ids = Arrays.asList("10001", "100002");
    List<PersonDto> result = new ArrayList<PersonDto>();

    persons.stream().forEach(person -> {
        ids.stream().forEach(id -> {
            System.out.println(ruleMapper.getRule(person.getRule()).applyRule(person, id));

            result.add(ruleMapper.getRule(person.getRule()).applyRule(person, id));

        });
    });


    for (PersonDto person : result) {
        System.out.println(person);
    }

}

private int getRandomRule() {
    Random r = new Random();
    int low = 1;
    int high = 2;
    int result = r.nextInt(high - low) + low;

    return result;
 }

}

预期输出

PersonDto [id=10001, name=John Doe_ 0, rule=rule1]
PersonDto [id=100002, name=John Doe_ 0, rule=rule1]
PersonDto [id=10001, name=John Doe_ 1, rule=rule1]
PersonDto [id=100002, name=John Doe_ 1, rule=rule1]
PersonDto [id=10001, name=John Doe_ 2, rule=rule1]
PersonDto [id=100002, name=John Doe_ 2, rule=rule1]
PersonDto [id=10001, name=John Doe_ 3, rule=rule1]
PersonDto [id=100002, name=John Doe_ 3, rule=rule1]
PersonDto [id=10001, name=John Doe_ 4, rule=rule1]
PersonDto [id=100002, name=John Doe_ 4, rule=rule1]
PersonDto [id=10001, name=John Doe_ 5, rule=rule1]
PersonDto [id=100002, name=John Doe_ 5, rule=rule1]
PersonDto [id=10001, name=John Doe_ 6, rule=rule1]
PersonDto [id=100002, name=John Doe_ 6, rule=rule1]
PersonDto [id=10001, name=John Doe_ 7, rule=rule1]
PersonDto [id=100002, name=John Doe_ 7, rule=rule1]
PersonDto [id=10001, name=John Doe_ 8, rule=rule1]
PersonDto [id=100002, name=John Doe_ 8, rule=rule1]
PersonDto [id=10001, name=John Doe_ 9, rule=rule1]
PersonDto [id=100002, name=John Doe_ 9, rule=rule1]

实际输出

PersonDto [id=100002, name=John Doe_ 0, rule=rule1]
PersonDto [id=100002, name=John Doe_ 0, rule=rule1]
PersonDto [id=100002, name=John Doe_ 1, rule=rule1]
PersonDto [id=100002, name=John Doe_ 1, rule=rule1]
PersonDto [id=100002, name=John Doe_ 2, rule=rule1]
PersonDto [id=100002, name=John Doe_ 2, rule=rule1]
PersonDto [id=100002, name=John Doe_ 3, rule=rule1]
PersonDto [id=100002, name=John Doe_ 3, rule=rule1]
PersonDto [id=100002, name=John Doe_ 4, rule=rule1]
PersonDto [id=100002, name=John Doe_ 4, rule=rule1]
PersonDto [id=100002, name=John Doe_ 5, rule=rule1]
PersonDto [id=100002, name=John Doe_ 5, rule=rule1]
PersonDto [id=100002, name=John Doe_ 6, rule=rule1]
PersonDto [id=100002, name=John Doe_ 6, rule=rule1]
PersonDto [id=100002, name=John Doe_ 7, rule=rule1]
PersonDto [id=100002, name=John Doe_ 7, rule=rule1]
PersonDto [id=100002, name=John Doe_ 8, rule=rule1]
PersonDto [id=100002, name=John Doe_ 8, rule=rule1]
PersonDto [id=100002, name=John Doe_ 9, rule=rule1]
PersonDto [id=100002, name=John Doe_ 9, rule=rule1]

Answer 1

persons.stream().forEach(person -> {
    ids.stream().forEach(id -> {
        result.add(ruleMapper.getRule(person.getRule()).applyRule(person, id));
    });
});

这会循环所有人。对于每个人，它都会循环遍历所有 ID。对于每个 ID，您将规则应用于该人。但由于该规则是修改该人的 ID，因此最后一个 ID 是存储在该人中的 ID。

如果重写为for循环，则相当于

for (Person person: persons) {
    for (String id: ids) {
        person.setId(id);
    }
}

我不确定代码应该做什么。也许第一个 ID 应该设置在第一个人上，第二个 ID 设置在第二个人上，依此类推。如果是这种情况，则循环遍历其中一个列表的索引，并获取每个索引的人和 ID。 (并祈祷两个列表具有相同的大小):

for (int i = 0; i < persons.size(); i++) {
    persons.get(i).setIf(ids.get(i));
}