编程语言
首页 > 编程语言> > javascript-一个案例有多个中断是不好的做法吗?

javascript-一个案例有多个中断是不好的做法吗?

作者:互联网

这是我正在做的事情的超级笨拙版本,但请想象需要在switch语句中对一个对象执行多次验证.在验证的任何时候,我们可能会决定我们不想继续使用该对象.

愚蠢的例子:

var yummies = [
  {name: 'apples', quantity: 3},
  {name: 'apples', quantity: 4}
];

var purchases = [];

for (var i = 0; i < yummies.length; i++) {
  var yummy = yummies[i];
  switch (yummy.name) {
      case 'apples':
        if (purchases.indexOf(yummy.name) > -1) {
          break;
        }

        //A ton of other code including, possibly other breaks

        purchases.push(yummy.name);
        break;
  }
}

由于我仍然想继续执行循环,因此返回无效.我认为我可以使用continue语句可以工作,但是看起来确实看不到开关/案例中的continue.

无论如何,在一个案例中有两个中断.有什么理由不这样做吗?

解决方法:

当然,这只是一个意见,但我认为这本身并不坏.但是,嵌套的case,if和其他控制结构不会使您的代码可读性强.在这种情况下,您可能会考虑将代码片段移至单独的函数.

因此您的代码可能如下所示:

function purchaseApple(yummyName) {
  if (purchases.indexOf(yummyName) == -1) {
    purchases.push(yummyName);
  }
}

for (var i = 0; i < yummies.length; i++) {
  var yummy = yummies[i];
  switch (yummy.name) {
      case 'apples':
        purchaseApples(yummy.name);
        break;
  }
}

或者甚至也可以将条件移至函数:

function tryPurchaseApple(yummyName) {
  if (yummyName == 'apples' && purchases.indexOf(yummyName) == -1) {
    purchases.push(yummyName);
  }
}

for (var i = 0; i < yummies.length; i++) {
  var yummy = yummies[i];
  tryPurchaseApple(yummy.name);
  tryPurchasePears(yummy.name);
}

当然,这些只是建议.很难在这种情况下做出明智的榜样,没有违法行为.

标签:loops,switch-statement,javascript
来源: https://codeday.me/bug/20191027/1947698.html