java 对象中更新字段的正确方法(设计模式)

Correct approach(design pattern) for update fields in java object

我有 java class Person:

public class Person {
    private String name;
    private int age;
    private String marriedStatus;
    private Date dob;
    //getters and setters
}

当我获得此对象的某些字段的新值时,我可以更新它。但是新字段以这种格式计算收入:Map<String, String> newValues 其中键 - 字段编号和值 - 字段的值。我创建了这个服务:

public class UpdateService {
    public Person updateFields(Person targetPerson, Map<String, String> newValues){
        return null;
    }
}

我创建了一个单元测试,请你帮助实现它。

public class UpdateServiceTest {

    /*associations between field number and field name
    12 - name (length min: 2, max: 20. First letter must uppercase )
    18 - marriedStatus (only married, divorced, single)
    21  - age (only between 18 and 120)
    14  - dob (some format)
     */

    private Date dob;

    @Before
    public void setUp() {
        dob = new GregorianCalendar(2000, Calendar.NOVEMBER, 20).getTime();
    }

    @Test
    public void returnPersonWithUpdatedFields() {
        UpdateService updateService = new UpdateService();

        Person targetPerson = new Person();
        targetPerson.setName("Name");
        targetPerson.setMarriedStatus("MarriedStatus");
        targetPerson.setAge(20);
        targetPerson.setDob(dob);

        Map<String, String> newValues = new HashMap<String, String>();
        newValues.put("12", "Bill");
        newValues.put("18", "married ");
        newValues.put("21", "25");

        Person person = updateService.updateFields(targetPerson, newValues);

        assertEquals("Bill", person.getName());
        assertEquals("married", person.getMarriedStatus());
        assertEquals(25, person.getAge());
        assertEquals(dob, person.getDob());
    }
}

我需要找人并只更新收入在 Map<String, String> newValues 中的字段。并验证它。

这是您如何执行此操作的建议。

将每个字段建模为单独的 class,实现通用 Field 接口。

通过构造函数注入所有已知字段,当收到映射时,为每个条目查找匹配字段class以处理验证和更新。

通过这种方法,您可以分别测试每个字段的验证和更新逻辑。添加人员字段不会使您的服务 class 增长。测试 PersonUpdateService 只需要使用一两个模拟字段来验证查找和执行逻辑。我想说的是一个很好的关注点分离。

import java.util.*;

@Component
public class PersonUpdateService {

    private final List<Field> fields;

    @Autowired
    public PersonUpdateService(final List<Field> fields) {
        this.fields = fields;
    }

    public void updatePerson(final Person person, final Map<String, String> update) {
        final boolean updated = false;
        update.forEach((key, value) -> this.findField(key).update(person, value));
    }

    private Field findField(final String index) {
        return this.fields.stream().filter(f -> f.index().equals(index)).findAny().orElseThrow(
                () -> new IllegalArgumentException("Field not found: " + index));
    }

}

现场接口:

public interface Field {
    String index();

    void update(Person person, String newValue);
}

示例字段实现:

import java.util.regex.Pattern;

@Component
public class NameField implements Field {

    private static final String INDEX = "12";
    private static final String REGEX = "/^[A-Z][a-z0-9_-]{1,19}$/";
    private static final String CONSTRAINTS = "length min: 2, max: 20. First letter must uppercase";

    @Override
    public String index() {
        return INDEX;
    }

    @Override
    public void update(final Person person, final String newValue) {
        if (!Pattern.matches(REGEX, newValue)) {
            throw new ValidationException(CONSTRAINTS);
        }
        person.setName(newValue);
    }

}

编辑:添加了@Component 和@Autowired 注释以指示如何在Spring 中使用依赖注入。实现 Field 接口的可用组件将由 Spring 自动收集并通过构造函数注入。在服务的单元测试中,您可以注入一两个模拟字段。不要在服务测试中测试字段实现的实际 validation/updating 逻辑,而是为每个字段创建单独的单元测试 class.

编辑 2:以上关于编写单元测试的建议是从我(模仿者)的角度出发的。传统的单元测试人员可能会编写一个测试来覆盖完整的规范(就像您在 post 中提供的测试)。我不喜欢这样做的原因是边缘情况在这样的 integration-style 单元测试中更容易丢失,并且您对代码如何工作做出错误假设或您必须重复执行相同代码的可能性增加在你的测试中。然而,这是一场 long-standing 辩论,存在多种观点,各有千秋。