Inclure jQuery utilisant JS Shorthand

Question

Donc je travaille sur un site, et nous devons vérifier si jQuery est défini avant de l'écrire sur la page. Je veux le convertir pour employer le raccourci conditionnel de JS, mais pour une raison quelconque quand je fais cela l'instruction n'exécute pas. Voici mon code:

Original (this works, tested): 

if (typeof jQuery == 'undefined') { 
    document.write('<script type="text/javascript" src="', JSPATH_REL_symc + 'jquery-1.4.2.min.js','"><\/script>'); 
} 

Cette version fonctionne parfaitement. Maintenant je l'ai remplacé avec ceci, et pour quelque raison je rencontre des problèmes:

New w/Shorthand: 

(typeof jQuery == 'undefined') ? document.write('<script type="text/javascript" src="', JSPATH_REL_symc + 'jquery-1.4.2.min.js','"><\/script>') : ''; 

Une idée de ce qui ne va pas avec cette syntaxe?

Merci.

Answer 1

Rien ne semble être faux à première vue. (La var JSPATH_REL_symc est définie ailleurs, oui?). Le code est un peu funky (mélange de concaténation de chaînes et plusieurs arguments à document.write - document.write(str, str2 + str3, str4)), mais cela est facilement corrigé. Aussi, vous devriez être en mesure de simplifier (et raccourcir) le code encore plus loin:

this.jQuery || document.write('<script src="' 
           + JSPATH_REL_symc 
           + 'jquery-1.4.2.min.js"><\/script>'); 

L'attribut type de la balise script est pas obligatoire, tous les navigateurs évalueront implicitement son contenu ou le fichier lié en JavaScript.

Un test rapide montre ce code en action: http://jsfiddle.net/ZWQqM/ (moins l'argument JSPATH, et en ajoutant un attribut d'événement « onload » à la balise de script pour démontrer qu'il est réussi.)

Answer 2

Votre code ressemble un peu Faux parce que vous ne faites rien dans la seconde partie du conditionnel et que cela provoque probablement une erreur de syntaxe.

Fondamentalement, quand la condition est fausse, je pense qu'il est l'évaluation à:

''; 

qui n'est pas valide. , Vous habituellement l'utiliser comme ceci:

var someString = (someTest == true) ? "Great" : "Bad"; 
document.write(someString); 

Puisque vous utilisez document.write l'intérieur du conditionnel; Je pense que vous avez deux options:

1. Déplacez le l'intérieur conditionnel du document.write ou
2. Supprimer les citations des marques de la deuxième partie de la

conditionnelle Essayez ceci:

// Option 1 
document.write((typeof jQuery == 'undefined') ? '<script type="text/javascript" src="', JSPATH_REL_symc + 'jquery-1.4.2.min.js','"><\/script>' : ''); 


// Option 2 -- EDIT: This apparently isn't valid syntax due to the : ; at the end. Thanks @LarsH 
(typeof jQuery == 'undefined') ? document.write('<script type="text/javascript" src="', JSPATH_REL_symc + 'jquery-1.4.2.min.js','"><\/script>') : ; 

Personnellement, je pense que la déclaration if est plus claire.

EDIT: Comme l'a déclaré Ryan dans le commentaire ci-dessous, ''; est valide, donc je ne suis pas tout à fait sûr pourquoi ce code particulier est de lancer une erreur. Pouvez-vous donner une capture d'écran ou une description de l'erreur?