0

我正在使用 JSLint 使我的（工作）代码更简洁。

这是一个有用的工具，但它一点也不喜欢：

//...
switch (process.argv.length) {
    default:

    case 9:
        LOG = process.argv[8];
    case 8:
        EMERGENCY = process.argv[7];
    //...
        break;
    case 1:
    case 0:
        emergency('Error');
        break;
}

JSLint 说：

Missing 'case'.
Expected '}' to match 'switch' and instead saw 'case'
Expected '9' at column 5, not column 10.
Expected ';' and instead saw ':'
Expected ':' at column 5, not column 11
Unexpected ':'
Stopping.

全部在“案例 9：”行

如果用户提出太多参数，我将默认设置为第一个不阻止应用程序。

我可以把它放在最后并复制粘贴其中的所有传递，但我不喜欢这个想法，或者将所有这些都放在嵌套的 if 中，但这将是不可读的。

有没有办法让 JSLint 接受这个？

编辑：

这里是处理旧版本中开关盒角色的代码（接受较少的参数：

if (process.argv.length>=3){
    DOWNLOAD_DIR=PATH+process.argv[2];
    if(process.argv.length>=4){
        DOWNLOAD_ADD=process.argv[3];
        if (process.argv.length>=5)
        {
            SMIL_ADD=process.argv[4];
            if (process.argv.length>=6)
            {
                UPDATE_H=parseInt(process.argv[5],10);
                if (process.argv.length>=7)
                {
                    UPDATE_M=parseInt(process.argv[6],10);
                }
            }
        }
    }
}

正如你所看到的，这个开关比那些嵌套的 if 更具可读性，即使它让我以一种不寻常的方式使用默认值。

4

1 回答 1

1

你也需要break;在你的默认情况下有一个。

编辑：

对不起，我不太明白你的问题。首先，在您编辑的版本中，不需要嵌套每个 if 语句。此外，如果不满足这些条件，这些变量（例如，DOWNLOAD_DIR）等于什么？你不能做一些事情：

DOWNLOAD_DIR = (process.argv[2] !== undefined) ? PATH + process.argv[2] : '';
DOWNLOAD_ADD = (process.argv[3] !== undefined) ? process.argv[3] : '';
// etc, etc

然后你可以在最后添加：

LOG = (process.argv.length >= 9) ? process.argv[8] : '';

在您的情况下，听起来您想将这些空字符串设置为默认值，这意味着整个事情可以用很少的代码完成，例如：

var DOWNLOAD_DIR = (process.argv[2] !== undefined) ? PATH + process.argv[2] : '/downloads';
于 2013-09-12T13:55:30.617 回答