Функция сортировки в javascript, необходимо изменить правило ортировки?

Question

[Антон]

//сортировка
              $.get(URL + '/stations.json', function (data) {
                   var j1=0;
                   $.each(data, function (j1) {
                       Stations[this.sindex] = this;
                       StationsSort[j1]=this;
                       j1++;
                   });
//                   for (var i = 0; i <= StationsSort.length; i++)  {
			var date = new Date();

			dstmp=date.getTime();
                     $.each(StationsSort, function() {
			lastMsg=Math.floor(dstmp/1000)-this.last_at;
//цвет сортировки
			style=''
			if (lastMsg<10800){
				style="style=\"background: #8affbb;\"";
			}
			StationsSort.sort(function(s1,s2) {
                    if (s1.sindex>s2.sindex)
                       return 1;
					if (s1.sindex<s2.sindex)
                       return -1;
                     else
                       return 0;
                   });	   
			$('#station-list').append("<div "+style+" class='st-btn' title='"+ EncodeHTML(this.station_name) + "' onclick=\"StationProcess(this, '" + EncodeHTML(this.sindex) + "')\">" + EncodeHTML(this.sindex) +' - ' + EncodeHTML(this.station_name) + "</div>");
//                   $('#station-list').append("<div class='st-btn' title='"+ EncodeHTML(this.station_name) +            "' onclick=\"StationProcess(this, '" + EncodeHTML(this.sindex)                                        + "')\">" + EncodeHTML(this.station_name) + "</div>")
                   });
               });

Это функция и часть кода сортировки списка.
Помогите разобраться в каком месте кода вызывается функция StationsSort.sort(function(s1,s2) ?
Вот здесь "StationsSort[j1]=this;" ?
Здесь есть функция сортировки по номеру "StationsSort.sort(function(s1,s2)" и окрашивания "if (lastMsg<10800)" в зависимости от параметра lastMsg. Задача, реализовать сортировку исходя из параметра lastMsg.
Написать в функции "StationsSort.sort(function(s1,s2)", что то типо "if (lastMsg<10800)" не срабатывает, так как видимо параметр lastMsg, нужно как то передать в функцию "StationsSort.sort(function(s1,s2)", а я этого не умею делать:(

Answer 1

[gleendo]

StationsSort.sort(function(s1,s2) {
 // some code
})

Это не функция. Функцией является сам метод sort(), которому можно передать функцию как параметр, которая будет указывать как сортировать элементы. А StationsSort является объектом который нужно отсортировать. И вызывается метод sort() в данном блоке кода, а не где то еще.

Для примера:

let arr = [5, 4, 7, 2, 1, 8, 6, 0];

arr.sort((a, b) => {
	if (a < b) {
  	return -1;
  } else if (a > b) {
  	return 1;
  } else {
  	return 0;
  }
}); // [0, 1, 2, 4, 5, 6, 7, 8]

PS. Для начала приведите код в нормальный вид, согласно какому нибудь code style. Сейчас же в вашем коде черт ногу сломит.
Сделайте хотя бы так:

$.get(URL + '/stations.json', function(data) {
  var j1 = 0;

  $.each(data, function(j1) {
    Stations[this.sindex] = this;
    StationsSort[j1] = this;
    
    j1++;
  });

  var date = new Date();
  dstmp = date.getTime();

  $.each(StationsSort, function() {
    lastMsg = Math.floor(dstmp / 1000) - this.last_at;
    style = '';

    if (lastMsg < 10800) {
      style = "style=\"background: #8affbb;\"";
    }

    StationsSort.sort(function(s1, s2) {
      if (s1.sindex > s2.sindex)
        return 1;
      if (s1.sindex < s2.sindex)
        return -1;
      else
        return 0;
    });

    $('#station-list').append("<div " + style + " class='st-btn' title='" + EncodeHTML(this.station_name) + "' onclick=\"StationProcess(this, '" + EncodeHTML(this.sindex) + "')\">" + EncodeHTML(this.sindex) + ' - ' + EncodeHTML(this.station_name) + "</div>");
  });
});

Comments

[Антон]

Да блин, в том то и дело, что он не мой. Вот я и не могу в нем разобраться:(
К тому же я не JS программист.
Я вас правильно понимаю, sort() это стандартная (то есть уже готовая) JS функция? А StationsSort это как бы переменная(то есть если бы это был Си:))) )?

