Problemer med focus?

hvad gør den istedet ?

Den sætter ikke focus på noget

og hvad er der i variablen "navn" .. og findes det tilsvarende felt ?

ja for den første linie virker fint

prøv:

if (fejl == 1) {
        document.parameter.elements['antal_' + navn].value = document.parameter.elements['oldAntal_'+ navn].value;
        document.parameter.elements['antal_' + navn].focus();
        alert('Du må kun indtaste tal mellem 1 - 99');
        return false;
    }

Der sker stadig ikke noget

jeg tror problemmet er at din alert tager fokusset fra din boks. Prøv at bytte om på de to:
if (fejl == 1) {
        document.parameter.elements['antal_' + navn].value = document.parameter.elements['oldAntal_'+ navn].value;
alert('Du må kun indtaste tal mellem 1 - 99');       
document.parameter.elements['antal_' + navn].focus();
       
        return false;
    }

Det hjalp ikke, men det var vel også det jeg startede med at skrive

kunne du ikke skrive lidt mere af koden - umiddelbart ser det rigtig nok ud, især hvis du siger den første linie virker.

function beregn(navn){
   
    var stk = document.parameter.elements["antal_" + navn].value;
    var fejl = 0;

    if(!(stk.match(/^[0-9\+ ]+$/))) {
        fejl = 1;
    }
   
    stk = parseInt(stk);
   
    if(stk < 0 || stk > 99 ){
        fejl = 1;
    }

    if (fejl == 1) {
        document.parameter.elements['antal_' + navn].value = document.parameter.elements['oldAntal_'+ navn].value;
        alert('Du må kun indtaste tal mellem 1 - 99');
        document.parameter.elements['antal_' + navn].focus();
        return false;
    }
   
    var lokalPris = 0;
    var stkPris = 0;
    var oldStr = parseFloat(document.parameter.elements['oldStr_'+ navn].value);
    var oldAntal = parseFloat(document.parameter.elements['oldAntal_'+ navn].value);
    gebyr = <%response.write session("startGebyr")%>;       
   
    stkPris = findPris(document.parameter.elements['str_' + navn].value);
   
    oldPris = oldStr * oldAntal
    lokalPris =  stk * stkPris
    var tempPris = parseFloat(document.getElementById("pris").innerHTML);
    var pris = tempPris - oldPris;
    pris = pris + lokalPris
   
    antal = (antal - oldAntal) + stk;
   
    document.getElementById("antal").innerHTML = antal;   
   
    document.getElementById("pris").innerHTML = pris;       
       
    document.getElementById("prisIAlt").innerHTML = (pris + gebyr);
   
    document.parameter.elements['oldStr_'+ navn].value = stkPris;
   
    document.parameter.elements['oldAntal_'+ navn].value = stk;
   
    document.parameter.pris.value = pris;
   
    document.parameter.antal.value = antal;
}

tænkte nu også på selve formen du bruger, hvis du kunne give et eksempel evt. bare et link.

<input type="text" name="antal_23" size="1" value="0" onChange="java script:beregn('23');" onFocus="java script:this.select();" class="brod_lille">

<ole>

Der skal _aldrig_ bruges 'java script:' i en event-handler  :)

/mvh
</bole>

Hvorfor ikke det?

Fordi, en event-handler ikke kan andet end afvikle JavaScript - her er brugen af JavaScript-pseudo-protokollen helt misforstået.

Den er til brug i f.eks. href'en i et link - eller til direkte indtastning i adresselinien - men _ikke_ i en event-handler.
/mvh

Kan du så give et eksempel på hvordan jeg så kan trigge min funktion hvis ikke jeg skal bruge "onChange"?

du skal bare fjerne java script: - altså:
    <input type="text" name="antal_23" size="1" value="0" onChange="beregn('23');" onFocus="this.select();" class="brod_lille">

ja ok på den måde :-)

Men den fokusere stadig ikke på det rigtige felt

prøv at lave det til:

onChange="return beregn('23');" eller onblur="return beregn('23');"

Prøv med en URL til siden ... det begynder at minde lidt om Lykkehjulet, det her  :D
/mvh

enig med olebole. Det ville være lettere hvis vi kan se siden hvor det går galt for normalt er der ikke noget problem med focus() metoden.

jeg takker for hjælpen