Prévu une cession ou d'un appel de fonction et plutôt vu une expression

Je suis totalement cool avec ce JSLint erreur. Comment puis-je tolérer? Est-il un drapeau ou une case à cocher?

Vous obtenez quand vous faites des choses comme: 

v && arr.push(v);

par opposition à: 

if (v) {
    arr.push(v);
}

Les deux font exactement la même chose. Si vous mettez: 

window.test = function(v) {
    'use strict';
    var arr = [];
    if (v) {
        arr.push(v);
    }
    return arr;
};

dans le minifier il minifies à cela de toute façon:

window.test=function(a){var b=[];a&&b.push(a);return b};
InformationsquelleAutor ryanve | 2012-03-02
  1. 75

    Je ne pense pas que JSLint a une option pour désactiver cette fonction.

    JSHint (une fourche avec plus d'options) dispose d'une option pour elle, cependant: La expr option, documentés en tant que "si ExpressionStatement devraient être autorisés que des Programmes".

    • Vous pouvez également les télécharger et les utiliser localement (c'est ce que je fais). Je le lance via NodeJS avec mon préféré pour le réglage des options, de façon spectaculaire aide à ma confiance dans le code est correct. 🙂
    • jshint.com/docs/options/#expr pour sauver quelqu'un moment dans le futur
    • Peut décrire plus s'il vous plaît, pourquoi cette mise en garde qui s'est passé?
    InformationsquelleAutor T.J. Crowder
  2. 36

    Vous pouvez ajouter la ligne suivante à ignorer cet avertissement:

    /*jshint -W030 */

    Vous pouvez en lire plus à ce sujet ici.

    • La réponse est complètement étranger à la question, qui était d'environ JSLint, pas JSHint.
    • Mais son toujours les plus utiles réponse pour la majorité des gens qui débarquent ici.
    • "totalement indépendante"? Parce que JSHint est complètement étranger à JSLint?
    • Indépendants ou pas, c'est toujours la bonne réponse 🙂
    InformationsquelleAutor ReedD
  3. 9

    Il n'y a pas d'option pour cela dans JSLint. Vous pouvez le contourner en utilisant:

    var dummy = v && arr.push(v);

    NB: dummy évalue à true après.

    Une autre solution pourrait être:

    function expression(statement) { 
 'use strict';
 return statement; 
}
expression(v && arr.push);
    • Ça va marcher, mais c'est idiot. Le minifier seront effectivement en sorte que et mais pour des raisons de lisibilité, il serait mieux d'utiliser seulement la if déclaration que celle de l'OMI.
    • C'est vrai. Mais vous avez demandé une façon de tolérer l'est. A ajouté une autre solution de contournement. C'est comme Crockford a dit quelque part: jsLint est conçu pour être une douleur dans le cul pour les programmeurs ;~)
    • MAL le merveilleux && hack ne fonctionne pas dans jsLint... peut-être dépend de la version...
    InformationsquelleAutor KooiInc
  4. 1

    Les gens qui sont à la recherche, comment faire pour supprimer lors de l'utilisation de ESLint. Vous pouvez ingnore en écrivant le commentaire juste au-dessus de la ligne.

    //eslint-disable-next-line no-unused-expressions

    Vous pouvez également supprimer l'avertissement pour la totalité du fichier en plaçant le commentaire suivant au début du fichier

    /* eslint-disable no-unused-expressions */
    InformationsquelleAutor Shivam