jQuery prop() и компилятор Google Closure

Question

Я использую компилятор google закрытия, и я получаю потепление, которое я не понимаю. Мне нужно проверить, если переключатель проверяется или не так, у меня есть следующий код:

// ==ClosureCompiler== 
// @output_file_name default.js 
// @compilation_level ADVANCED_OPTIMIZATIONS 
// @externs_url http://closure-compiler.googlecode.com/svn/trunk/contrib/externs/jquery-1.8.js 
// ==/ClosureCompiler== 

function test() { 

    var TheBool = $('#SomeElement').prop('checked'); 

    if (TheBool === true) { 
    alert('checked'); 
    } 
} 

Я получаю предупреждение, что говорит, что это условие, которое всегда имеет значение ложь, даже если я знаю, что это не случай.

Вы можете попробовать это на http://closure-compiler.appspot.com/home и скопировать и вставить код, как я его (убедитесь, что флажок «Оптимизация: Advanced»)

Как мне сделать это предупреждение уйти?

Answer 1

Это, кажется, ошибка в externs file предоставлена ​​компанией Google.

Они неправильно объявляют jQuery.prototype.prop как возвращающий либо строку или jQuery, и игнорируют факт, что он может возвращать логическое;

/** 
* @param {(string|Object.<string,*>)} arg1 
* @param {(string|number|boolean|function(number,String))=} arg2 
* @return {(string|!jQuery)} 
*/ 
jQuery.prototype.prop = function(arg1, arg2) {}; 

... если это необходимо;

/** 
* @param {(string|Object.<string,*>)} arg1 
* @param {(string|number|boolean|function(number,String))=} arg2 
* @return {(string|boolean|!jQuery)} 
*/ 
jQuery.prototype.prop = function(arg1, arg2) {}; 

Я это исправил и uploaded it, а также при использовании этой декларации экстернов, ваша проблема решена;

// ==ClosureCompiler== 
// @output_file_name default.js 
// @compilation_level ADVANCED_OPTIMIZATIONS 
// @externs_url http://files.mattlunn.me.uk/permanent/jquery-1.8.js 
// ==/ClosureCompiler== 

function test() { 

    var TheBool = $('#SomeElement').prop('checked'); 

    if (TheBool === true) { 
    alert('checked'); 
    } 

} 

Вы сможете обойти это, не проверяя непосредственно === true; просто if (TheBool) будет достаточно.

---

FWIW, это было reported на своей странице вопросов и патч представил.

Answer 2

prop возвращает логическое значение. Поэтому нет необходимости проверять, есть ли это === true. Просто используйте:

if (TheBool) { 
    alert('checked'); 
} 

Чтобы сделать его более понятным при чтении, попробуйте:

var elementIsChecked = $('#SomeElement').prop('checked'); 
// `elementIsChecked` is either true or false 

if (elementIsChecked) { // `elementIsChecked` is coerced into a boolean, but it already is, so it doesn't matter 
    alert('checked'); 
} 

и, возможно, вы поймете, почему это не нужно, в первую очередь.

Выражения, используемые в if(), оцениваются как истинные или ложные, независимо от того, является ли их результат истинным или ложным. Это просто prop действительно возвращает true или false.