Avatar billede horus Nybegynder
26. juni 2009 - 13:26 Der er 4 kommentarer og
1 løsning

Omskrivning af kode der benytter jquery

Hejsa Eksperter,

Jeg har en stump kode, som gør brug af jquery. Jeg skal dog bruge det i en sammenhæng hvor jquery ikke er tilgængeligt, og har derfor brug for hjælp til at får det "oversat" til "classic" javascript - altså uden brug af jquery.

På forhånd tak,

Daniel


<snip>

    $.each( input , function( x , n ) {
        if( ! input[ x ].getElementsByTagName("DistanceMeters").length )
            return;
   
        Dist = parseFloat( input[ x ].getElementsByTagName("DistanceMeters")[0].firstChild.nodeValue );
       
        if( ! aMinDist || (aMinDist > Dist) )
            aMinDist = Dist;
           
        if( ! aMaxDist || (aMaxDist < Dist) )
            aMaxDist = Dist;

        var Hrt = input[ x ].getElementsByTagName("HeartRateBpm");
       
        if( Hrt.length ) {
            Hrt = Hrt[0].getElementsByTagName("Value");
            Hrt = parseInt( Hrt[0].firstChild.nodeValue );
               
            if( ! aMinHeart || (aMinHeart > Hrt) )
                aMinHeart = Hrt;
               
            if( ! aMaxHeart || (aMaxHeart < Hrt) )
                aMaxHeart = Hrt;
        }
       
        uTime = input[ x ].getElementsByTagName("Time")[0].firstChild.nodeValue;
        tStamp = GetTimeStamp( uTime );
       
        if( ! aMinTimeStamp || (aMinTimeStamp > tStamp) ) {
            aMinTimeStamp = tStamp;
            aMinTimeStr = uTime;
        }
           
        if( ! aMaxTimeStamp || (aMaxTimeStamp < tStamp) ) {
            aMaxTimeStamp = tStamp;
            aMaxTimeStr = uTime;
        }
    } );


</snip>
Avatar billede softspot Forsker
26. juni 2009 - 13:45 #1
Er det ikke bare $.each der skal udskiftes...?

for(var i = 0, l = input.length; i < l; i++) {
  (function( x , n ) {
        if( ! input[ x ].getElementsByTagName("DistanceMeters").length )
            return;
   
        Dist = parseFloat( input[ x ].getElementsByTagName("DistanceMeters")[0].firstChild.nodeValue );
       
        if( ! aMinDist || (aMinDist > Dist) )
            aMinDist = Dist;
           
        if( ! aMaxDist || (aMaxDist < Dist) )
            aMaxDist = Dist;

        var Hrt = input[ x ].getElementsByTagName("HeartRateBpm");
       
        if( Hrt.length ) {
            Hrt = Hrt[0].getElementsByTagName("Value");
            Hrt = parseInt( Hrt[0].firstChild.nodeValue );
               
            if( ! aMinHeart || (aMinHeart > Hrt) )
                aMinHeart = Hrt;
               
            if( ! aMaxHeart || (aMaxHeart < Hrt) )
                aMaxHeart = Hrt;
        }
       
        uTime = input[ x ].getElementsByTagName("Time")[0].firstChild.nodeValue;
        tStamp = GetTimeStamp( uTime );
       
        if( ! aMinTimeStamp || (aMinTimeStamp > tStamp) ) {
            aMinTimeStamp = tStamp;
            aMinTimeStr = uTime;
        }
           
        if( ! aMaxTimeStamp || (aMaxTimeStamp < tStamp) ) {
            aMaxTimeStamp = tStamp;
            aMaxTimeStr = uTime;
        }
    })(i,null);
}

Jeg ved ikke lige hvad parameteren n er til for, men jeg synes ikke den bliver brugt til noget, så jeg har bare sendt null med ind til funktionen i denne paremeter...
Avatar billede horus Nybegynder
26. juni 2009 - 14:55 #2
virker jo perfekt. TAK! Smider du lige et svar?
Avatar billede softspot Forsker
26. juni 2009 - 15:03 #3
Velbekomme :-)
Avatar billede softspot Forsker
26. juni 2009 - 15:06 #4
Det er jo i øvrigt nok spild af resurser at lave en function som kaldes for hver iteration. Det ville nok være mere naturligt blot at fjerne funktionen og indlejre funktionens kode i løkken:


for(var x = 0, l = input.length; x < l; x++) {
    if( ! input[ x ].getElementsByTagName("DistanceMeters").length )
        return;
   
    Dist = parseFloat( input[ x ].getElementsByTagName("DistanceMeters")[0].firstChild.nodeValue );
       
    if( ! aMinDist || (aMinDist > Dist) )
        aMinDist = Dist;
           
    if( ! aMaxDist || (aMaxDist < Dist) )
        aMaxDist = Dist;

    var Hrt = input[ x ].getElementsByTagName("HeartRateBpm");
       
    if( Hrt.length ) {
        Hrt = Hrt[0].getElementsByTagName("Value");
        Hrt = parseInt( Hrt[0].firstChild.nodeValue );
               
        if( ! aMinHeart || (aMinHeart > Hrt) )
            aMinHeart = Hrt;
             
        if( ! aMaxHeart || (aMaxHeart < Hrt) )
            aMaxHeart = Hrt;
    }
       
    uTime = input[ x ].getElementsByTagName("Time")[0].firstChild.nodeValue;
    tStamp = GetTimeStamp( uTime );
       
    if( ! aMinTimeStamp || (aMinTimeStamp > tStamp) ) {
        aMinTimeStamp = tStamp;
        aMinTimeStr = uTime;
    }
           
    if( ! aMaxTimeStamp || (aMaxTimeStamp < tStamp) ) {
        aMaxTimeStamp = tStamp;
        aMaxTimeStr = uTime;
    }
}
Avatar billede softspot Forsker
26. juni 2009 - 22:30 #5
Tak for point :-)
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester