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

return无效，因为我仍想继续循环。我想我可以使用continue声明可以使用，但看起来真的没有看到continue中的switch/case。

无论如何，单break中有两个case。有什么理由不这样做吗？

Answer 1

这当然只是一种意见，但我认为这本身并不好。但是，嵌套案例，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);
}

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

案件有多次休息是不好的做法吗？

1 个答案: