How do I turn this conditional logic into a ternary expression? [closed]

Question

This question is unlikely to help any future visitors; it is only relevant to a small geographic area, a specific moment in time, or an extraordinarily narrow situation that is not generally applicable to the worldwide audience of the internet. For help making this question more broadly applicable, visit the help center.

Closed 8 years ago.

Here is small little jquery plugin that checks if the element passed in is an input element and if it is empty:

$.fn.inputIsEmpty = function() {
    if (!this.is('input:text')) {
        return false;
    } else {
        return $.trim($(this).val()).length == 0;
    }
};

The above logic works. So I tried turning this into a sweetened ternary expression:

$.fn.inputIsEmpty = function() {
    this.is('input:text') ? $.trim($(this).val()).length == 0 : false;
};

But it just returns undefined - what am I doing wrong?

Answer 1

I would make use of operator short circuiting:

return this.is('input:text') && $.trim(this.val()).length === 0;

If this.is('input:text') is false, the second part won't need to be evaluated, as false && any boolean will be false regardless of the other boolean's value. It acts like your if statement.

Answer 2

$.fn.inputIsEmpty = function() {
   return (this.is('input:text') ? ($.trim($(this).val()).length == 0) : false);
};