减少AngularJS中已评估ng禁用的事件触发器

时间:2014-04-30 17:22:15

标签: javascript angularjs

使用案例

我在页面上有一组控件,大约60多个。每个控件都会根据表单本身输入的数据来计算字符串表达式,以决定是否禁用该控件。表达式由最终用户创建,因此可以是任何内容。

我遇到的问题是表单和评估相当复杂,并且会降低UI的速度。最终代码将具有需要在范围对象中查找数据并且将包含循环等的表达式...

示范

这是jsfiddle来模拟我想要改进的内容。它不是最终代码,因为该模块相当大。但是,这应该是我想要实现(或减少)的图片。这个演示很快,因为表达式非常简单,但我希望能够减少调用函数is的次数。

[ edit ]我已经更新了演示,尽可能多地包含最终生产代码的核心逻辑。这与即将完成的工作一样接近。

模板

<div ng-app ng-controller="ctrl">
    <div ng-repeat="widget in widgets">
        <label>Widget #{{$index}}
            <input type="text" ng-model="widget.value" 
                   ng-disabled="is(widget.expression)" />
        </label>
    </div>
</div>

控制器

var ctrl = function ($scope) {
    $scope.widgets = [];

    /** Core Logic Below Here **/
    var getValueByName = function (name) {
        for (i = 0, k = $scope.widgets.length; i < k; i++) {
            if ($scope.widgets[i].name == name) {
                return $scope.widgets[i].value;
            }
        }
        return 0;
    };

    var replaceNameWithValue = function () {
        var result = this.replace(/( |^)(widget_\d+?)( |$)/g, function ($0, $1, $2, $3) {
            return parseInt(getValueByName($2), 10);
        });
        return result;
    };

    $scope.is = function (expression) {
        console.log('firing');
        try {
            var exp = replaceNameWithValue.call(expression);
            return $scope.$eval(exp);
        } catch (e) {}
        return false;
    };

    /** Logic to generate random test cases **/
    var numWidgets = 60;
    var operators = ['+', '-', '*'];
    var comparators = ['===', '!==', '>', '<', '>=', '<='];

    var getRandomInt = function (max) {
        return Math.floor(Math.random() * max);
    };

    var makeExpression = function () {
        var a = getRandomInt(numWidgets);
        var b = getRandomInt(numWidgets);
        var c = getRandomInt(20);
        var op = operators[getRandomInt(operators.length)];
        var cf = comparators[getRandomInt(comparators.length)];

        return {
            expression: ['widget_' + a, op, 'widget_' + b, cf, c].join(' ')
        };
    };

    var makeRandomObject = function (index) {
        var obj = makeExpression();
        obj.name = "widget_" + index;
        obj.value = 0;
        return obj;
    };

    var preLoad = function (numWidgets) {
        for (i = 0; i < numWidgets; i++) {
            $scope.widgets.push(makeRandomObject(i));
        }
    };

    preLoad(numWidgets);
};

问题

如何提高这些评估的效果?

解?

我注意到评估是在keyUp事件上执行的。我怀疑我无法使用原生ng-disabled,并且必须创建一个触发模糊事件的指令,但我想看看还有哪些其他想法。

1 个答案:

答案 0 :(得分:1)

每次Angular运行摘要周期时,您都可以继续使用ngDisabled,而不是为每个小部件调用一个函数,方法是让它评估一个仅在指令的链接函数指定的某些条件下发生变化的简单值。

这是modified Fiddle。要做到这一点,你需要:

添加属性限制指令,该指令使用以下行为装饰元素:

  • 在父控制器中注册观察者
  • 绑定元素上的侦听器,触发时会调用检查所有表达式的函数

...

directive('widgetWatcher', function($timeout){
    return {
        restrict: 'A',
        require: 'ngModel',
        scope: true,
        link: function(scope, elm, attr) {          

            // register observer for this directive by passing its scope
            scope.addExpressionObserver(scope);

            // Check expressions at most once every 1s or immediately when input is blurred
            var debounce;
            elm.bind('input', function() {
                $timeout.cancel(debounce);
                debounce = $timeout( function() {
                    scope.$apply(function() {
                        scope.checkExpressions();
                    });
                }, 1000);
            });
            elm.bind('blur', function() {
                scope.$apply(function() {
                    scope.checkExpressions();
                });
            });

        }
    }
})

将数组添加到父控制器:

  • 向数组添加观察者的函数(每个指令的范围)
  • 另一个迭代集合,评估每个表达式并为每个小部件设置isDisabled范围变量......这由输入或模糊发生的指令调用

...

$scope.expressionObservers = [];

$scope.addExpressionObservers = function(dScope) {
    $scope.expressionObservers.push(dScope);
};

$scope.checkExpressions = function() {
    angular.forEach($scope.expressionWatchers, function(val){
        val.isDisabled = $scope.is(val.widget.expression);
    });
};

不知道这是否解决了你所有的问题,或者是否有更好的方法在Angular中做这样的事情,但希望它至少会给你另一种方法来考虑(这就是你要求)。