对于具有固定字段数的对象，最好使用一个setter方法或多个方法？

Question

我有一个带有类型为object的成员变量的类。 该对象具有固定数量的字段。 我试图决定是否应该使用一个或多个设置函数来改变这些字段。

为了使问题更具体，我编写了以下课程，以两种不同方式对简单的组织管理结构进行建模：

1. 多种设定功能

    class Management { constructor() { this.numberOfManagers = 100; this.salaryDetails = { regionalManagerSalary: 80000, stateManagerSalary: 110000, executiveManagerSalary: 200000 }; } setRegionalManagerSalary(salary) { this.salaryDetails.regionalManagerSalary = salary; } setStateManagerSalary(salary) { this.salaryDetails.stateManagerSalary = salary; } setExecutiveManagerSalary(salary) { this.salaryDetails.executiveManagerSalary = salary; } } const management = new Management(); management.setRegionalManagerSalary(100000); management.setStateManagerSalary(120000); management.setExecutiveManagerSalary(210000); 

2. 一个设定器功能

    class Management { constructor() { this.numberOfManagers = 100; this.salaryDetails = { regionalManagerSalary: 80000, stateManagerSalary: 110000, executiveManagerSalary: 200000 }; } setManagerSalary(typeOfManagerSalary, salary) { this.salaryDetails[typeOfManagerSalary] = salary; } } const management = new Management(); management.setManagerSalary('regionalManagerSalary', 100000); management.setManagerSalary('stateManagerSalary', 120000); management.setManagerSalary('executiveManagerSalary', 210000); 

实施1.会更好还是实施2.会更好？

Answer 1

我的建议是有一个方法说setSalaryDetails，它将薪水对象作为输入，您可以直接设置。 就像是 -

setSalaryDetails（salaryDetails）{this.salaryDetails =工资细节; }

调用此方法时，可以使用将传递给它们的那些字段集来适当地创建该对象。

Answer 2

如果不会在很多可写性之间进行权衡，我将始终考虑更高的可读性方法。 第二个更通用，但是第一个更清晰。

所以我认为第一个比较好，因为更清楚的是， salaryDetails中只有3个字段。 只是考虑一下您不是作者，当您看到第二个问题时，您不知道salaryDetails会有多少个字段，因为有人可以打电话给management.setManagerSalary('someNewSalary', 100000); 在某个地方添加一些新字段。

除非您有多个管理对象，并且每个对象可能在salaryDetails下都有一些特定字段， salaryDetails我认为第二种方法不会更好。

Answer 3

最好完全没有二传手。 初始化构造函数中的所有必需属性（让构造函数为此提供参数）。

您发布的代码不是OOP，而是伪装的过程代码。 如果要使其成为OOP，则代码必须保留在数据所在的位置，即在类中。

作为第一步改进，您的代码应如下所示：

class Management {
    constructor(nb, regionalSalary, stateSalary, executiveSalary) {
        this.numberOfManagers = nb;
        this.salaryDetails = {
            regionalManagerSalary: regionalSalary,
            stateManagerSalary: stateSalary,
            executiveManagerSalary: executiveSalary
        };
    }
}

const management = new Management(100, 100000, 120000, 210000);

更短，更干净，更不易出错。

您发布的片段不足以告诉您要实现的目标，但是构造函数的新签名告诉我，您将太多职责混合到一个类中。

较干净的设计是对每种管理器类型使用单独的类：

/* abstract */ class Manager {
    constructor(salary) {
        this.salary = salary;
    }
}

class RegionalManager extends Manager { }
class StateManager extends Manager { }
class ExecutiveManager extends Manager { }

class Management {
    constructor(nb, regionalManager, stateManger, executiveManager) {
        this.numberOfManagers = nb;
        this.managers = {
            regionalManager: regionaManager,
            stateManager: stateManager,
            executiveManager: executiveManager
        };
    }
}

const management = new Management(
    100,
    new RegionalManager(100000),
    new StateManager(120000),
    new ExecutiveManager(210000)
);

现在，您可以在Class Management中实现类Manager所有管理器类型所共有的逻辑，该类中每种管理器类型所特有的逻辑，以及按数字处理管理器而无需关心它们各自特征的逻辑。 这样，代码使用的if和switch条件更少，更易于阅读，理解和进一步开发。