如何提高這小小的一段代碼

Question

我只是想知道如何改進這個代碼：

pagination.children('li.active').prev().prev().prev().prev().prevAll('li.pgn').css("display","none"); 

我只是不喜歡.prev()鏈，我沒有找到一個方法來改善，但我敢肯定你知道;-)

編輯如何：

分頁是我ul其中含有大量的li一個誰擁有active類。

我想選擇li.active之前的所有li，但不是前面的前5個。

爲例（忘記pgn類）：

<li></li> 
<li></li> 
<li></li> 
<li></li> 
<li></li> 
<li></li> 
<li></li> 
<li class="active"></li> 

這將會給這樣的：

<li style="displat: none;"></li> 
<li style="display: none;"></li> 
<li></li> 
<li></li> 
<li></li> 
<li></li> 
<li></li> 
<li class="active"></li> 

Answer 1

可以使用:gt()選擇。

pagination.children('li.active').prevAll('li:gt(4)').hide(); 

http://jsfiddle.net/MrvDX/

Answer 2

試試這個

編輯

var pagination = $('ul'); 

var len = pagination.find('li.active').prevAll('li').length; 
pagination.find('li.png').slice(0 , len-5).css('display','none')​​​​​​​​​​​​​​​​​; 

WORKING FIDDLE

Answer 3

你可以嘗試這樣的事情......

var $prevAll = pagination.children('li.active').prevAll('li.pgn'); 
var prevCount = $prevAll.length; 

$prevAll.filter(function(index){ 
    return (prevCount - index < 5) 
}).hide() //instead of - .css("display", "none"); 

使用過濾方法並將函數傳遞給它。

http://api.jquery.com/filter/

這裏的工作的例子。

http://jsfiddle.net/aTzzK/2/

Answer 4

我覺得你這樣做是這樣的事實表明這樣一個事實，你的頁面/應用程序的設計可以提高。

爲什麼你需要找到與活動類的LI？你爲什麼不知道這個？當然，它是你的應用程序，首先產生標記。

看起來好像您試圖從標記中確定狀態，而我喜歡從標記中確定狀態。你應該有一個對象，告訴你選擇了LI的哪個索引。然後，您可以使用更簡單的jQ選擇器來選擇您感興趣的索引。