如何在JavaScript中使用函数内部声明的变量？

问题根源 #

1. 你函数里的result是局部变量，只在pswrd_gnrtr函数内部有效，外部声明的同名变量和它完全是两个独立的变量，所以外部拿不到函数内的结果。
2. 原代码里charAt的索引计算有错误：Math.floor((Math.random() * values.length)+1)+1会生成超出values字符串长度的索引，导致返回空字符，生成的密码会有缺失。

推荐解决方案：让函数返回生成的密码 #

修改函数，去掉alert，改用return返回生成的密码，外部调用时接收这个返回值即可，这是最规范的做法。

修正后的代码：

function pswrd_gnrtr(length){
    var values = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789@#$";
    var result = "";
    for(var i=0; i<length; i++){
        // 修正索引：Math.random()*values.length 生成0到values.length-1的随机数，floor后就是合法索引
        result += values.charAt(Math.floor(Math.random() * values.length));    
    }
    return result; // 返回生成的密码
}

// 外部使用示例
var myPassword = pswrd_gnrtr(8); // 生成8位密码
console.log(myPassword); // 打印结果，也可以用alert(myPassword)

备选方案：使用全局变量（不推荐） #

如果一定要用全局变量，需要在函数外部声明result，并且函数内部不要用var重新声明它，直接赋值：

var result = ""; // 全局变量
function pswrd_gnrtr(length){
    var values = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789@#$";
    result = ""; // 直接给全局变量赋值，不要加var
    for(var i=0; i<length; i++){
        result += values.charAt(Math.floor(Math.random() * values.length));    
    }
}

// 外部使用示例
pswrd_gnrtr(8);
console.log(result);

注意 #

全局变量容易污染全局作用域，引发命名冲突，所以优先使用函数返回值的方式，代码更清晰、可控。

内容的提问来源于stack exchange，提问作者Aakash Rajbhar