JQuery .each() 回调函数不会在每次迭代/循环上运行

Question

这是应该发生的事情。
1.获取点击链接的rel属性
2. 对于每个具有“条目”类的 div：
(i)获取它的“左”位置
(ii) 计算其外部高度
(iii) 遍历“a.tag_filter”的所有实例。如果它在“rel”中找到与最初单击的字符串相同的字符串，则将 1 添加到“V”并跳出循环。
(iv)如果循环后“V”等于 0，我们知道该“.entry”中不存在相同的标签，因此将其淡出。
(v)一旦淡出完成循环遍历淡出后的所有“.entry”并获取它们的“左”值。
(vi) 如果褪色条目的左值 = 当前 '.entry' 的左值，则将其重新定位到新的 'top' 值。

目前正在发生什么。
它遍历并淡出所有正确的“.entry”元素，只有在所有元素都淡出后，它才会重新定位剩余的“.entry”元素。

在每个元素淡出后，我希望重新定位循环运行，这样它基本上一次定位剩余元素，而不是一次定位所有元素。

这是我的代码编辑：

$('a.tag_filter').click(function(e){
        e.preventDefault();
        var selectTag = $(this).attr('rel');

    $('div.spotlight_entry_container_grid').each(function(i){
        var $entry = $(this);
        var tagArray = [];

        $('a.tag_filter', this).each(function(){
            tagArray.push ($(this).attr('rel'));
        }); 

        if($.inArray(selectTag,tagArray) == -1){
            var leftPos = $entry.css("left"); 
                    var topPos = $entry.css("top"); 

            $entry.fadeOut(1000, function(){
                var nextLeftPos;
                            var nextTopPos;

                $('div.spotlight_entry_container_grid:gt('+i+')').each(function(j) {   
                    var $laterEntry = $(this); 
                    nextLeftPos = $laterEntry.css("left");
                                nextTopPos = $laterEntry.css("top");
                    //we need to keep the entries in their columns.
                    //matching left values will do it. No need to animate left values.
                    if(leftPos == nextLeftPos){
                        $laterEntry.animate({ top: topPos});
                    }
                    }); 
            });
        }   
    });
    });

希望这是有道理的
任何帮助，将不胜感激！ 我可能正在做一些疯狂的事情，但我无法发现它。 谢谢

Answer 1

在这里

$('a.tag_filter', this).each(function(){
                        var curTag = $(this).attr('rel');
                        if(curTag == selectTag){
                                v++;
                                return false;
                        }
                }); 

在 $().each() 内部返回 false 会中断对包装集中每个元素的循环。

来自documentation

从 each 中返回 'false' 函数完全停止循环 通过所有元素（这是 就像在正常情况下使用“休息” 环形）。从内部返回“真” 循环跳到下一次迭代 （这就像使用“继续” 一个正常的循环）。

另外，为了提高性能，我建议在每个 each() 中缓存 $(this)，而不是多次引用它。

编辑：

进一步查看代码后，我认为以下应该这样做

$('a.tag_filter').click(function(e){

        // prevent the default anchor behaviour
        e.preventDefault();
        var selectTag = $(this).attr('rel');

        $('div.entry').each(function(i){
                var $entry = $(this);                  

                // get an array of the anchor tag rel attributes
                var tagArray = [];
                $('a.tag_filter', this).each(function() {
                        tagArray.push ($(this).attr('rel'));
                });

                // if we can't find the selected tag in the entries tags
                if ($.inArray(selectTag,tagArray) == -1) {        
                    var leftPos = $entry.css("left"); 
                    var topPos = $entry.css("top");                      

                    $entry.fadeOut(1000, function(){
                        var nextLeftPos;
                        var nextTopPos;

                        $('div.entry:gt('+i+')').each(function(j) {             
                            var $laterEntry = $(this); 
                            nextLeftPos = $laterEntry.css("left");
                            nextTopPos = $laterEntry.css("top");
                            // for the first element, set top and left to the faded out element values
                            if (j == 0) {                               
                                $laterEntry.animate({ top: topPos, left: leftPos });
                            }
                            // for later elements in the loop, ste the values to the previous element values
                            else {
                                $laterEntry.animate({ top: nextTopPos, left: nextLeftPos  });
                            }
                        });
                    });                                                                  
                }
         });  



 });

Answer 2

你不需要缓存 $(this)，jQuery 自动缓存 this 选择器用于函数回调。