一个案例多次中断是不好的做法吗？

Question

这是我正在做的事情的超级简化版本，但想象一下需要对 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;
  }
}

Areturn不起作用，因为我仍然想继续循环。我想我可以使用一个continue可以工作的语句，但是continue在switch/case.

无论如何，两个breaks 在一个case. 有什么理由不这样做？

Answer 1

这当然只是一种意见，但我认为它本身并不坏。但是，嵌套的 case、ifs 和其他控制结构不会使您的代码非常易读。在这种情况下，您可能会考虑将代码段移动到单独的函数中。

所以你的代码可能是这样的：

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);
}

当然，这些只是建议。在这种情况下很难做出明智的例子，没有冒犯。;)