CIO Tech Eksperten IT-JOB IT-Kurser Events Podcast Søg

Log ind eller opret profil

Du kan også logge ind via nedenstående tjenester

steen_hansen Forsker

28. august 2016 - 22:15 Der er 12 kommentarer

UPDATE med WHERE IN?

Jeg laver et udtræk på nogle brugere, som jeg kører gennem en løkke som vist nedenfor.

For at give formularfelterne unikke names og IDs, sætter jeg brugernes initialer ind bagved "name" og "id".

For ikke at skulle lave 200+ (9 formularfelter for hver bruger)

xxx='" & Request.Form("yyy") & "', ..............

i hånden, spekulerer jeg på, om man kan løse dette nemmere v.hj.a. WHERE IN?

Jeg håber du kan se hvordan jeg trækker alle brugere ud gennem én løkke. Det fungerer fint den vej. Tricket må være at kunne UPDAT nemmest muligt. Håber det kan lade sig gøre.

Og jeg håber mit spørgsmål og problemstillingen er til at forstå. Ellers spørg :)

<%

Set RSEast = Conn.Execute("SELECT * FROM [TABEL] ORDER BY initials ASC")

If Request.Form("mode") = "updatenow" Then
Dim allowdp
If LCase(Request.Form("allow") = "on" Then
allowdp = 1
Else
allowdp = 0
End If

SQL = "UPDATE [TABEL] SET [kolonne]='" & Request.Form("activity1<%=initEast%>") & "', allow=" & allowdp & " WHERE userID =" & Request.Form("foid") & ""

Response.Redirect("/............")

End If

%>

<html>

<body>

<%
If Not RSEast.EOF Then
Do Until RSEast.EOF

Dim initEast
initEast = LCase(RSEast("initials"))

%>

<form name="xxx" action="yyy" method="post">
<div><%=UCase(initEast)%></div>
<div><input type="hidden" name="foid" id="foid"></div>
<div>
<select size="1" name="activity1<%=initEast%>" id="activity1<%=initEast%>" class="selbox">
<option value="xxxx">.....</option>
</select>
</div>
<div><input type="text" name="act<%=<%=initEast%>" id="act<%=<%=initEast%>"></div>
<div><input type="checkbox" name="allow<%=initEast%>" id="allow<%=initEast%>" checked>
<div><input type="submit" name="xxx" value="yyy"></div>
<form>

<%

RSEast.MoveNext
Loop

End If

RSEast.Close
Set RSEast = Nothing

%>

</body>

Synes godt om

arne_v Ekspert

29. august 2016 - 02:05 #1

Hmmm - jeg kan ihvertfald ikke gennemskue hvad det er du vil.

Synes godt om

steen_hansen Forsker

29. august 2016 - 06:47 #2

Alle de her formular felter bliver rullet ud på én gang gennem løkken. Det fungerer fint.

Når så de felter, som skal udfyldes på brugerne, er udfyldt, håber jeg man kan opdatere tabellen på en enkel måde med WHERE in.

Synes godt om

steen_hansen Forsker

29. august 2016 - 06:47 #3

Jeg ved ikke om det giver mening

Synes godt om

Slater Ekspert

29. august 2016 - 08:51 #4

Du kan kun opdatere med WHERE IN hvis alle rækkerne skal have samme værdier.

Det sagt, så må der næsten være noget helt galt med din struktur, hvis alternativet er at sidde og skrive 200 opdateringslinjer i koden. Du må kunne gøre det, så det kan køre i en løkke.

Synes godt om

softspot Forsker

29. august 2016 - 09:14 #5

Hvis du gerne vil kunne rettet alle brugeres informationer inden du opdaterer, skal du ikke lave en form for hver bruger. Derimod skal du lave en form med alle felterne, dvs. form-elementet skal laves udenfor løkken (startes før do until og sluttes efter loop). Din gem/opdater-knap skal ligeledes ligge etfer loop (men før det lukkende form-element)

I din postback-kode skal du så finde de felter i formularen, som hører til hver bruger. Til dette har du behov for en liste af initialer, da det er disse du anvender til at identificere hver bruger med. Listen skal så gennemløbes i en løkken og for hver sæt initialer skal alle felter i forms-collection gennemsøges, som du selv er inde på med Request.Form("feltnavn" & initialer). Du skal blot hente de relevante felter og opdatere brugeren for hvert sæt initialer.

Du kan altså ikke opdatere alle på én gang med WHERE IN, da det er forskellige data for hver bruger.

P.S. Din kode til at generere formularen har syntaksfejl.

Synes godt om

steen_hansen Forsker

29. august 2016 - 22:06 #6

Slater: Som jeg umiddelbart ser det, er jeg næsten nødt til at gå den hårde vej: Nemlig at lave en Request.Form("xxxs") på hvert enkelt formularfelt. Jeg har så et spinkelt håb om, at der er en herfra, der ligger inde med de vises sten :o)

softspot: Det kan jeg ikke lige gennemskue:

Administrator indtaster data i formularfelt

name="[FORMULARFELT]1XXXXX"
name="[FORMULARFELT]2XXXXX"

osv

Den samme administrator indtaster data i formularfeltet for brugeren med initialerne YYYYY:

name="[FORMULARFELT]1YYYYY"
name="[FORMULARFELT]2YYYYY"

Dermed kan jeg kave formularfelterne unikke, selvom de bliver kørt gennem løkken.

Jeg tror ikke der er nogen syntaksfejl i udrulningsløkken. Der kommer ikke fejl, og de data jeg forsøgsvis har indtastet i databasen via PHPMyAdmin, bliver trukket fint ud i formularfelterne med

