Question

在JSHint中运行此代码时，我收到了几个“未定义”错误：

MERLIN.namespace('MERLIN.http');

MERLIN.http = function ($, window) {
    'use strict';
    // import dependencies

    function request(config) {
        if (!config || typeof config !== 'object') {
            return;
        }
        // perform request
        $.ajax({
            type: config.type || 'GET',
            url: config.url,
            dataType: config.dataType,
            data: config.data || {},
            processData: config.process || false,
            beforeSend: function () {
                indicator(config.panel, config.indicator);
            },
            complete: function () {
                indicator(config.panel, config.indicator);
            },
            success: function (resp) {
                var callback = config.success || null;
                if (typeof callback !== 'function') {
                    callback = false;
                }
                if (callback) {
                    callback.apply(this, [resp]);
                }
            },
            error: function (xhr, textStatus, errorThrown) {
                httpError({
                    xhr: xhr,
                    status: textStatus,
                    error: errorThrown,
                    panel: config.panel
                });
            }
        });
    };

    function indicator(panel, type) {
        if ((!panel || typeof panel !== 'string') || (!type || typeof type !== 'string')) {
            return;
        }
        var indicatorType = (type === 'large') ? type = 'indicatorLarge' : type = 'indicatorSmall';
        return $(panel).toggleClass(indicatorType);
    };

    function httpError() {
        return this;
    };

    return {
        request: request,
        error: httpError
    };

} (jQuery, this);

我不确定为什么为'指示符'和'httpError'抛出未定义的错误以及为什么使用'return this'是一种潜在的严格违规行为。我知道我可以安全地忽略与命名空间相关的未定义错误，因为通用命名空间函数先前在单独的文件中定义。

这只是一个实用主义与严格验证的案例吗？

谢谢：）

Answer 1

关于'indicator' is not defined.和类似的错误：JSHint派生自JSLint，由Douglas Crockford编写。 Crockford有一个关于在定义之前出现在源文本中的函数调用的事情，即使它是完全正确和合法的代码并且没有任何歧义。我实际上认为这是JSLint（和启用该错误时的JSHint）主动反帮助的情况 - 我想知道什么时候真正未定义，而不是什么时候它是未定义的 - 根据 - Crockford's风格的规则。（不是我有意见。）

您可以通过将indicator和httpError的声明移到request以上来避免这些错误，但除了JSHint的错误错误之外，没有理由这样做。

关于return this;上的错误，我相信JSLint / JSHint告诉你你正在返回全局对象的方式，因为它期望以小写字母开头的函数只被称为函数和不是pseudo-methods。为什么httpError会返回this？你调用它的方式，this将是全局对象。

因此，虽然在这种情况下你正在返回全局对象是正确的，但你也可以完全虚假地得到那个错误。例如，此代码产生该错误：

var Foo = (function() {
    "use strict";

    function Foo() {

    }
    function bar() {
        return this; // "ERROR: [8:16]: Strict violation."
    }
    Foo.prototype.bar = bar;

    return Foo;
})();

没有任何严格的违规行为。 bar将返回this，如果我正确调用bar（例如var f = new Foo(); f.bar();）将是通过Foo创建的对象的实例，而不是全局对象

如果我更改了该代码，那么我就不会help my tools help me：

var Foo = (function() {
    "use strict";

    function Foo() {

    }

    Foo.prototype.bar = function() {
        return this;
    };

    return Foo;
})();

...错误消失了，因为JSLint / JSHint假设将this设置为除全局对象之外的其他东西来调用该函数。但后来我的功能是匿名的，这不太理想。

但是你可以通过使函数名称以小写字母以外的东西开头来使JSLint / JSHint高兴。例如，我通常的命名约定有效：

var Foo = (function() {
    "use strict";

    function Foo() {

    }
    function Foo$bar() {
        return this;
    }
    Foo.prototype.bar = Foo$bar;

    return Foo;
})();

没有产生错误。名称Foobar，Foo_bar和$bar也都可以使用。

JSHint在揭示模块模式时抛出“未定义”警告

1 个答案: