我在页面上有一组控件,大约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
,并且必须创建一个触发模糊事件的指令,但我想看看还有哪些其他想法。
答案 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();
});
});
}
}
})
将数组添加到父控制器:
...
$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中做这样的事情,但希望它至少会给你另一种方法来考虑(这就是你要求)。