value="<%=RS("ditten%>"

og

value="<%=RS("datten")%>

når alle brugerne med deres respektive data køres gennem løkken. Alle de unikke data er placeret korrekt i de tilhørende formularfelter.

Her er det så jeg havde håbet, at man kan lave et eller andet á la:

UPDATE [TABEL] SET ditten='" & Request.Form("ditten"" & initials & "") & "', datten='" & Request.Form("datten"" & initials & """) & "' WHERE userID = foid .........

Jeg ved godt UPDATE er helt i hegnet. Det er bare for at indikere hvor det er jeg vil hen ad.

Ligesom jeg ruller formularfelterne med deres unikke værdier på de respektive brugere ud gennem en løkke, håber jeg at recordsets kan opdateres den anden vej på samme måde. Måske gennem en løkke også. Men jeg kan ikke gennemskue hvad det er du mener. Hvis du har mulighed for at lave et lille, hurtigt eksempel, ville det være fint.

<input type="hidden" name="foid" id="foid"> skulle gerne indeholde det unikke userID (Auto Increment, PRIMARY). Min tanke var, at det kunne bruges til et eller andet.

Synes godt om

softspot Forsker

29. august 2016 - 22:38 #7

Hvis du nu laver et felt til hver brugers initialer og kalder dette felt usrInitials for ALLE brugerne, vil initialerne blive sendt til serveren som en kommasepareret liste (hvilket så forudsætter, at du ikke anvender komma i dine initialer). Dermed kan du hente dette felt fra Request.Form("usrInitials") og omdanne indholdet til et array med split-funktionen (det er muligt du lige skal slå den faktiske syntaks for split op på nettet).

initialsArray = Split(Request.Form("usrInitials"), ",")

Array'et indeholder nu alle de initialer du skal bruge til at hente feltdata ud af forms-collection med, hvilket kunne se nogenlunde således ud:

for each initials in initialsArray
    felt1 = Request.Form("ditten" & initials) & ""
    felt2 = Request.Form("datten" & initials) & ""

    sql = "UPDATE table SET ditten = '" & ditten & "', " & _
          "datten = '" & felt2 & "' " & _
          "WHERE initials = '" & initials & "'"

    ' Logikken til at udføre din update mod 
    ' databasen laves her....
next

Bemærk at jeg anvender initialerne som nøgle til at opdatere brugeren med. Det betyder samtidig, at du ikke har behov for at oprette et id-felt i formularen til hver bruger.

NB: Jeg vil helt klart fraråde dig at anvende denne kode i et produktionsmiljø, da den er sårbar overfor SQL-injection. Jeg anvender metoden her, fordi du selv har anvendt den og dermed forventes at forstå hvad der sker. Du bør til hver en tid anvende Command og parameteriserede kald til databasen for at undgå denne SQL-injection sårbarhed

Synes godt om

steen_hansen Forsker

30. august 2016 - 00:03 #8

Det ser spændende ud!

"Hvis du nu laver et felt til hver brugers initialer......"

Det skal vel være

<input type="hidden" name="init" id="init" value="<%=RS("initials")%>">

Initialerne bliver udskrevet ud for hver bruger:

<div><%=Ucase("initials")%></div>

Men det er der kun for at administrator kan se initialerne. De skal jo ikke kunne rettes, da de er unikke og faste.

Synes godt om

steen_hansen Forsker

30. august 2016 - 00:15 #9

Rettelse:

<input type="hidden" name="usrInitials" id="usrInitials" value="<%=RS("initials")%>">

Synes godt om

arne_v Ekspert

30. august 2016 - 03:25 #10

Hvorfor bruger du ikke en traditionel simpel loesning:

visalle.asp som viser en liste/tabel med alle og har et edit link med id

retenkelt.asp som kaldes med id og har en enkelt form til at rette felter i

Synes godt om

softspot Forsker

30. august 2016 - 09:18 #11

Ja, initialerne er vel egentlig den rigtige primære nøgle til en bruger, idet de er entydige på tværs af alle brugere. Id er en teknisk primærnøgle, som i dette tilfælde (i princippet) er unødvendig.

Det er korrekt, at initialerne skal lægges ind i formularen som et skjult felt - selvom du også bare kunne bruge input-feltet til at vise initialerne og så gøre feltet readonly. Såvidt jeg husker, skal du blot tilføje readonly-attributten på input-feltet for at låse det for redigering.

Synes godt om

steen_hansen Forsker

11. september 2016 - 09:28 #12

Jeg har været optaget af andet arbejde, sorry. Jeg vender tilbage indenfor et par dage.

Synes godt om

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.

Følg dette spørgsmål

Opret Preview

Flere spørgsmål fra ASP kategorien

Titel	Indlæg	Oprettet	Seneste aktivitet
JSON Af ingeman i ASP	3	07/04/202417:53	08/04/202412:09
Vis detaljeret fejlkoder Af poulmadsen i ASP	6	09/03/202418:16	10/03/202417:59
IIS virker ikke på asp sider... Af bsn i ASP	10	15/10/202313:12	18/10/202308:48
Udregn afstand mellem 2 koordinater - virker ikke med variabler Af sorenmt84 i ASP	5	25/09/202321:26	26/09/202321:58
Refresh uden reloaf Af poulmadsen i ASP	7	19/07/202312:04	20/07/202311:12

Se alle spørgsmål i kategorien Opret spørgsmål

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

Alle kategorier på Eksperten

Seneste artiklerRSS