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其中，键-字段号和值-字段的值。我创建了此服务：

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接口。

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

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

编辑2：以上关于编写单元测试的建议是从我（模拟主义者）的角度来看的。古典的单元测试人员可能会编写一个测试来涵盖完整的规格（例如您在文章中提供的测试）。我不喜欢这样做的原因是，在这种集成样式的单元测试中，边缘案例更容易丢失，并且您对代码的工作方式做出错误的假设或者必须在代码中反复执行相同代码的机会增加了测试。但是，这是一个长期的辩论，有多种观点，各有千秋。

Java对象中更新字段的正确方法（设计模式）

1 个答案: