JavaScript样式/优化:String.indexOf()v.Regex.test()

Question

我最近遇到过这段JavaScript代码:

if (",>=,<=,<>,".indexOf("," + sCompOp + ",") != -1)

我很好奇,因为写这个测试我会做的:

if (/(>=|<=|<>)/.test(sCompOp))

这只是一种风格差异,还是其他代码的作者对优化有所了解,我不知道？或者也许有不同的理由这样做,或者不使用正则表达式......？

在我看来,使用它String.indexOf()有点难以阅读(但是,我对正则表达式很满意),但是有没有比写同等正则表达式"更好"的情况？

通过"更好"可能更快或更有效,(虽然显然取决于浏览器的JavaScript引擎),或者其他一些我不知道的原因.任何人都可以开导我吗？

Answer 1

我做了一些测试.第一种方法是稍快,但还不足以让任何真正的差别甚至在大量使用... 除非时候sCompOp可能是一个很长的字符串.因为第一种方法搜索固定长度的字符串,所以无论多长时间,它的执行时间都非常稳定sCompOp,而第二种方法可能会遍历整个长度sCompOp.

此外,第二种方法可能会匹配无效的字符串 - "blah blah blah <= blah blah"满足测试......

鉴于你可能正在做其他地方解析运算符的工作,我怀疑边缘情况是否会成为一个问题.但即使不是这种情况,对表达式的一个小修改也会解决这两个问题:

/^(>=|<=|<>)$/

测试代码:

function Time(fn, iter)
{
   var start = new Date();
   for (var i=0; i<iter; ++i)
      fn();
   var end = new Date();
   console.log(fn.toString().replace(/[\r|\n]/g, ' '), "\n : " + (end-start));
}

function IndexMethod(op)
{
   return (",>=,<=,<>,".indexOf("," + op + ",") != -1);
}

function RegexMethod(op)
{
   return /(>=|<=|<>)/.test(op);
}

function timeTests()
{
   var loopCount = 50000;

   Time(function(){IndexMethod(">=");}, loopCount);
   Time(function(){IndexMethod("<=");}, loopCount);
   Time(function(){IndexMethod("<>");}, loopCount);
   Time(function(){IndexMethod("!!");}, loopCount);
   Time(function(){IndexMethod("the quick brown foxes jumped over the lazy dogs");}, loopCount);
   Time(function(){IndexMethod("<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<");}, loopCount);

   Time(function(){RegexMethod(">=");}, loopCount);
   Time(function(){RegexMethod("<=");}, loopCount);
   Time(function(){RegexMethod("<>");}, loopCount);
   Time(function(){RegexMethod("!!");}, loopCount);
   Time(function(){RegexMethod("the quick brown foxes jumped over the lazy dogs");}, loopCount);
   Time(function(){RegexMethod("<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<");}, loopCount);
}

timeTests();

测试在IE6,FF3,Chrome 0.2.149.30