Простой сортировщик HTML таблиц

[Octane]

Еще у меня из-за замыкания начинается паранойя:

proto = {
	method: function (…) {
		var a, b, c, d, …;
		Object.keys(…).forEach(function (…) {
			//к примеру, тут не нужны a, b, c, d
		});
	}
}

приходится выносить в отдельный метод:

proto = {
	_method: function (…) {
		//тут this → window
	},
	method: function (…) {
		var a, b, c, d, …;
		Object.keys(…).forEach(this._method);
	}
}

но метод в прототипе, ничего не делающий с this, кажется лишним, выносим его в отдельный объект:

utils = {
	method: function (…) {…}
}

proto = {
	method: function (…) {
		var a, b, c, d, …;
		Object.keys(…).forEach(utils.method);
	}
}

в итоге придется искать в коде, что же там за utils.method, который возможно состоит из пары строчек и самостоятельно не имеет смысла, появляется проблема с передачей дополнительных переменных.

а можно было всего-лишь написать:

proto = {
	method: function (…) {
		var a, b, c, d, …, keys = Object.keys(…), i = keys.length;
		while (i--) {
			…
		}
	}
}

поэтому forEach использую, только когда действительно нужен локальный скоп, а еще в светлом будущем ожидается let

[Kolyaj]

proto = {
    method: function (…) {
        var a, b, c, d, …, keys = Object.keys(…), i = keys.length;
        while (i--) {
            //к примеру, тут не нужны a, b, c, d
        }
    }
}

И в чём разница?

[Octane]

Ну так они никуда не замыкаются, паранойя не наступает

[Kolyaj]

А что меняется от того, замыкаются они или не замыкаются? Не пойму, откуда паранойя растёт.

[Octane]

Да ничего страшного не происходит. Я понимаю, что разница будет заметна только в тестах, где 100500 раз одно и тоже повторяют, но просто зачем все эти лишние действия (создание функции, связь скопов, call, …) будут происходить, если используя while, их не будет.

[float]

я тоже сначала заморачивался по этому поводу, а потом перестал, когда сравнил скорость...

[x-yuri]

Два года писал этот код,

с этим надо что-то делать... например, разобраться, все-таки, почему он не закончен. И что из этого всего стоило потраченных усилий, а что - нет

О, я много где сэкономил на производительности! :-) Но вы всё равно не найдёте. Да я и сам сейчас не вспомню всего.

в результате имеем код, который желательно не менять. Либо надо осознавать, что изменения могут свести на нет усилия по оптимизации. Комментарии тогда надо было делать для таких неочевидных моментов. Собственно, это их основное назначение

вот, float, это правильный подход при борьбе с паранойей, я считаю

[B~Vladi]

Да ничего страшного не происходит. Я понимаю, что разница будет заметна только в тестах, где 100500 раз одно и тоже повторяют, но просто зачем все эти лишние действия (создание функции, связь скопов, call, …) будут происходить, если используя while, их не будет.

О, я тоже параноик, круто.

[stopkran]

в результате имеем код, который желательно не менять. Либо надо осознавать, что изменения могут свести на нет усилия по оптимизации. Комментарии тогда надо было делать для таких неочевидных моментов

В точку. Любые изменения этого кода приводят к ухудшению производительности. Он находится в состоянии устойчивого равновесия. Вещь в себе. На данном этапе.

Но на следующем - возникает потребность не в совершенстве кода, а в новой функциональности. И маховик начинает раскручиваться заново: добавляем, например, раскраску строк в зависимости от значений, и оказывается, что простой алгоритм не способен решить эту задачу.

[x-yuri]

И маховик начинает раскручиваться заново:

вот поэтому я не люблю готовые визуальные компоненты. Наличие готового компонента хорошо, а умение создать такой же, знание подводных камней, еще лучше. Поэтому самое главное ты, на мой взгляд, упустил: ты не помнишь всех проблем с которыми ты сталкивался. Хотя не думаю, что все проблемы были действительно важны...