在 .js 文件中声明全局变量是不好的做法吗？

我有一个 .js 文件，我正在其中初始化两个使用的参数 在一个单独的函数中：

var submyvar1;
var submyvar2;

function init(myvar1 , myvar2){
    submyvar1= myvar1;
    submyvar2= myvar2;
}

function (){
    //subvar1 & subvar 2 used here
}

像这样声明全局变量是一种不好的做法吗？

如果是这样，有什么替代方法，将整个 .js 文件包装在一个对象中？

至少是不好练习时，你可以使用立即调用函数表达式:

(function() {
    var x;
    var y;

    window.init = function(xValue, yValue) {
        x = xValue;
        y = yValue;
    }

    window.doWhateverYouWant = function() {
        console.log(x + y);
    }
}());

你可以使用这个模式除非您需要在此 .js 文件之外访问全局变量，当 .js 文件之间交叉访问或<script>元素是必需的，全局变量是一个简单的解决方案（尽管您可以使用 AMD 或其他东西代替）。