保持我的代码DRY为`$ onInit`和`$ onChanges`方法

Posted 2021-03-27

我有一段代码在几乎所有组件中重复：

import {deviceCommands} from "../../../core/data/Commands";


let _generalProperties = {
    'deviceName': deviceCommands.NAME.key,
    'deviceModel': deviceCommands.MODEL.key
};

export default class GeneralDeviceSettingsCtrl {
    constructor($scope) {
        let $ctrl = this;
        $ctrl.$onChanges = function (changes) {
            for(let prop in _generalProperties) {
                $ctrl[prop] = $ctrl.test.vm.data[_generalProperties[prop]];
            }
        };

        $ctrl.$onInit = function () {
            for(let prop in _generalProperties) {
                $scope.$watch(() => $ctrl.test.vm.data[_generalProperties[prop]],
                    (newValue, oldValue) => {
                        if (newValue !== oldValue) {
                            $ctrl[prop] = $ctrl.test.vm.data[_generalProperties[prop]];
                        }
                    });
            }
        };
    }
}

唯一不同的是_generalProperties变量，这是我的观点所特有的。

我怎样才能让它干涸？做一个基类？用装饰器？

答案

我认为有很多不同的方法，但如果你坚持使用类和继承，你可以将_generalProperties提供给你的父构造函数并重用整个实现。

例如：

export class BaseSettingsComponent {
    constructor(properties){
        this._properties = properties;
    }

    $onInit(){ /* ... implement base component stuff*/ }
}

然后，在每个使用相同组件逻辑的页面上，您可以扩展基类，向父类提供属性并让基类完成工作。

import { BaseSettingsComponent } from '../common/base-settings-component.js';

let _generalProperties = {
    'deviceName': deviceCommands.NAME.key,
    'deviceModel': deviceCommands.MODEL.key
};

export class GeneralDeviceSettingsCtrl extends BaseSettingsComponent {
    constructor(){
        super(_generalProperties);
    }

    $onInit(){ super.$onInit(); /* ... implement base component stuff*/ }
}

使用这种方法时要记住的一件重要事情是，如果你需要在子类上实现$onInit或$onChanges，你必须调用super.$onInit()，否则你会因为覆盖而丢失父行为。

最后，为了更清晰的代码，您还可以丢弃_generalProperties的声明，直接将其提供给super构造函数。

import { BaseSettingsComponent } from '../common/base-settings-component.js';

export class GeneralDeviceSettingsCtrl extends BaseSettingsComponent {
    constructor(){
        super({
            'deviceName': deviceCommands.NAME.key,
            'deviceModel': deviceCommands.MODEL.key
        });
    }
}