Выбирайте элементы между A и B эффективно

Question

У меня есть таблица с большим количеством тд. Какой был бы самый эффективный способ выбрать выбор 1 для выбора 2 и все между ними?

При выборе 1 я даю выбранный td идентификатор и по выбору 2, я даю этому выбору другой идентификатор, чтобы впоследствии искать эти идентификаторы.

Текущий код:

// small helpers 
function _for(e,f) { var i, len=e.length; for(i=0;i<len;i++){ f(e[i]); }} 
function _id(e) { return document.getElementById(e); } 

// VARs 
var main = _id('monthTables'), 
    td = main.querySelectorAll('td'), 
    go = false, 
    stop = false, 
    i = 0, 
    s1i = 0, 
    s2i = 999; 

// Loop throught td's 
_for(td, function(e){ 

    if(e.id == 'sel1') { go = 1; s1i = i; } 
    if(e.id == 'sel2') { stop = 1; s2i = i; } 

    if(s1i < s2i && go) { 
     if(go) { e.classList.add('range'); } 
    } 

    if(stop) { go = 0; } 

}) // end loop 

---

Живой пример (выберите две даты):
http://cdn.rawgit.com/tommiehansen/lightRange/master/test.html

Answer 1

код хорошо, на мой взгляд, может быть, вы могли бы добавить несколько оптимизаций (например, остановка цикла при обнаружении второго выбранного идентификатора)

E Xample для цикла:

function _for(e,f) { 
    var i, len=e.length; 
    for(i=0;i<len;i++){ if(f(e[i]) === false) break; } 
} 

поэтому он будет ломаться, когда функция возвращает е false

// Loop throught td's 
_for(td, function(e){ 

    //... 

    if(stop) return false; 

}) // end loop 

Также я предлагаю вам использовать getElementsByTagName вместо querySelectorAll если вы хотите немного больше performance ..но может быть это злая микро оптимизация.

Другой оптимизацией было бы начать цикл из первого выбранного td, возможно storing the index, когда вы выберете td и используете это значение в tha т, вы отправили в качестве начального индекса