2015-10-05 73 views
0

我有一个自定义的比较器,用于排序某些字符串,该字符串按降序对字符串进行排序并不像预期的那样工作。JavaScript字符串的自定义排序比较器

var list = document.getElementById('mylist'); 
 

 
var items = list.childNodes; 
 
var itemsArr = []; 
 
for (var i in items) { 
 
    if (items[i].nodeType == 1) { // get rid of the whitespace text nodes 
 
    itemsArr.push(items[i]); 
 
    } 
 
} 
 
var sortOrder = "DESC"; 
 

 
itemsArr.sort(function(a, b) { 
 
    switch (sortOrder) { 
 
    case "ASC": 
 
     var val = a.innerHTML.localeCompare(b.innerHTML); 
 
     break; 
 
    case "DESC": 
 
     var val = a.innerHTML.localeCompare(b.innerHTML); 
 
     if (val === 1) 
 
     val = -1; 
 
     if (val === -1) 
 
     val = 1; 
 
     break; 
 

 
    } 
 
    return val 
 
}); 
 

 
/*itemsArr.sort(function(a, b) { 
 
    return a.innerHTML == b.innerHTML 
 
      ? 0 
 
      : (a.innerHTML > b.innerHTML ? 1 : -1); 
 
});*/ 
 

 
for (i = 0; i < itemsArr.length; ++i) { 
 
    list.appendChild(itemsArr[i]); 
 
}
<ul id="mylist"> 
 
    <li id="list-item3">Service</li> 
 
    <li id="list-item4">Vendor</li> 
 
    <li id="list-item2">Service_Type</li> 
 
    <li id="list-item1">Call_Date</li> 
 
    <li id="list-item3">Customer Sat Rating</li> 
 
    <li id="list-item4">TTA</li> 
 
    <li id="list-item2">Support_Rep</li> 
 
    <li id="list-item1">Source_State</li> 
 
    <li id="list-item3">Severity</li> 
 
    <li id="list-item4">Customer_Type</li> 
 
    <li id="list-item2">CALL ID</li> 
 
    <li id="list-item1">Resolved_Ontime</li> 
 
    <li id="list-item2">Product</li> 
 
    <li id="list-item1">Minutes_On_Phone</li> 
 
</ul>

对于不同的数据我整理早些时候它升序和降序的罚款。对于当前的html数据,它没有。有人可以显示我不正确的地方吗?

+1

你不应该有重复的ID,他们弄乱了DOM查找表。 –

+1

要用'.localeCompare()'进行降序排序,您只需要使用一元'-'运算符:'val = -val;' – Pointy

+0

在我的工作代码中,我没有重复的ID。这个html代码片段是用于stackoverflow的。 Still Ill编辑它 – phantomsays

回答

2

你将它设置为DESC,然后右后卫再次:

case "DESC": 
    var val = a.innerHTML.localeCompare(b.innerHTML); 
    if (val === 1) 
    val = -1; 
    if (val === -1) 
    val = 1; 
    break; 

将其更改为:

case "DESC": 
    var val = -1 * a.innerHTML.localeCompare(b.innerHTML); 
    break; 
+2

你不需要'*'运算符。只需将'-'一元运算符应用于返回值:'-a.innerHTML.localeCompare(b.innerHTML)' – Pointy

+0

我的不好!愚蠢的错误。感谢您指出了这一点 – phantomsays