grote selectbox met zoekfunctie

[Verwijderd]

Ik heb een select boxje met daarin zo'n 25.000 elementen. Nou is het natuurlijk niet handig om daar zelf een in op te moeten zoeken, dus heb ik maar de volgende zoekfunctie gemaakt die doormiddel van een formulier, een select en een tekstvak gaat zoeken.

code:

    function zoek(formname, selectname, textname)
    {
        eval("var selectbox = document."+formname+"."+selectname+";");
        eval("var textbox = document."+formname+"."+textname+";");
        var zoek = new RegExp("^"+textbox.value,'i');
        for ( var i = 0; i < selectbox.options.length; i ++ )
        {
           if (selectbox.options[i].innerHTML.match(zoek))
             {
              selectbox.options[i].selected = true;
                    break;
             }
        }
    }

maar nu heb ik alleen het probleem dat het VEEL te lang duurt.
weet iemand misschien een snellere manier??

[ Voor 10% gewijzigd door Verwijderd op 09-01-2004 15:15 ]

[whoami]

Javascript hoort niet thuis in P&W, maar in W&G. Dit kan je in de forumdescriptions lezen.

Daarnaast kan je ook code-tags gebruiken om je code te formatten, en je post zo duidelijker te maken.
P&W -> W&G

[Verwijderd]

Oke...leek mij meer scripten dan design....

[Annie]

Probeer het eens zonder regular expression. Dus gewoon de string-waarde vergelijken.

JavaScript:

if (selectbox.options[i].text.toLowerCase() == zoek.toLowerCase())

Als je niet op de volledige string wil controleren kan je natuurlijk op een substring van de option-text matchen.

[ Voor 84% gewijzigd door Annie op 09-01-2004 17:44 . Reden: bugje in code :'( ]

[crisp]

een paar tips:

1) ditch die ranzige evals (juk!)
2) gebruik idd een string functie in dit geval ipv die regexp
3) in je for-loop vraag je nu elke iteratie weer de length property uit terwijl die niet veranderd gedurende de loop
4) vraag inderdaad de text property uit ipv de innerHTML

dan krijg je zoiets:

JavaScript:

function zoek(formname, selectname, textname) {

  var text = document.forms[formname].elements[textname].value.toLowerCase();
  var k = text.length

  var options = document.forms[formname].elements[selectname].options;
  var j = options.length;

  for (var i = 0; i < j; i++) {

    if (options[i].text.substr(0,k).toLowerCase() == text) {
      options[i].selected = true;
      break;
    }

  }

}

[Verwijderd]

thnx, ik ga het eens proberen!

edit:
das idd een stuk sneller!

[ Voor 40% gewijzigd door Verwijderd op 12-01-2004 13:22 ]