降低圈复杂度，多条if语句

Question

我有以下代码：

private Facility updateFacility(Facility newFacility, Facility oldFacility) {
    if (newFacility.getCity() != null)
        oldFacility.setCity(newFacility.getCity());
    if (newFacility.getContactEmail() != null) 
        oldFacility.setContactEmail(newFacility.getContactEmail());
    if (newFacility.getContactFax() != null) 
        oldFacility.setContactFax(newFacility.getContactFax());
    if (newFacility.getContactName() != null) 
        oldFacility.setContactName(newFacility.getContactName());
    // ......
}

大约有 14 个这样的检查和分配。也就是说，除了少数，我需要修改 oldFacility 对象的所有字段。我得到了这个代码 14 的圈复杂度，根据 SonarQube，这是“授权大于 10”。关于如何降低圈复杂度的任何想法？

Answer 1

在您的程序中的某个时刻，您必须实现以下逻辑：

• 如果新设施定义了属性，则相应地更新旧设施
• 如果不是，请不要覆盖旧设施的先前值。

无需全局查看您的项目，您可以做的是将该逻辑移动到每个属性的 setter 中：

public class Facility {

    public void setSomething(String something) {
        if (something != null) {
            this.something = something;
        }
    }

}

这样，您的update方法就是：

private Facility updateFacility(Facility newFacility, Facility oldFacility) {
    oldFacility.setSomething(newFacility.getSomething());
    // etc for the rest
}