javascript - 两个相似的 if 条件合二为一

Question

已关闭。此问题不符合 Stack Overflow guidelines 。目前不接受答案。

---

这个问题似乎与 help center 中定义的范围内的编程无关。 .

已关闭 9 年前。

Improve this question

在 Javascript 函数中，我有两个结构相似的条件。我想知道你是否可以将它们合二为一。该代码来 self 的随机加密安全数字/字符生成器。以下是 Java 脚本代码的片段:

function randomString(length, charset, allowRandomSourceFallback) {
    var i,
        result = "";

    // First we're going to try to use a built-in CSPRNG
    if (window.crypto && window.crypto.getRandomValues) {
        values = new Uint32Array(length);
        window.crypto.getRandomValues(values);

        for (i = 0; i < length; i++) {
            result += charset[values[i] % charset.length];
        }
    }
    // IE calls it msCrypto (built-in CSPRNG)
    else if (window.msCrypto && window.msCrypto.getRandomValues) {
        values = new Uint32Array(length);
        window.msCrypto.getRandomValues(values);

        for (i = 0; i < length; i++) {
            result += charset[values[i] % charset.length];
        }
    }
    // -> No built-in functionality -> use the function Math.random()
    else {
        for (i = 0; i < length; i++) {
            result += charset[Math.floor(Math.random() * charset.length)];
        }
    }
    return result;
}

Answer 1

为什么不检查可用的内容然后使用它:

var availableCrypto = window.crypto || window.msCrypto; // will get whatever is available

然后像这样使用它:

availableCrypto.getRandomValues(values);

给你:

function randomString(length, charset, allowRandomSourceFallback) {
    var i,
        result = "",
        availableCrypto = window.crypto || window.msCrypto;

    // First we're going to try to use a built-in CSPRNG
    if (availableCrypto && availableCrypto.getRandomValues) {
        values = new Uint32Array(length);
        availableCrypto.getRandomValues(values);

        for (i = 0; i < length; i++) {
            result += charset[values[i] % charset.length];
        }
    }
    // -> No built-in functionality -> use the function Math.random()
    else {
        for (i = 0; i < length; i++) {
            result += charset[Math.floor(Math.random() * charset.length)];
        }
    }
    return result;
}