[gleendo]

Антон sort() - это стандартный метод (функция) в JS. А StationsSort это не переменная, а объект, элементы которого нужно отсортировать

[Антон]

Хорошо, а как теперь еще и по цвету отсортировать или по значению lastMsg.
Вот как я это понял, но что- то не работает:(
$.get(URL + '/stations.json', function(data) {
var j1 = 0;

$.each(data, function(j1) {
Stations[this.sindex] = this;
StationsSort[j1] = this;

j1++;
});

var date = new Date();
dstmp = date.getTime();

$.each(StationsSort, function() {
lastMsg = Math.floor(dstmp / 1000) - this.last_at;
style = '';

StationsSort[j1] = lastMsg;

if (lastMsg < 10800) {
style = "style=\"background: #8affbb;\"";
}

StationsSort.sort(function(s1) {
//if (s1.sindex > s2.sindex)
if (s1 > 10800)
return 1;
//if (s1.sindex < s2.sindex)
if (s1 < 10800)
return -1;
else
return 0;
});

$('#station-list').append("" + EncodeHTML(this.sindex) + ' - ' + EncodeHTML(this.station_name) + "");
});
});

[gleendo]

Антон: Во первых, как я уже писал, оформляйте код нормально. Во вторых, сортировка по любому значению делается точно также.

let json = [{
  "color": "red",
  "lastMsg": 2000
}, {
  "color": "blue",
  "lastMsg": 1000
}, {
  "color": "green",
  "lastMsg": 1200
}, {
  "color": "yellow",
  "lastMsg": 3000
}];

console.log(`Sorting by color (Before):`);

for (let i = 0; i < json.length; i += 1) {
	console.log(`color: ${json[i].color}, lastMsg: ${json[i].lastMsg}`);
}

console.log(``);

json.sort((a, b) => {
	if (a.color < b.color) {
  	return -1;
  } else if (a.color > b.color) {
  	return 1;
  } else {
  	return 0;
  }
});

console.log(`Sorting by color (After):`);

for (let i = 0; i < json.length; i += 1) {
	console.log(`color: ${json[i].color}, lastMsg: ${json[i].lastMsg}`);
}

Sorting by color (Before):
// color: red, lastMsg: 2000
// color: blue, lastMsg: 1000
// color: green, lastMsg: 1200
// color: yellow, lastMsg: 3000

// Sorting by color (After):
// color: blue, lastMsg: 1000
// color: green, lastMsg: 1200
// color: red, lastMsg: 2000
// color: yellow, lastMsg: 3000

[Антон]

evg_: Код представленный вами, я более менее понимаю, но со своим сопоставить не могу:( К примеру в этом месте dstmp = date.getTime(); на сколько я понимаю, заносятся данные о времени в переменную dstmp, для последующей обработки. А в этом месте Stations[this.sindex] = this; данные берутся их массива. При этом на сколько я понял сам список формируется здесь $.each(data, function(j1) {
Stations[this.sindex] = this;
StationsSort[j1] = this;

j1++;
});

А это StationsSort[j1] = this; обращение к функции сортировки?
А вот, что это такое $.get(URL + '/stations.json', понять не могу?

И как корректно отправить lastMsg в функцию сортировки, как я понял, функция сортировки в данном коде рассчитана на обработку массива?

Не пойму где находится конечная переменная, та что уже выводит все результаты на дисплей так сказать?:)

Понимаю, что нужно изучать JS с ноля, но у меня к сожалению, на это совсем нет времени:( Может быть поделитесь ссылкой на годный урок как раз по теме сортировки?

[gleendo]

Антон: Вы меня вообще не слышите? Я вам в третий раз говорю, оформляйте код правильно. Если честно, то нет никакого желания сидеть и разбираться в том, что вам нужно. Вы сами поставленную задачу понимаете? В чем она заключается?
Где вы взяли данный код? Данный код очень простой, для тех, кто хоть немного разобрался с JS. Я делаю вывод что вы только "вчера" узнали про JS. Желательно бы начать с начала. Так у вас ничего нормального не получится. Это полный бред, то, чем вы сейчас занимаетесь.
Скиньте саму задачу. Хочу посмотреть чего вы добиваетесь. Возможно перепишу вам с нуля все в понятном виде.

Answer 2

[kn1ght_t]

за такой кодстайл хочется покалечить)

Comments

[Антон]

Согласен:(