Stop den løkke

Sæt:

break;

ind i slutningen af din if.

Det ville selvfølgelig også virke, men jeg tænkte mere på, om ikke det var bedre programmering at ombygge/erstatte "for" og "if" med en anden form for lykke, hvis formål kun var at finde et enkelt objekt og så standse automatisk.

(I så fald at en sådan ikke findes, godtager jeg selvfølgelig dit svar)

du kunne også lave fx en boolan variabel uden for løkken der efterhånden sættes til true for fundet:

var OK = false
for (var i = 1;i < gruppe[1].length && !OK; i++)  {   
                            if ( gNum == gruppe[1][i].artikelnr ) {
                                  document.AdminA.artikelnr.value = gruppe[1][i].artikelnr;
                                  document.AdminA.titel.value = gruppe[1][i].titel;
                                  document.AdminA.forfatter.value = gruppe[1][i].forfatter;
                                  document.AdminA.beskrivelse.value = gruppe[1][i].beskrivelse;
                                  document.AdminA.filnavn.value = gruppe[1][i].filnavn;
                                  OK = true;
                              }
                          }

men det er på ingen måde en forbedring, og gør præcis det samme som mercur8's break.
Jeb ville bruge break.

for (var i = 1;i < gruppe[1].length && gNum != gruppe[1][i].artikelnr ; i++)  {   
                            if ( gNum == gruppe[1][i].artikelnr ) {
                                  document.AdminA.artikelnr.value = gruppe[1][i].artikelnr;
                                  document.AdminA.titel.value = gruppe[1][i].titel;
                                  document.AdminA.forfatter.value = gruppe[1][i].forfatter;
                                  document.AdminA.beskrivelse.value = gruppe[1][i].beskrivelse;
                                  document.AdminA.filnavn.value = gruppe[1][i].filnavn;
                              }
                          }

ok principielt det samme som jakoba's.

maddog: Så vil if sætningen da aldrig blive udført.

lightbrain: Hvis du vil lave en anden form for løkke skal du bruge en do .. while.

Min fejl. Ignorer ovenstående.

>...skal du bruge en do .. while.
Lad mig endelig se, hvordan det gøres.

lightbrain: Det var mig der vrøvlede mht. til maddog's indlæg. Det vil virke.

Ok, men er det ikke mere programmeringsmæssigt korrekt at bruge "do...while", når man kun skal finde et enkelt objekt?

Nææ ... jeg mener alle 3 forslag er lige korrekte.
Du vinder intet performance mæssigt, ved at anvende den ene frem for den anden.

Tag den mulighed, du bedst kan lide og overskue.

Ok, jeg forhøjer lige pointene, da indlæget er blevet en længere (end planlagt).

>>jakoba og =maddog=
Hvis I vil have nogle points, så tryk lige svar.

Afslutningsvis vil jeg gerne gøre lidt reklame med et andet indlæg, som - selvom det er placeret under 'ASP' - er blevet til et javascript spørgsmål. (I behøver kun at læse de allersidste kommentarer): http://www.eksperten.dk/spm/228028

var i = 1
while (gNum == gruppe[1][i].artikelnr) {     
  i++
}

document.AdminA.artikelnr.value = gruppe[1][i].artikelnr;
document.AdminA.titel.value = gruppe[1][i].titel;
document.AdminA.forfatter.value = gruppe[1][i].forfatter;
document.AdminA.beskrivelse.value = gruppe[1][i].beskrivelse;
document.AdminA.filnavn.value = gruppe[1][i].filnavn;

elegant miko. men det skal være
while (gNum != gruppe[1][i].artikelnr)

mikoalngelo: Du må mene gNum != gruppe[1][i].artikelnr

Og nu har jeg ikke megen forstand på javascript, men starter array indexering ikke ved 0? For så skal det jo være, var i = 0 også.

Ups, jo =maddog=

mercur8 Spørg lyshjerne om det...

mercur8> principielt jo, men for løkken starter også ved 1. Det kan være at index 0 er reserveret til labels f.x. Vi kender jo ikke den bagved liggende kode.

Super - tak for alle besvarelserne :-)

sæl tak do