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