Uncaught Errore di sintassi, l'espressione non riconosciuto: [object Object]

Question

Lavorare currenly sulle notizie scroller - vedi il mio esempio vivo qui - EXAMPLE

Quando premo successivo precedente freccia/sto ricevendo un log degli errori Uncaught Syntax error, unrecognized expression: [object Object]

Perché il problema? Dov'è l'errore nella sintassi?

codice jQuery:

 (function($) { 
    /*! Scroller 
     ---------------------------------------------*/ 
     $.fn.Scroller = function() { 

      //Set height 
      $('.scroller').each(function() { 
       var height = 0; 
       $(this).children('div').each(function() { 

        if (height < $(this).height()) { 
         height = $(this).height(); 
        } 

       }); 
       $(this).css("height", height + "px"); 

      }); 

      $('.scroller').each(function() { 

       var NextArrow = $(this).parent().find('.next'); 
       var PrevArrow = $(this).parent().find('.prev'); 


       // Set a timeout 
       var timeOut = setTimeout(nextNotice, 5000); 

       // pause on hover 
       $(this).hover(

       function() { 
        clearTimeout(timeOut); 
       }, function() { 
        timeOut = setTimeout(nextNotice, 5000); 
       }); 

       // Next notice function called on timeout or click 
       //set a flag that use to be an oberver to listen when the fadeIn done 
       var flag = true; 

       function nextNotice(event) { 

        var CurrentScrollerDiv = $(this).parent().find('.scroller'); 

        if (!flag) { 
         return false; 
        } 
        clearTimeout(timeOut); 

        flag = false; 
        timeOut = setTimeout(nextNotice, 5000); 

        if ($(CurrentScrollerDiv + ' div:visible').is(CurrentScrollerDiv + ' div:last-child')) { 
         $(CurrentScrollerDiv + ' div:visible').fadeOut(300); 
         $(CurrentScrollerDiv + ' div:first-child').fadeIn("slow", function() { 
          flag = true; 
         }); 
        } else { 
         $(CurrentScrollerDiv + ' div:visible').fadeOut(300).next('div').fadeIn("slow", function() { 
          flag = true; 
         }); 
        } 
        return false; 
       } 

       $(NextArrow).click(nextNotice); 
       $(PrevArrow).click(function(event) { 

        var CurrentScrollerDiv = $(this).parent().find('.scroller'); 

        if (flag) { 
         return false; 
        } 
        clearTimeout(timeOut); 

        flag = false; 
        timeOut = setTimeout(nextNotice, 5000); 

        if ($(CurrentScrollerDiv + ' div:visible').is(CurrentScrollerDiv + ' div:first-child')) { 
         $(CurrentScrollerDiv + ' div:visible').fadeOut(300); 
         $(CurrentScrollerDiv + ' div:last-child').fadeIn("slow", function() { 
          flag = true; 
         }); 
        } 
        else { 
         $(CurrentScrollerDiv + ' div:visible').fadeOut(300).prev('div').fadeIn("slow", function() { 
          flag = true; 
         }); 
        } 
        return false; 

       }); 

      }); 

     }; 

    })(jQuery); 


    $(document).ready(function() { 
     //Blog 
     $('.itBlog > div:first-child').show(); 

     //Scroller 
     $('.scroller').Scroller(); 

    }); 

Answer 1

Costruire selettori da oggetti esistenti, utilizzare the second parameter of $:

$('div:visible', CurrentScrollerDiv) 

O the find function:

CurrentScrollerDiv.find('div:visible'); 

CurrentScrollerDiv non è una stringa in modo che non possa essere concatenato con una stringa per generare una base di stringhe d argomento selettore.

---

jQuery(selector, [ context ] ) 
    jQuery(selector, [context]) <-- you want this one, and 
    jQuery(element)     `selector` is a string 
    jQuery(elementArray) 
    jQuery(jQuery object) 
    jQuery() 
jQuery(html, [ ownerDocument ] ) 
    jQuery(html, [ownerDocument]) 
    jQuery(html,props) 
jQuery(callback ) 
    jQuery(callback) 

Answer 2

Questa è la linea problematica:

if ($(CurrentScrollerDiv + ' div:visible').is(CurrentScrollerDiv + ' div:last-child')) { 

Si utilizza concatenazione di stringhe in CurrentScrollerDiv, che .toString() s la variabile, che non è affatto ciò che si desidera. Non provare a stringare concatenare oggetti jQuery o elementi DOM. Utilizzare jQuery di .find() invece:

if (CurrentScrollerDiv.find('div:visible').is(CurrentScrollerDiv.find('div:last-child')) { 

V'è, tuttavia, quasi certamente un modo più efficiente per scrivere che if dichiarazione.

Answer 3

Ecco selettore sbagliato:

var CurrentScrollerDiv = $(this).parent().find('.scroller'); 

$(CurrentScrollerDiv + ' div:visible') 

correzione

var CurrentScrollerDiv = $(this).parent().find('.scroller'); 
$('div:visible', CurrentScrollerDiv);