Строгого Нарушение использование этого ключевого слова и выявление шаблон модуля

Question

удается получить следующий пройти JSLint/jshint

/*jshint strict: true */ 
var myModule = (function() { 
    "use strict"; 

    var privVar = true, 
     pubVar = false; 

    function privFn() { 
     return this.test; // -> Strict violation. 
    } 

    function pubFn() { 
     this.test = 'public'; // -> Strict violation. 
     privFn.call(this); // -> Strict violation. 
    } 

    return { 
     pubVar: pubVar, 
     pubFn: pubFn 
    }; 

}()); 

myModule.pubFn(); 

Я понимаю, что это было вызвано использованием this в объявлении функции, но я читал что-то Крокфорд написал и он сказал, что нарушение предназначено для предотвращения глобального переменного загрязнения - но единственной глобальной переменной здесь является тот, который я явно определяю ... myModule. Все остальное хранится в непосредственной функциональной области, и я должен иметь возможность использовать this для ссылки на модуль.

Любые идеи, как я могу получить этот шаблон для прохождения?

Update: если я использую выражение функции вместо декларации, это похоже на работу, т.е.

var pubFn = function() { ... 

Я не поклонник этого формата, хотя, предпочитают иметь имя функции и именованные параметры ближе, и декларация выглядит/чувствует себя чище. Я, честно говоря, не понимаю, почему это бросает нарушение - нет причин для этого в этом шаблоне.

Answer 1

JSHint имеет option под названием validthis, который:

[...] подавляет предупреждения о возможных жестких нарушений, когда код работает в строгом режиме и вы используете this в не-конструктор функции [. ..], когда вы уверены, что ваше использование this действует в строгом режиме.

Используйте его в функцию, которая JSHint жалуется, что в вашем случае будет выглядеть следующим образом:

function privFn() { 
    /*jshint validthis: true */ 
    return this.test; // -> No Strict violation! 
} 

function pubFn() { 
    /*jshint validthis: true */ 
    this.test = 'public'; // -> No Strict violation! 
    privFn.call(this); // -> No Strict violation! 
} 

Казалось бы, как боль, чтобы указать, что в каждой функции, где это применяется, но если вы установите опцию в верхней части своей функции модуля, вы можете скрыть подлинных нарушений строгого режима от вас самих.

Answer 2

К сожалению, это предполагаемая ошибка для этой установки, поскольку jslint/jshint не знает, что функция, объявленная в глобальном контексте, должна быть позже использована как метод объекта.

Answer 3

Реальная проблема здесь состоит в том, что, если вы звоните privFn внутри контекста в модуля (внутри IIFE), this будет undefined, когда в строгом режиме; window если нет в строгом режиме. Увы, функция не сработает, если вызвана из IIFE.

Это потому, что функции не имеют владельца (объект), когда вызывается из внутри IIFE, тогда как возвращаемого объект модуля является владельцем функций, когда они вызываются из вне контекста IIFE, например, this === myModule при вызове myModule.pubFn().

Как строгий режим, так и JSHint/JSLint пытаются помочь вам, и вы никогда не должны просто игнорировать ошибки/предупреждения, созданные ими, но вместо этого выяснить, почему они предупреждают вас.

Если вы на 100% уверены, что privFn, pubFn и т. Д. Не будут вызываться нигде, кроме внешнего модуля, просто введите комментарий /*jshint validthis: true */ в любые функции, которые генерируют предупреждение. В качестве альтернативы, один комментарий в IIFE не позволит JSHint генерировать эту ошибку для любой функции внутри модуля.

---

Одна из многих возможных решений

Магазин объем this (в self в данном примере) явно относятся к модулю. Это покажет и обеспечит ваши намерения.

/*jshint strict: true */ 
var myModule = (function() { 
    "use strict"; 

    var privVar = true, 
     pubVar = false, 
     self = this; 

    function privFn() { 
     return self.test; 
    } 

    function pubFn() { 
     self.test = 'public'; 
     //privFn.call(this); // Will have no effect, as `privFn` does not reference `this` 
     privFn(); 
    } 

    return { 
     pubVar: pubVar, 
     pubFn: pubFn 
    }; 
}()); 

myModule.pubFn();