jslint工具的一个提示是:
++和——
++(递增)和——(递减)
众所周知,操作符会导致糟糕的代码
鼓励过度狡诈。他们
仅次于有缺陷的架构
使病毒和其他
安全威胁。这是一个加分项
选项,禁止使用这些
操作符。
我知道PHP结构像$foo[$bar++]可能很容易导致off-by-one错误,但我想不出一个更好的方法来控制循环比:
while( a < 10 ) do { /* foo */ a++; }
or
for (var i=0; i<10; i++) { /* foo */ }
jslint突出显示它们是因为有一些类似的语言缺少“++”和“——”语法或处理方式不同,还是有其他避免“++”和“——”的理由,我可能忽略了?
如果你读过JavaScript The Good Parts,你会发现Crockford在for循环中替换i++的是i+=1(而不是i=i+1)。这是非常干净和可读的,并且不太可能变成“棘手”的东西。
Crockford在jsLint中设置了禁止自增和自减的选项。你可以选择是否听从建议。
我个人的原则是不要做任何与自增或自减相结合的事情。
我从多年的C语言经验中了解到,如果我简单地使用它,就不会出现缓冲区溢出(或数组下标越界)。但我发现,如果我陷入了在同一语句中执行其他操作的“过度棘手”实践,我确实会遇到缓冲区溢出。
因此,对于我自己的规则来说,使用i++作为for循环中的增量是可以的。
在作为前缀和后缀使用时,操作符的含义不同,这可能会导致难以发现的错误。考虑下面的例子,使用bubbleSort:
function bubbleSort(array) {
if(array.length === 1) return array;
let end = array.length - 2;
do {
for (let i = 0; i < array.length; i += 1) {
if (array[i] > array[i + 1]) {
swap(array, i, i + 1);
}
}
} while (end--);
}
bubbleSort([6,5]);
让我们想象一下,在运行程序的过程中,我们将一个包含两项的值传递给排序函数。代码按原样运行很好:“do/while”循环在达到条件之前首先执行。然而,程序认为结束是假的,并在变量递减之前退出循环。
现在考虑下面的代码,其中——符号用作前缀,而不是后缀。这段代码将进入一个无限循环:
function bubbleSort(array) {
if(array.length === 1) return array;
let end = array.length - 2;
do {
for (let i = 0; i < array.length; i += 1) {
if (array[i] > array[i + 1]) {
swap(array, i, i + 1);
}
}
} while (--end);
}
bubbleSort([6,5]);
现在,当我们达到while条件时,我们在检查它之前递减结束值。返回-1,这在Javascript中是一个真值。
我对它们的使用方式没有强烈的意见,但我只是想说明,如果不小心使用它们,它们会导致真正的错误。
我认为在两种情况下应该避免使用:
1)当你有一个在很多行中使用的变量,你在使用它的第一个语句(或者最后,或者更糟糕的是,在中间)上增加/减少它:
// It's Java, but applies to Js too
vi = list.get ( ++i );
vi1 = list.get ( i + 1 )
out.println ( "Processing values: " + vi + ", " + vi1 )
if ( i < list.size () - 1 ) ...
在这样的例子中,您很容易忽略变量是自动递增/递减的,甚至会删除第一个语句。换句话说,只在非常短的块中使用它,或者变量只出现在块中的几个结束语句中。
2)在同一语句中出现多个关于同一个变量的++和-的情况。很难记住在这样的情况下会发生什么:
result = ( ++x - --x ) * x++;
考试和专业测试会询问类似上述的例子,事实上,我在寻找其中一个的文档时偶然发现了这个问题,但在现实生活中,人们不应该被迫对一行代码考虑这么多。