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

Question

我有java类 Person :

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

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

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 中的字段。并验证它。

Answer 1

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

将每个字段建模为单独的类，实现通用的 Field 接口(interface)。

通过构造函数注入(inject)所有已知字段，并在收到映射时，为每个条目查找匹配的字段类以处理验证和更新。

通过这种方法，您可以分别测试每个字段的验证和更新逻辑。添加人员字段不会使您的服务等级增长。测试 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 中使用依赖注入(inject)。 Spring 将自动收集实现 Field 接口(interface)的可用组件，并通过构造函数注入(inject)。在服务的单元测试中，您可以注入(inject)一两个模拟字段。不要在服务测试中测试字段实现的实际验证/更新逻辑，而是为每个字段类创建单独的单元测试。

编辑2:以上关于编写单元测试的建议是从我(mockist)的角度来看的。经典的单元测试人员可能会编写一个测试来覆盖完整的规范(就像您在帖子中提供的测试)。我不喜欢这样做的原因是，在这种集成式单元测试中，边缘情况更容易丢失，并且您对代码如何工作做出错误假设的机会增加，或者您必须在您的代码中重复执行相同的代码。测试。然而，这是一个长期的争论，有多种观点，但都有各自的优点。