массив сортировки javascript

Question

Мой массив не сортируется должным образом. Может кто-нибудь сообщить мне, что я делаю не так?

...
 sortArray = new Array ("hello", "Link to Google", "zFile", "aFile");

//sort array
        if (dir == "asc") { 
            sortArray.sort(function(a,b){return a - b}); 
        } else { 
            sortArray.sort(function(a,b){return b - a});
        }

        for(var i=0; i<sortArray.length; i++) { 
            console.log(sortArray[i]);
        }

журнал показывает их в том же порядке, в котором они были введены.

Answer 1

Вы хотите сделать сравнение в своем роде, а не вычитание:

if (dir == "asc") {
    sortArray.sort(function(a, b) {
        a = a.toLowerCase();
        b = b.toLowerCase();
        return a === b ? 0 : a > b : 1 : -1;  
    });
} else {
    sortArray.sort(function(a, b) {
        a = a.toLowerCase();
        b = b.toLowerCase();
        return b === a ? 0 : b > a : 1 : -1;  
    });
}

Я также использовал toLowerCase(), чтобы 'Ссылка на Google' была размещена надлежащим образом.

РЕДАКТИРОВАТЬ: Обновлено, чтобы исправить проблему сравнения согласно комментарию.

См. Пример →

Answer 2

Проблема в том, что "a - b" обрабатывает строки как числа, что возвращает NaN.Вы получите поведение, которое вы ищете (при условии, что вы ищете сортировки с учетом регистра), если вы замените свои сортировки на:

    if (dir == "asc") { 
        sortArray.sort(function(a,b){return a < b ? -1 : 1}); 
    } else { 
        sortArray.sort(function(a,b){return b < a ? -1 : 1});
    }

Answer 3

Вы пытаетесь отсортировать, вычитая строки, к которым вы получите NaN.

Answer 4

Функции компаратора возвращают NaN, так как он получает две строки и выполняет вычитание, операция, которая не совсем определена для строк.

То, что вы должны иметь, это что-то вроде:

function(a,b){
   return a>b? 1 : (a<b ? -1 : 0);
}

или вы можете использовать localeCompare:

function(a,b){
   return a.localeCompare(b);
}

Не забудьте относиться к делу надлежащим образом, например, "L" < "a" пока "l" > "a"