JavaScript不要在循环中创建函数并显示出圈复杂度。我无法绕过JSHint的错误信息。下面是我使用的循环:
var keys = document.querySelectorAll('#calculator span');
var operators = ['+', '-', 'x', '÷'];
var decimalAdded = false;
for (var i = 0; i < keys.length; i++) {
keys[i].onclick = function (e) {
var input = document.querySelector('.screen');
var inputVal = input.innerHTML;
var btnVal = this.innerHTML;
if (btnVal === 'C') {
input.innerHTML = '';
decimalAdded = false;
} else if (btnVal === '=') {
var equation = inputVal;
var lastChar = equation[equation.length - 1];
equation = equation.replace(/x/g, '*').replace(/÷/g, '/');
if (operators.indexOf(lastChar) > -1 || lastChar === '.'){
equation = equation.replace(/.$/, '');
}
if (equation){
input.innerHTML = (new Function('return ' + equation))();
}
decimalAdded = false;
} else if (operators.indexOf(btnVal) > -1) {
var lastChar = inputVal[inputVal.length - 1];
if (inputVal !== '' && operators.indexOf(lastChar) === -1){
input.innerHTML += btnVal;
}
else if (inputVal === '' && btnVal === '-'){
input.innerHTML += btnVal;
}
if (operators.indexOf(lastChar) > -1 && inputVal.length > 1) {
input.innerHTML = inputVal.replace(/.$/, btnVal);
}
decimalAdded = false;
} else if (btnVal === '.') {
if (!decimalAdded) {
input.innerHTML += btnVal;
decimalAdded = true;
}
} else {
input.innerHTML += btnVal;
}
e.preventDefault();
};
}
如何降低这个函数的圈复杂度为12
圈复杂度基本上是对通过函数的可能路径数量的度量。所以它把所有的循环和条件相乘。你有一个for()
循环,里面有一整堆嵌套的if()
和else
块,所以是的,这会给一个非常大的复杂性分数。
减少它的方法是将代码拆分为多个函数。每个函数都应该完成部分任务,因此复杂度得分较低。因为复杂度随着每个条件的增加而增加,所以将它们分开会得到几个总复杂度低得多的函数。
具体如何分解函数取决于你自己,但一般的建议是尽可能地分解它们;理想的函数是只做一件事,别无其他。