как исправить jslint Подвыражение '&&' должно быть заключено в ошибку parens

Question

Я поставил все в скобки, но приведенный ниже код по-прежнему выдает ошибку в jslint:

Problem at line 5 character 104: The '&&' subexpression should be wrapped in parens.

if ((typeof (c1) === 'string') && (typeof (c2) === 'string') && (c1 !== n...

Как исправить?

"use strict";

function t() {
    var c1, c2;
    if (((typeof (c1)) === 'string') && ((typeof (c2)) === 'string') && (c1 !== null) && (c2 !== null) && ((c1.trim()) === '') || ((c2.trim()) !== '')) {
        return;
    }
}

Answer 1

Он жалуется на форму if(a && b && c || d), потому что (я полагаю) не сразу очевидно, будет ли && или || иметь приоритет.Исправьте это, чтобы выглядеть как if(a && b && (c || d)), и он перестанет жаловаться.

Answer 2

Я вполне уверен, вам нужно следующее :

function t() {
    var c1, c2;
    if (typeof c1 === 'string' && typeof c2 === 'string' && c1 !== null && c2 !== null && (c1.trim() === '' || c2.trim() !== '')) {
        return;
    }
}

Не все знают приоритет булевой логики, поэтому они хотят, чтобы вы заключили операторы c1.trim() || c2.trim() в скобкитак что понятно, как они работают.

В качестве примечания, я думаю, что смешно, что jslint хочет пробелы между моими операторами и моими операндами.Я думаю, что это гораздо яснее, когда нет пробела.

Answer 3

Я думаю, что это нужно:

    if (((typeof (c1) === 'string') && (typeof (c2) === 'string') && (c1 !== null) && (c2 !== null)) && ((c1.trim()) === '') || ((c2.trim()) !== '')) {

обернуть 4-х выраженных выражений слева от && в 100.