Optimering af kode

måske du skal starte med at forklare hvad koden forventes at gøre. der mangler hvertfald en 'end if', men det er ikke helt tydelig hvor den skal stå.

jeg forstår hellere ikke pointet med at bruge 'Execute' ... hvad er det du ønsker at opnå ?

/nute

sorry - der mangler en end if her:

  end if
end if
next

meninger er at få eller de request.form("person_id_x") der er - ud i tilsvarende variabler, som str_person_id_x - for at kunne viderebehandle dem.

Sådan at jeg ender op med f.eks.

str_person_id_1 = 101
str_person_id_2 = 200

osv.....


Den her:

if eval("str_person_id_" & counter) = "" then
    ' en eller anden fejl - feltet er tomt
  else
    ' feltet er IKKE tomt - do your thing
  end if

- er bare for eksemplets skyld.

hmm ... så er det ikke så meget jeg ville ha optimeret koden (andet end at jeg ville have brugt arrays, istedet for at oprette variabler on the fly:

for each variable_name in request.form
x = x +1
if inStr( variable_name,"person_id") > 0 then
  counter = counter + 1
  Execute "str_" & variable_name & " = """ & request.form( variable_name ) & """"
  if eval("str_person_id_" & counter) = "" then
    ' en eller anden fejl - feltet er tomt
  else
    ' feltet er IKKE tomt - do your thing
  end if
end if
next

/nute

nute>> hvordan ville du have gjort det med arrays?

- som et 2-dimensionelt array eller?

Fx:

Dim arrVars()
counter = 1
for each variable_name in request.form
    if inStr( variable_name, "person_id") > 0 then
        Redim Preserve arrVars( 1, counter )
          arrVars( 0, counter ) = variable_name
        arrVars( 1, counter ) = Request.form( variable_name )
        counter = counter + 1
    end if
next
for i = 1 to UBound( arrVars, 2 )
    Response.Write( "Felt: <b>" & arrVars(0, i ) & "</b>, værdi: <b>" & arrVars(1, i ) & "</b><br>" )
next

Ser for den sags skyld ok ud - men jeg har så ikke variablerne til at arbejde videre med.

det er sant nok.

Jeg har dog aldrig forstået den helt store hensigt med dynamiske variable. jeg mener at hvis man ikke ved hvad ens variable hedder "i starten" af sin kode, så kan man hellere ikke vide hvad de hedder "i midten" af sin kode. Jeg synes arrays er meget bedre at arbejde med, for man ved altid hvad man har. Jeg har hellere aldrig oplevet nogen der bruger dynamiske variable, i mit professionelle liv, og mange af mine dygtige kolleger, samt den kode jeg læser hver dag, bruger det ikke. Nu kender jeg ikke til "de gode gamle sprog", men jeg har på følelsen af at dynamiske variable er noget der hører forhistorien til ... men det er bare mit ignorante bud :-)

/nute